UI: Add filter to list encrypted volumes#9548
Conversation
|
@Pearl1594 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9548 +/- ##
===========================================
Coverage 15.54% 15.54%
- Complexity 11995 11998 +3
===========================================
Files 5499 5499
Lines 481706 481720 +14
Branches 58660 61791 +3131
===========================================
+ Hits 74879 74884 +5
- Misses 398541 398548 +7
- Partials 8286 8288 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
UI build: ✔️ |
DaanHoogland
left a comment
There was a problem hiding this comment.
clgtm and tested in qa, but I think it can do with more testing (with real encrypted volumes)
|
@DaanHoogland In order to address your comment, I've also tested this PR in a local environment by creating an encrypted volume and testing the new filter. |







Description
This PR fixes: #9130
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Screenshots (if appropriate):
How Has This Been Tested?
Added toggle button for filtering only encrypted volumes:

When set to true, passes 'isencrypted' parameter to the listVolumes API call:

When set to false, returns all non-encrypted volumes:

How did you try to break this feature and the system with this change?