api/server: list networks by name#11470
Conversation
|
@blueorangutan package |
|
@weizhouapache a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #11470 +/- ##
============================================
+ Coverage 17.36% 18.40% +1.03%
- Complexity 15234 15236 +2
============================================
Files 5886 5447 -439
Lines 525680 488727 -36953
Branches 64159 57380 -6779
============================================
- Hits 91260 89927 -1333
+ Misses 424120 388661 -35459
+ Partials 10300 10139 -161
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 14672 |
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 14762 |
api/src/main/java/org/apache/cloudstack/api/command/user/network/ListNetworksCmd.java
Outdated
Show resolved
Hide resolved
…rk/ListNetworksCmd.java
|
@blueorangutan package |
|
@weizhouapache a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 14802 |
|
@blueorangutan test |
|
@weizhouapache a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests |
|
[SF] Trillian test result (tid-14165)
|
|
@blueorangutan package |
|
@rosi-shapeblue a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 15358 |
|
The PR was tested with the following 4 networks:
Expected behavior:
Test Results 1. Exact Name Match (Case): list networks name="Test-Network"
2. Case-Insensitive Match: list networks name="test-network"
3. Partial Name – No Match: list networks name="Test"
4. Keyword Search – Partial Match: list networks keyword="Test"
5. Name + Zone Filter: list networks name="Test-Network" zoneid=
6. Non-Existing Name: list networks name="NonExistingNetwork"
Summary:
Raw Query Outputs |
RosiKyu
left a comment
There was a problem hiding this comment.
LGTM
Test Results Summary
- Name matches are exact but case-insensitive.
- Keyword works as expected for partial matches.
- No partial matches are returned when using name.
- If case-sensitive matching is intended, current behavior needs adjustment.
- If case-insensitive matching is acceptable, behavior is correct.
* api/server: list networks by name * Update api/src/main/java/org/apache/cloudstack/api/command/user/network/ListNetworksCmd.java


Description
This PR improves listNetworks API to list networks by exact name
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?