From 5afc22b0ce170698032a7385d00f56b70eb852da Mon Sep 17 00:00:00 2001 From: Frank Maximus Date: Tue, 9 Jan 2018 17:33:37 +0100 Subject: [PATCH] CLOUDSTACK-9749: Fix Password server running on internal LB VM --- systemvm/debian/opt/cloud/bin/cs/CsAddress.py | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/systemvm/debian/opt/cloud/bin/cs/CsAddress.py b/systemvm/debian/opt/cloud/bin/cs/CsAddress.py index 6ad50459b570..8b68f538fa40 100755 --- a/systemvm/debian/opt/cloud/bin/cs/CsAddress.py +++ b/systemvm/debian/opt/cloud/bin/cs/CsAddress.py @@ -549,10 +549,21 @@ def post_config_change(self, method): self.fw_router() self.fw_vpcrouter() + cmdline = self.config.cmdline() + # On deletion nw_type will no longer be known if self.get_type() in ('guest'): if self.config.is_vpc() or self.config.is_router(): CsDevice(self.dev, self.config).configure_rp() + + # If redundant then this is dealt with + # by the master backup functions + if not cmdline.is_redundant(): + if method == "add": + CsPasswdSvc(self.address['public_ip']).start() + elif method == "delete": + CsPasswdSvc(self.address['public_ip']).stop() + logging.error( "Not able to setup source-nat for a regular router yet") @@ -564,11 +575,6 @@ def post_config_change(self, method): app = CsApache(self) app.setup() - cmdline = self.config.cmdline() - # If redundant then this is dealt with by the master backup functions - if self.get_type() in ["guest"] and not cmdline.is_redundant(): - pwdsvc = CsPasswdSvc(self.address['public_ip']).start() - if self.get_type() == "public" and self.config.is_vpc() and method == "add": if self.address["source_nat"]: vpccidr = cmdline.get_vpccidr()