erikbocks opened a new pull request, #12864:
URL: https://github.com/apache/cloudstack/pull/12864

   ### Description
   
   In order to help users to identify more easily the reason that their NIC's 
secondary IP address were allocated to the VM, a description field was added to 
the secondary IPs. Now, during secondary IP address allocation, it is possible 
to inform a descriptive text, that will be displayed along with the IP address 
in the NIC details or in the API's responses.
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [X] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   - [ ] Build/CI
   - [ ] Test (unit or integration test code)
   
   ### Feature/Enhancement Scale or Bug Severity
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [x] Minor
   
   ### Screenshots (if appropriate):
   
   <details><summary>Figure 1 - Form description field</summary>
   
   <img width="522" height="504" alt="image" 
src="https://github.com/user-attachments/assets/9e967f51-5bee-4f9f-ab5d-5aa3f660ade0";
 />
   
   </details>
   
   
   <details><summary>Figure 2 - IP's description present at NIC 
details</summary>
   
   <img width="1188" height="706" alt="image" 
src="https://github.com/user-attachments/assets/d619b1cb-188d-4aa6-94c3-03e10b6ea988";
 />
   
   </details>
   
   ### How Has This Been Tested?
   
   In an environment where the PR changes were installed, I opened the 
secondary IP addition tab and validated that the description field was added 
successfully to the form (Figure 1 in the `Screenshots` tab). Via CMK, I 
validated that the description field was also added to the `addIpToNic`API.
   
   Then, I added a secondary IP address to my VM with an example text in it 
(Figure 2). After that, I made some tests to features involving secondary IPs, 
as the addresses management and configuration, as well as a communication test 
with static NAT assigned to the secondary IP.
   
   With CloudMonkey, I also validated that the description field was now 
returned in NICs' responses in the `listNics` API.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to