Skip to content

Add get verb to pods/attach in Connect service account role for new kubernetes backend#875

Merged
dbkegley merged 1 commit into
mainfrom
kegs-fix-k8s-ws-rbac
May 29, 2026
Merged

Add get verb to pods/attach in Connect service account role for new kubernetes backend#875
dbkegley merged 1 commit into
mainfrom
kegs-fix-k8s-ws-rbac

Conversation

@dbkegley
Copy link
Copy Markdown
Contributor

@dbkegley dbkegley commented May 29, 2026

Bump rstudio-connect chart version to 0.20.3 and update NEWS.md

Update the rstudio-connect chart service account RBAC to allow websocket connections which are used by the kubernetes backend to execute jobs remotely on k8s. Connect 2026.04 correctly falls back to SPDY on websocket connection upgrade failures. The missing RBAC ensured that all connections were falling back to SPDY.

Connect 2026.05.0 introduced a regression which prevented fallback to SPDY, meaning all steaming connections now fail (exec/attach). This PR allows the initial websocket connection to succeed. The next release of Connect will fix the SPDY fallback

@dbkegley dbkegley requested a review from a team as a code owner May 29, 2026 15:38
Comment thread charts/rstudio-connect/NEWS.md Outdated

## 0.20.3

- Add `get` on `pods/attach` to the direct Kubernetes runner Role in `templates/rbac.yaml` to support attach/exec authorization checks used by Kubernetes clients.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we include a note in the NEWS about websocket support being the reason why we're adding this permission?

Comment on lines 109 to 112
resources:
- "pods/exec"
verbs:
- "create"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will we also need get for pods/exec for killing jobs?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks!

Bump rstudio-connect chart version to 0.20.3 and update NEWS.md
@dbkegley dbkegley force-pushed the kegs-fix-k8s-ws-rbac branch from 62a5068 to 0c1ada6 Compare May 29, 2026 16:02
Copy link
Copy Markdown
Contributor

@lucasrod16 lucasrod16 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dbkegley dbkegley merged commit 7a1858d into main May 29, 2026
8 checks passed
@dbkegley dbkegley deleted the kegs-fix-k8s-ws-rbac branch May 29, 2026 17:54
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.

3 participants