GabrielBrascher opened a new pull request #4714: URL: https://github.com/apache/cloudstack/pull/4714
### Description This PR removes some unnecessary code and enhances a few log messages at IpAddressManagerImpl. Additionally, method `com.cloud.network.NetworkModelImpl.areServicesEnabledInZone` does not use list `checkedProvider`; the respective list should either be removed or populated. I have chosen to add provider names at `checkedProvider`, instead of removing it; however, I would appreciate any opinion/suggestion. ### 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) - [ ] Enhancement (improves an existing feature and functionality) - [X] Cleanup (Code refactoring and cleanup, that may add test cases) ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
