lucas-a-martins commented on code in PR #9969:
URL: https://github.com/apache/cloudstack/pull/9969#discussion_r1958417320


##########
setup/bindir/cloud-setup-databases.in:
##########
@@ -229,6 +229,10 @@ for full help
             "GRANT ALL ON cloud_usage.* to cloud@`%`;",
             "GRANT process ON *.* TO cloud@`localhost`;",
             "GRANT process ON *.* TO cloud@`%`;",
+            "IF foo > 0 THEN",

Review Comment:
   @DaanHoogland I was unable to skip the `DROP` without skipping the `IF` and 
`END IF`, the command results in an error. Considering that the `IF` is used 
only for the query I want to skip, I tought skipping it too wouldn't be a 
problem.
   
   My goal when removing these queries is because the new flag was removing 
users from database when `host = % || localhost`, but the idea of this PR is to 
not create additional users, not to remove existing ones. 



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