cloudstack icon indicating copy to clipboard operation
cloudstack copied to clipboard

Allow users to Search ACL Rule

Open btzq opened this issue 1 year ago • 5 comments

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, https://github.com/apache/cloudstack/issues/8841

Allow users to Search ACL Rule

  • In large scale projects like ours, we have around 20-40 ACL Rules.
  • As the rules increase, its hard to read and identify.
  • Supporting search ACL Rule would help.
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.

btzq avatar Apr 01 '24 15:04 btzq

There is no generic keyword search (working!) as for other list APIs. There is no search on the field reason. There is no search by order/number. There is no search by cidr. There is no search by ....

for other fields the search is implemented, @btzq : networkId, id, traffictype, protocol, action

DaanHoogland avatar Apr 11 '24 10:04 DaanHoogland

@btzq , I started looking at this one, but am a bit strugling with still. I think we can improve a bit on what the extension of functionality would be. Care to chime in?

In #9566 I am adding reason to the generic keyword search listNetworkACLs command, but that is for the lists and not for the items.

Am I correct in understanding that this is mainly a UI issue?

DaanHoogland avatar Aug 22 '24 07:08 DaanHoogland

Hey @DaanHoogland , just to clarify you mean 'Lists' and 'Items' are as below, right?

Screenshot 2024-08-23 at 10 35 34 am

If yes, then in my opinion, just being bale to search the specific List/Rule will be sufficient. Its just to make navigating the dozens of List/Rule easier by only showing me the row users wanna see.

btzq avatar Aug 23 '24 02:08 btzq

I have implemented search for protocol and reason in #9566 @btzq . I think the other items are going to be more difficult but I'll think about those. Can you have a look/test?

DaanHoogland avatar Aug 23 '24 09:08 DaanHoogland

Im not sure how to compile Cloudstack from source tho... What about a video? I could verify it there.

Also, i dont think its necessary to search by the Items.

Search by Protocol and Reason is enough.

btzq avatar Aug 23 '24 12:08 btzq