openstack
/
charm-mysql-router
Code Issues Proposed changes

Use report host on bootstrap 

LP Bug #1914649 saw an edge case where --report-host was necessary for
mysql-router bootstrap. Adding it to cover this edge case.

Change-Id: I0ea751aa4015ad6e569d9c3e9f8364cf36801c18
Closes-Bug: #1914649
This commit is contained in:
David Ames 2021-02-04 13:03:28 -08:00
parent 5245a1592a
commit 7c8acbc18f
2 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/lib/charm/openstack/mysql_router.py
View File

@ -388,6 +388,7 @@ class MySQLRouterCharm(charms_openstack.charm.OpenStackCharm):
"--directory", self.mysqlrouter_working_dir,
"--conf-use-sockets",
"--conf-bind-address", self.shared_db_address,
"--report-host", self.db_router_address,
"--conf-base-port", str(self.mysqlrouter_port)]
# Avoid multiple routers trying to bind to the same api port
# Bug #1911907

2
unit_tests/test_lib_charm_openstack_mysql_router.py
View File

@ -392,6 +392,7 @@ class TestMySQLRouterCharm(test_utils.PatchHelper):
"--directory", mrc.mysqlrouter_working_dir,
"--conf-use-sockets",
"--conf-bind-address", mrc.shared_db_address,
"--report-host", mrc.db_router_address,
"--conf-base-port", _port],
stderr=self.stdout)
self.set_flag.assert_called_once_with(
@ -408,6 +409,7 @@ class TestMySQLRouterCharm(test_utils.PatchHelper):
"--directory", mrc.mysqlrouter_working_dir,
"--conf-use-sockets",
"--conf-bind-address", mrc.shared_db_address,
"--report-host", mrc.db_router_address,
"--conf-base-port", _port,
"--disable-rest"],
stderr=self.stdout)