Skip to content

feat: add session label controller for Workbench session pods#217

Open
amdove wants to merge 10 commits intomainfrom
labels-controller
Open

feat: add session label controller for Workbench session pods#217
amdove wants to merge 10 commits intomainfrom
labels-controller

Conversation

@amdove
Copy link
Copy Markdown
Collaborator

@amdove amdove commented Apr 8, 2026

Description

PTD support for the session label injection feature. See posit-dev/team-operator#123 for the full solution description.

Changes in this repo:

  • team_operator.py: auto-enables the controller in Helm when any site's site.yaml has spec.workbench.sessionLabels configured — no manual flag needed
  • aws_workload_helm.py / azure_workload_helm.py: adds user-group-* to the kube-state-metrics metricLabelsAllowlist so injected labels are exposed as Prometheus metrics
  • docs/CONFIGURATION.md: sessionLabels schema reference
  • docs/team-operator/README.md: usage guide
  • Justfile: fix symlink-binaries to skip binaries already in .local/bin (prevents circular symlinks); add codesign step after CLI build for macOS

Category of change

  • New feature (non-breaking change which adds functionality)

Checklist

  • I have reviewed my own diff and added inline comments on lines I want reviewers to focus on or that I am uncertain about

@amdove amdove marked this pull request as ready for review April 14, 2026 20:29
@amdove amdove requested a review from a team as a code owner April 14, 2026 20:29
@amdove amdove requested a review from stevenolen April 14, 2026 20:30
stevenolen
stevenolen previously approved these changes Apr 14, 2026
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.

2 participants