Skip to content

fix(devspace): select the deployed pods and stop forwarding a port - #32

Merged
vitramir merged 1 commit into
mainfrom
fix/devspace-selector-and-ports
Jul 30, 2026
Merged

vitramir merged 1 commit into
mainfrom
fix/devspace-selector-and-ports

Conversation

@vitramir

Copy link
Copy Markdown
Contributor

The selectors named app.kubernetes.io/instance=authorization, but the umbrella labels every subchart with the release name — instance=agyn-platform. Nothing matched, so devspace dev failed on the patch hook with a jsonpath error against an empty list, and after that waited indefinitely for a pod its own selector could never find. Selecting by service name alone matches under either deployment.

Both places needed it: the AUTHORIZATION_SELECTOR used by the patch hook, and the dev: block's own labelSelector.

The host port forward goes with it — authorization is reached in-cluster, and binding 50051 on the developer's machine takes a port from whatever else wants it for no benefit.

Found while running authorization from source to verify the can_list_sandboxes model fix; without this, devspace cannot start this service at all.

The selectors named app.kubernetes.io/instance=authorization, but the
umbrella labels every subchart with the release name — instance is
agyn-platform. Nothing matched, so `devspace dev` failed outright on the
patch hook and then waited forever for a pod that its own selector could
never find. Selecting by service name alone matches either way.

The port forward is dropped with it: authorization is reached in-cluster,
and binding 50051 on the host takes a port from whatever else wants it
for no benefit.

Co-Authored-By: Claude <noreply@anthropic.com>
@vitramir
vitramir merged commit 741c5ee into main Jul 30, 2026
2 checks passed
@vitramir
vitramir deleted the fix/devspace-selector-and-ports branch July 30, 2026 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant