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

Repos global list on additional clusters #5466

Merged
merged 2 commits into from
Oct 17, 2022
Merged

Conversation

castelblanque
Copy link
Collaborator

@castelblanque castelblanque commented Oct 17, 2022

Signed-off-by: Rafa Castelblanque [email protected]

Description of the change

This PR adds the list of global repos when context is set to a cluster other than the default one.
List is set on read only mode.
Same UI warning message is kept, as it still describes correctly the situation.

image

Targeted at the case stated in #2717, users that do not have access to the main/default cluster where Kubeapps is installed.

Benefits

  • Users in non-default cluster can see the list of global repositories, at least in read-only mode.

Possible drawbacks

N/A

Applicable issues

Rafa Castelblanque added 2 commits October 17, 2022 10:21
Signed-off-by: Rafa Castelblanque <[email protected]>
Signed-off-by: Rafa Castelblanque <[email protected]>
@netlify
Copy link

netlify bot commented Oct 17, 2022

Deploy Preview for kubeapps-dev canceled.

Name Link
🔨 Latest commit 29118bb
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/634d1d205741b4000976367b

@@ -172,6 +196,7 @@ function PkgRepoList() {
namespace={currentNamespace}
helmGlobalNamespace={helmGlobalNamespace}
carvelGlobalNamespace={carvelGlobalNamespace}
disabled={!supportedCluster}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Also disabled the "Add package repository" button if the cluster is not the default one.

Copy link
Contributor

@antgamdia antgamdia left a comment

Choose a reason for hiding this comment

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

Great, having a read-only view of the global repos is a good idea.

What about adding a button to "manage repos in the main cluster" (or sth like that), to let users access to the management view more quickly ?

@castelblanque
Copy link
Collaborator Author

What about adding a button to "manage repos in the main cluster" (or sth like that), to let users access to the management view more quickly ?

Do you mean adding a button like that close the current "Add package repository" (top right)?
I would do it as part of a separate issue, because we should also check upfront which access level the user has. e.g. not show it if the Kubeapps cluster is not accessible to the user, or any needed RBAC is missing for the user.

@castelblanque castelblanque merged commit cf90f8a into main Oct 17, 2022
@castelblanque castelblanque deleted the 2717-repos-mgmt-screen branch October 17, 2022 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inconsistent message if viewing app repositories on non-kubeapps cluster
3 participants