mirror of
https://github.com/fosrl/pangolin.git
synced 2026-07-16 06:08:33 -05:00
[GH-ISSUE #2125] public-resources and private-resources dont trigger blueprint updates #38870
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @angelrwu on GitHub (Dec 19, 2025).
Original GitHub issue: https://github.com/fosrl/pangolin/issues/2125
Describe the Bug
If you make a blueprint that uses
public-resourcesinstead ofproxy-resourcesorprivate-resourcesinstead ofclient-resources. The pangolin blueprint updater doesn't trigger and will output that an empty list of items was successfully updated "[]"I believe the bug is related to this section https://github.com/fosrl/pangolin/blob/main/server/lib/blueprints/applyNewtDockerBlueprint.ts#L37-L40 or this https://github.com/fosrl/pangolin/blob/main/server/lib/blueprints/parseDockerContainers.ts#L54-L61
, but I'm unsure.
Environment
To Reproduce
Insert a minimal blueprint with public-resources key or private-resources to get the bad case, and to get the happy path replace those for client and proxy.
Set the logging to debug to see the console output for the newt blueprint message and the pangolin blueprint update message
examples:
bad output when public and private resource are the key in blueprint
good output when proxy and client resource are the key in blueprint
Expected Behavior
Public and private resources key should trigger Pangolin to do resource updates or additions.
Alternatively the docs should be updated not to use public and private and use proxy and client, which would match the UI.
@djcrafts commented on GitHub (Dec 21, 2025):
@angelrwu Hey! I've opened PR #2132 that should fix this issue.
The problem was that the Docker label parser and early-exit check only looked for proxy-resources and client-resources, ignoring the public-resources and private-resources aliases. The fix extends both to recognize all four resource types, and the existing ConfigSchema transformation handles merging them correctly.
Changes are minimal and backward compatible - existing labels will continue to work as before.