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

   ### Description
   
   This PR aims to add the concept of quarantine for public IPs. The feature is 
fully described in issue #7369. The main goal of this functionality is to block 
the allocation of a disassociated public IP address by a configurable duration, 
`public.ip.address.quarantine.duration`. This can prevent the allocation of 
this recently disassociated IP to another user. More critically, this feature 
can prevent the allocation of an IP that has been blacklisted for inappropriate 
use by the previous owner.
   
   Fixes: #7369 
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [x] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [ ] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ### Feature/Enhancement Scale or Bug Severity
   
   #### Feature/Enhancement Scale
   
   - [x] Major
   - [ ] Minor
   
   ### How Has This Been Tested?
   I added some unit tests to validate the core logic of this feature. 
Moreover, I tested in a local lab the whole process of disassociation of a 
public IP address and trying to allocate different accounts with different 
roles. As designed, only the previous owner was able to allocate the 
quarantined IP. Furthermore, I tested if the previous behavior is maintained, 
as the default value for the global configuration 
`public.ip.address.quarantine.duration` is `0`, no public IP address is added 
to quarantined, as expected.


-- 
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