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

Only read access is necessary to view #123

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

ralphbean
Copy link
Member

We have three roles: admin, maintainer, and contributor.

The contributor role grants you read-only verbs.

As this was previously coded, having contributor rights in a namespace would not be sufficient for it to show up in the workspace switcher.

We have three roles: admin, maintainer, and contributor.

The contributor role grants you read-only verbs.

As this was previously coded, having contributor rights in a namespace
would not be sufficient for it to show up in the workspace switcher.

Signed-off-by: Ralph Bean <[email protected]>
@yftacherzog
Copy link
Member

yftacherzog commented Feb 9, 2025

@ralphbean are we still using workspace-manager somewhere?

@ralphbean
Copy link
Member Author

Oh, perhaps not?

I found my way here by grepping through infra-deployments and guessing.

What I'm really looking for is the thing that's responsible for serving the workspaces/ API response on the rh02 cluster.

It correctly serves two namespaces for me where I have admin rbac, but it does not serve a third in its response where I have only a "contributor" read-only role.

@ralphbean
Copy link
Member Author

In infra-deployments, I see:

components/konflux-ui/production/base/proxy/proxy.yaml:      - image: quay.io/konflux-ci/workspace-manager@sha256:6b7bfcf197cbff2ae9b1d84abd73636ce5733921524fb550438d8a8143a536b5

@yftacherzog
Copy link
Member

In infra-deployments, I see:

components/konflux-ui/production/base/proxy/proxy.yaml:      - image: quay.io/konflux-ci/workspace-manager@sha256:6b7bfcf197cbff2ae9b1d84abd73636ce5733921524fb550438d8a8143a536b5

Oh. Was not expecting that in production anywhere.
After my last comment @gbenhaim mentioned to me that it's indeed still used (I thought in the Fedora cluster), and is expected to be deprecated in a few weeks.

@ralphbean ralphbean merged commit dd70fdc into konflux-ci:main Feb 10, 2025
7 checks passed
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