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 [email protected] or file a JIRA ticket
with INFRA.
---