Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1428#issuecomment-215854412 this LGTM. we have been running this in our environment for over a month (ACS 4.8). This PR fixes the issue we had with cloudstack-mangaement not starting when sql ha is enabled. On a related note, we may have another issue somewhere else. When cloudstack is failed over to the slave sql server, we cannot launch a console or issue a reboot of a systemvm from the web-interface. @kiwiflyer have you tested those two actions when failed over to the slave sql server? I just noticed this on our side yesterday while doing maintenance.
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---