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

Unable to download cli config for non-admin users #274

Open
2 tasks
niravparikh05 opened this issue Nov 15, 2023 · 3 comments · May be fixed by #313
Open
2 tasks

Unable to download cli config for non-admin users #274

niravparikh05 opened this issue Nov 15, 2023 · 3 comments · May be fixed by #313
Assignees
Labels
bug Something isn't working good first issue Good for newcomers
Milestone

Comments

@niravparikh05
Copy link
Contributor

Expected vs actual behavior

  • User should be able to download cli config

Steps to reproduce the bug

  1. Create a user ( mostly observed for non-admin users )
  2. Login to dashboard as that user
  3. Click on download cli config from tools system menu

Are you using the latest version of the project?

You can check your version by running helm ls|grep '^<deployment-name>' or using pctl, pctl version, and provide the output.

What is your environment setup? Please tell us your cloud provider, operating system, and include the output of kubectl version --output=yaml and helm version. Any other information that you have, eg. logs and custom values, is highly appreciated!

(optional) If you have ideas on why the bug happens or how it can be solved, please provide it here

  • I've described the bug, included steps to reproduce it, and included my environment setup with all customizations.
  • I'm using the latest version of the project.
@niravparikh05 niravparikh05 added bug Something isn't working needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Nov 15, 2023
@nirav-rafay nirav-rafay added this to the v0.2.8 milestone Jan 19, 2024
@nirav-rafay nirav-rafay added the good first issue Good for newcomers label Jan 19, 2024
@Bharadwajshivam28
Copy link

I can take on this issue but I need some more details and steps to get started

@niravparikh05 niravparikh05 self-assigned this Mar 4, 2024
@akshay196
Copy link
Member

Got this error:
image

@akshay196 akshay196 removed the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Mar 12, 2024
@niravparikh05
Copy link
Contributor Author

niravparikh05 commented Mar 21, 2024

@Bharadwajshivam28 Issue is with missing permissions across non-admin roles, we should create a new permission file with below and add this to all non-admin roles in roles.json under ztka. Will you be able to take care of it ?

{ "name": "cli.config.read", "resource_urls": [ { "url": "/cli/config", "methods": [ "GET" ] } ], "base_url": "/auth/v3", "description": "CLI Configuration", "authenticated": true, "scope": "PROJECT" }

In addition to above, ensure it works for existing installations as well ( we may need db scripts for these as initialization module currently does not take care of adding these new permissions ).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants