Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing permission to watch node object #11407

Merged
merged 1 commit into from
Jul 15, 2022

Conversation

jenting
Copy link
Contributor

@jenting jenting commented Jul 15, 2022

Description

We missed the add watch node clusterrole permission to ws-manager.

Related Issue(s)

We missed adding the watch node permission in PR #11337.

How to test

Release Notes

None

Documentation

None

Werft options:

  • /werft with-preview

Signed-off-by: JenTing Hsiao <hsiaoairplane@gmail.com>
@jenting jenting requested a review from a team July 15, 2022 08:53
@github-actions github-actions bot added the team: workspace Issue belongs to the Workspace team label Jul 15, 2022
@roboquat roboquat merged commit 21642a9 into main Jul 15, 2022
@roboquat roboquat deleted the jenting/add-watch-node-permission branch July 15, 2022 13:34
@roboquat roboquat added deployed: workspace Workspace team change is running in production deployed Change is completely running in production labels Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed: workspace Workspace team change is running in production deployed Change is completely running in production release-note-none size/XS team: workspace Issue belongs to the Workspace team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants