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.

Sign up for GitHub

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

Jump to bottom

Docs don't mention that read-only plugin access is required for ACL volumes #18527

Closed
Zygimantass opened this issue Sep 18, 2023 · 2 comments · Fixed by #20557
Closed

Docs don't mention that read-only plugin access is required for ACL volumes #18527

Zygimantass opened this issue Sep 18, 2023 · 2 comments · Fixed by #20557
Assignees
@tgross
Labels
theme/acl theme/docs Documentation issues and enhancements theme/storage type/bug
Projects
Nomad - Community Issues Triage

Comments

@Zygimantass
Copy link

Issue

Documentation does not mention that read-only plugin access plugin { policy = "read" } is required for ACL volumes (like it does for UI + node / agent read-only access)
@tgross tgross added this to Needs Triage in Nomad - Community Issues Triage via automation Sep 20, 2023
@schmichael schmichael added the theme/docs Documentation issues and enhancements label Sep 21, 2023
@schmichael schmichael moved this from Needs Triage to Needs Roadmapping in Nomad - Community Issues Triage Sep 21, 2023
@lgfa29
Copy link
Contributor

lgfa29 commented Oct 5, 2023

Hi @Zygimantass 👋

Would you mind expanding a little bit more on what's missing? If possible with a link to the page and a small description of what's missing.

Thanks!

@lgfa29 lgfa29 self-assigned this Oct 5, 2023
tgross added a commit that referenced this issue May 10, 2024
@tgross
docs: note that plugin policy is required in the UI for CSI volumes
6bcfcbc 
The ACL docs have a section explaining that some parts of the UI need slightly
wider read permissions than expected. These docs should include that you need
`plugin:read` to look at CSI volume pages in the UI.

Fixes: #18527
@tgross tgross mentioned this issue May 10, 2024
Merged
@tgross tgross assigned tgross and unassigned lgfa29 May 10, 2024
@tgross tgross moved this from Needs Roadmapping to In Progress in Nomad - Community Issues Triage May 10, 2024
@tgross
Copy link
Member

tgross commented May 10, 2024

Fixed in #20557

tgross added a commit that referenced this issue May 10, 2024
@tgross
docs: note that plugin policy is required in the UI for CSI volumes (#…
1251c1d 
…20557)

The ACL docs have a section explaining that some parts of the UI need slightly
wider read permissions than expected. These docs should include that you need
`plugin:read` to look at CSI volume pages in the UI.

Fixes: #18527
Nomad - Community Issues Triage automation moved this from In Progress to Done May 10, 2024
This was referenced May 10, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tgross tgross

Labels
theme/acl theme/docs Documentation issues and enhancements theme/storage type/bug
Projects
Nomad - Community Issues Triage
Status: Done
Nomad - Community Issues Triage
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs: note that plugin policy is required in the UI for CSI volumes hashicorp/nomad
4 participants
@schmichael @lgfa29 @tgross @Zygimantass