GutoVeronezi commented on PR #7417: URL: https://github.com/apache/cloudstack/pull/7417#issuecomment-1500783920
At this time this PR is a POC; some points need to be better discussed or still have to be addressed: Topics to be better discussed: a. (i) create a single file for all `VIEW`s or (ii) a file for each `VIEW`; b. when removing a `VIEW`, (i) remove the file and add the `DROP VIEW` statement in the `schema` file or (ii) keep the `VIEW` file and add the `DROP VIEW` statement to it; c. commands in the `schema` files that rely on `VIEW`s. Points yet to be addressed: - [ ] via GitHub Actions, enforce the `VIEW` management through the new workflow; - [ ] update our [Coding Convention] with the new `VIEW` management workflow; I also added a validation to execute this workflow only after upgrading to 4.19.0.0, for back-compatibility purposes. If I am missing something, please, let me know. [Coding Convention]: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Coding+conventions -- 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]
