diff --git a/actions/openstack_upgrade.py b/actions/openstack_upgrade.py index 6c04ac1f..aca6b699 100755 --- a/actions/openstack_upgrade.py +++ b/actions/openstack_upgrade.py @@ -53,7 +53,8 @@ def openstack_upgrade(): if (do_action_openstack_upgrade('keystone', do_openstack_upgrade, register_configs())): - os.execl('./hooks/config-changed-postupgrade', '') + os.execl('./hooks/config-changed-postupgrade', + 'config-changed-postupgrade') if __name__ == '__main__': openstack_upgrade() diff --git a/hooks/keystone_utils.py b/hooks/keystone_utils.py index b6f16f17..f336ffd2 100644 --- a/hooks/keystone_utils.py +++ b/hooks/keystone_utils.py @@ -1117,9 +1117,7 @@ class ManagerServer(): def _launch_manager(self): script = os.path.abspath(os.path.join(os.path.dirname(__file__), 'manager.py')) - release = CompareOpenStackReleases( - get_os_codename_install_source(config('openstack-origin')) - ) + release = CompareOpenStackReleases(os_release('keystone')) # need to set the environment variable PYTHONPATH to include the # payload's directory for the manager.py to find the various keystone # clients diff --git a/unit_tests/test_actions_openstack_upgrade.py b/unit_tests/test_actions_openstack_upgrade.py index b0c10a33..4475393a 100644 --- a/unit_tests/test_actions_openstack_upgrade.py +++ b/unit_tests/test_actions_openstack_upgrade.py @@ -56,7 +56,7 @@ class TestKeystoneUpgradeActions(CharmTestCase): self.assertTrue(self.do_openstack_upgrade.called) self.os.execl.assert_called_with('./hooks/config-changed-postupgrade', - '') + 'config-changed-postupgrade') @patch.object(openstack_upgrade, 'register_configs') @patch('charmhelpers.contrib.openstack.utils.config')