diff --git a/cloudbaseinit/init.py b/cloudbaseinit/init.py index d01e544c..e2679c94 100644 --- a/cloudbaseinit/init.py +++ b/cloudbaseinit/init.py @@ -39,7 +39,7 @@ def _get_plugins_section(self, instance_id): if not instance_id: return self._PLUGINS_CONFIG_SECTION else: - return instance_id + "/" + self._PLUGINS_CONFIG_SECTION + return ("%s/%s" % (instance_id, self._PLUGINS_CONFIG_SECTION)) def _get_plugin_status(self, osutils, instance_id, plugin_name): return osutils.get_config_value(plugin_name, diff --git a/cloudbaseinit/tests/test_init.py b/cloudbaseinit/tests/test_init.py index d339671e..307d9e94 100644 --- a/cloudbaseinit/tests/test_init.py +++ b/cloudbaseinit/tests/test_init.py @@ -60,14 +60,18 @@ def _test_get_plugin_section(self, instance_id): if not instance_id: self.assertEqual(self._init._PLUGINS_CONFIG_SECTION, response) else: - self.assertEqual( - instance_id + "/" + self._init._PLUGINS_CONFIG_SECTION, - response) + expected_response = ( + "%s/%s" % (instance_id, self._init._PLUGINS_CONFIG_SECTION)) + self.assertEqual(expected_response, response) def test_get_plugin_section_id(self): fake_id = "100" self._test_get_plugin_section(instance_id=fake_id) + def test_get_plugin_section_id_int(self): + fake_id = 100 + self._test_get_plugin_section(instance_id=fake_id) + def test_get_plugin_section_no_id(self): self._test_get_plugin_section(instance_id=None)