Repository navigation
Option to set ACL Rule to be the very bottom by default #8865
Description
ISSUE TYPE
- Improvement Request
COMPONENT NAME
Improvement Request (UI, Functionality)
CLOUDSTACK VERSION
4.19
CONFIGURATION
N/A
OS / ENVIRONMENT
N/A
SUMMARY
Note: This ticket is broken down as requested from the origianl post, #8841
Option to set ACL Rule to be the very bottom by default
- At the very end of our ACL Rule, we usually have a 'Deny All'
- But when i use API or Terraform to create ACL Rules, the arrangement and sequence (including the rule number) gets all messed up.
- Would be great if we could have a parameter and function in the UI to make the rule fix itself at the bottom of the list.
STEPS TO REPRODUCE
N/A
N/A
EXPECTED RESULTS
To be able to manage ACL Rules more easily for large scale projects
ACTUAL RESULTS
Challenging to use ACL rules for large scale projects.
Activity
DaanHoogland commentedon Apr 11, 2024
@btzq , you can use the parameter
numberto set the order of an acl item in the acl list. Does this fullfill this functionality? We may need to improve the documentation for this to make it more clear.DaanHoogland commentedon Apr 11, 2024
I think the createNetworkAcl should be renamed to createNetworkAclItem to be more consistent with the other commands. Same for deleteNetworkACL(Item)!
btzq commentedon Apr 12, 2024
@DaanHoogland i wanst sure that the 'number' parameter was working. Because i was intuitively expecting that the row would automatically be moved/sorted to the bottom of the page. I think i was expecting this as well cause the ACL Rule row can be dragged up and down manually. So when i saw that the rule wasnt being moved to the bottom (when i keyed in the number parameter), i assumed it wasnt working.
DaanHoogland commentedon Apr 12, 2024
That's alright @btzq , but does this satisfy this request/issue?
btzq commentedon Apr 12, 2024
I think at least, the Cloudstack GUI should auto sort the ACL Rules based on the 'number'.
It would just be a UI change and would improve the usability.
DaanHoogland commentedon Apr 12, 2024
I think it does, at least from the UI. Can you describe the scenario in which it doesn't?
btzq commentedon Apr 12, 2024
@DaanHoogland i just retested and i think i know the confusion:
And if you were to try to rearange things later on, it becomes super complicated because it doesnt re-initiliase the rule numbers. For example:
So maybe what we need instead to make it less confusing is to re-initailise the rule numbers?
DaanHoogland commentedon Apr 12, 2024
ok, makes sense. This could also be a UI implementation or as alternative an extra API
reorderAclListor the likes. I would expect it to get the id of the list and anintervaloption, in order to specify that instead of 1,2,3,4,5,6,7,8,9,10 I want 1,3,5,7,9,11,13,15,17,19 as a result, or 10,20,30,40,50,60,70,80,90,100.ui: add move to top/bottom for acl rules list
shwstppr commentedon May 28, 2024
Fixed with #9119
Adding a new API for re-ordering list with rule numbers can be a separate discussion