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

Update rbac-for-monitoring.md #1104

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

armagankaratosun
Copy link

inheriting the "View Monitoring" role is not enough to access the monitoring ui components. additional access to services/proxy and endpoints is needed.

Fixes #[issue_number]

Reminders

  • See the README for more details on how to work with the Rancher docs.

  • Verify if changes pertain to other versions of Rancher. If they do, finalize the edits on one version of the page, then apply the edits to the other versions.

  • If the pull request is dependent on an upcoming release, make sure to target the release branch instead of main.

Description

Comments

inheriting the "View Monitoring" role is not enough to access the monitoring ui components. additional access to services/proxy and endpoints is needed.
@martyav
Copy link
Contributor

martyav commented Feb 5, 2024

@gunamata @mantis-toboggan-md @gaktive
You had previously reviewed #964, which relates to the view monitoring role. Could you have a look at this PR?

@gunamata
Copy link
Contributor

gunamata commented Feb 7, 2024

@prachidamle , As I see you handling the monitoring & logging charts in the recent org, Is this something you or someone on your team can take a look and provide some feedback please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engineering-assistance-needed sync-versions Reminder to sync versioned pages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants