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

ACL: Expose cluster's ACLs (read/only) #754

Closed
germanosin opened this issue Jul 30, 2021 · 6 comments · Fixed by #2818 or #3662
Closed

ACL: Expose cluster's ACLs (read/only) #754

germanosin opened this issue Jul 30, 2021 · 6 comments · Fixed by #2818 or #3662
Assignees
@iliax @NeiruBugz
Labels
scope/backend scope/frontend status/accepted An issue which has passed triage and has been accepted type/feature A new feature
Projects
Release 0.7
Milestone
0.7

Comments

@germanosin
Copy link
Contributor

Backend should provide list of ACL lists per resource group.
Frontend should display these lists with resource group type filter
@germanosin germanosin added type/enhancement En enhancement to an already existing feature scope/backend scope/frontend labels Jul 30, 2021
@germanosin germanosin added this to the 0.3 milestone Jul 30, 2021
@germanosin germanosin modified the milestones: 0.3, 0.4 Sep 16, 2021
@Haarolean Haarolean added this to 0.4 in Roadmap Oct 25, 2021
@Haarolean Haarolean added this to To do in Release 0.4 Jan 18, 2022
@jonas-grgt
Copy link
Contributor

Created API proposal for this feature, #1511

@Haarolean Haarolean linked a pull request Feb 1, 2022 that will close this issue
[ISSUE 754] Define listing of Acls API #1511
Closed
13 tasks
@Haarolean Haarolean added type/feature A new feature and removed type/enhancement En enhancement to an already existing feature labels Feb 1, 2022
@Haarolean Haarolean moved this from To do to In progress in Release 0.4 Feb 1, 2022
@Haarolean Haarolean removed this from In progress in Release 0.4 Feb 8, 2022
@Haarolean Haarolean modified the milestones: 0.4, 0.5 Feb 8, 2022
@Haarolean Haarolean moved this from 0.4 to 0.5 in Roadmap Feb 8, 2022
@Haarolean Haarolean added this to Features in Release 0.5 Apr 21, 2022
@Haarolean Haarolean removed this from 0.5 in Roadmap May 4, 2022
@sw-tracker
Copy link

Will this still be included in release 0.5?

@Haarolean
Copy link
Contributor

Will this still be included in release 0.5?

No, unfortunately. We're moving towards smaller but quicker releases, that's why we had to shred the current one.

@Haarolean Haarolean modified the milestones: 0.5, 0.6 Jun 14, 2022
@Haarolean Haarolean removed this from Features in Release 0.5 Jun 14, 2022
@Haarolean Haarolean added this to 0.6 in Roadmap Jun 14, 2022
@Haarolean Haarolean added the status/accepted An issue which has passed triage and has been accepted label Oct 23, 2022
@Haarolean Haarolean linked a pull request Oct 31, 2022 that will close this issue
Expose cluster ACL list #2818
Merged
13 tasks
@Haarolean Haarolean added this to Features in Release 0.6 Dec 23, 2022
@Haarolean Haarolean modified the milestones: 0.6, 0.7 Feb 23, 2023
@Haarolean Haarolean removed this from Features in Release 0.6 Feb 23, 2023
@Haarolean Haarolean removed this from 0.6 in Roadmap Feb 23, 2023
@Haarolean Haarolean added this to Features in Release 0.7 Mar 9, 2023
@Haarolean Haarolean moved this from Features to Review in progress in Release 0.7 Mar 9, 2023
@Haarolean Haarolean added this to 0.7 in Roadmap Mar 9, 2023
@NeiruBugz NeiruBugz linked a pull request Apr 13, 2023 that will close this issue
FE: Expose cluster ACL list #3662
Merged
13 tasks
@Haarolean Haarolean changed the title Expose cluster ACL list ACL: Expose cluster's ACLs (read/only) Apr 13, 2023
@Haarolean
Copy link
Contributor

Haarolean commented Apr 13, 2023 •
edited
Loading

Within this issue we'll implement just the display of ACLs. For the ACLs management, refer to #124.

Release 0.7 automation moved this from Review in progress to Done May 2, 2023
@Haarolean Haarolean reopened this May 3, 2023
Release 0.7 automation moved this from Done to In progress May 3, 2023
@Haarolean Haarolean moved this from In progress to Review in progress in Release 0.7 May 5, 2023
@Haarolean Haarolean moved this from Review in progress to Reviewer approved in Release 0.7 May 11, 2023
Release 0.7 automation moved this from Reviewer approved to Done May 11, 2023
@S1M0NM
Copy link

S1M0NM commented May 11, 2023

Hi, we just checked out v0.7.0 but it seems there is a issue with the ACL Page in readonly mode:

image
acl_read-only-delete-dialog

The delete button is displayed next to the ACLs and a confirmation dialog even opens.
An error occurs when confirming.

I would expect not to see the delete button at all.

In addition, the list always snaps into a different order when the ACLs are re-fetched, making meaningful browsing impossible.

@iliax
Copy link
Contributor

iliax commented May 12, 2023

@S1M0NM thank you for feedback, we will take a look and improve soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iliax iliax

@NeiruBugz NeiruBugz

Labels
scope/backend scope/frontend status/accepted An issue which has passed triage and has been accepted type/feature A new feature
Projects
No open projects
Release 0.7
Done
Milestone
0.7
7 participants
@jonas-grgt @iliax @germanosin @Haarolean @sw-tracker @NeiruBugz @S1M0NM