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

   ### Description
   
   Based on the comment 
https://github.com/apache/cloudstack/pull/9385#issuecomment-2227061721 from 
@DaanHogland, and considering that all the DB view definitions are currently 
refactored out of the DB upgrade schema sql scripts, I took the liberty to 
delete most of the remaining unused view definitions on these scripts, as they 
are overwritten in the end of the DB upgrade workflow.
   
   ### 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)
   - [ ] build/CI
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [x] Minor
   
   ### How Has This Been Tested?
   
   Tested resetting the database to the 4.0.0 version via the management script 
cloudstack-setup-databases. After the database reset, the management server was 
also reset to ensure it ran all upgrade scripts.
   
   Once the reset process was completed, various API operations were tested, 
particularly those that use database views to build their responses. Verifying 
the behavior was the same after the changes. During these tests, normal 
behavior was consistently observed. All API operations performed as expected, 
with no errors encountered.
   
   Furthermore, the log stream was monitored throughout the testing phase to 
detect any potential issues or anomalies. The logs remained clear, showing no 
errors or warnings related to the current refactor.


-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to