Merge ~ballot/charm-k8s-mm-pd-bot/+git/charm-k8s-mm-pd-bot:quotes into charm-k8s-mm-pd-bot:master

[Benjamin Allot]

Normalize quotes with black

[Laurent Sesquès (sajoupa)]

+1

review: Approve

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

Change successfully merged at revision 8671d0b6c001cc0e0ae09865386645511af1c755

diff --git a/pyproject.toml b/pyproject.toml
index d2f23b9..55ec8d7 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,3 +1,2 @@
 [tool.black]
-skip-string-normalization = true
 line-length = 120
diff --git a/src/charm.py b/src/charm.py
index b17ced0..7b2807f 100755
--- a/src/charm.py
+++ b/src/charm.py
@@ -18,18 +18,18 @@ from ops.model import (
 
 logger = logging.getLogger()
 
-REQUIRED_JUJU_SETTINGS = ['image_path', 'mm_pd_bot_cfg']
+REQUIRED_JUJU_SETTINGS = ["image_path", "mm_pd_bot_cfg"]
 REQUIRED_CFG_SETTINGS = {
-    'PagerDuty': ['account', 'api-token', 'private-channel', 'public-channel'],
-    'Prometheus': ['max_cache_size'],
-    'httpd': ['hostname', 'listen-ip', 'listen-port', 'magic-uuid'],
-    'MattermostBot': ['bot_url', 'bot_team'],
+    "PagerDuty": ["account", "api-token", "private-channel", "public-channel"],
+    "Prometheus": ["max_cache_size"],
+    "httpd": ["hostname", "listen-ip", "listen-port", "magic-uuid"],
+    "MattermostBot": ["bot_url", "bot_team"],
     # Those sections can be empty
-    'nickname to email': [],
-    'Mattermost Aliases': [],
-    'PagerDuty service to Mattermost Channel': [],
+    "nickname to email": [],
+    "Mattermost Aliases": [],
+    "PagerDuty service to Mattermost Channel": [],
 }
-BOT_CFG_SECTION = 'MattermostBot'
+BOT_CFG_SECTION = "MattermostBot"
 
 
 class MmPdBotK8sCharmConfigError(Exception):
@@ -51,9 +51,9 @@ class MmPdBotK8sCharm(CharmBase):
         :raises MmPdBotK8sCharmConfigError: Raise if you don't have a bot_token or the combination of bot_login and
         bot_password.
         """
-        bot_token = self.bot_config.get(BOT_CFG_SECTION, 'bot_token', fallback=None)
-        bot_login = self.bot_config.get(BOT_CFG_SECTION, 'bot_login', fallback=None)
-        bot_password = self.bot_config.get(BOT_CFG_SECTION, 'bot_password', fallback=None)
+        bot_token = self.bot_config.get(BOT_CFG_SECTION, "bot_token", fallback=None)
+        bot_login = self.bot_config.get(BOT_CFG_SECTION, "bot_login", fallback=None)
+        bot_password = self.bot_config.get(BOT_CFG_SECTION, "bot_password", fallback=None)
 
         if not bot_token and (not bot_login or not bot_password):
             err_msg = "bot_token or bot_login/bot_password in {0} section required".format(BOT_CFG_SECTION)
@@ -129,7 +129,7 @@ class MmPdBotK8sCharm(CharmBase):
 
     def _validate_mm_pd_bot_configuration(self) -> None:
         """Check the configuration part related to mm_pd_bot configuration."""
-        self.bot_config = self._load_mm_pd_bot_configuration(self.model.config['mm_pd_bot_cfg'])
+        self.bot_config = self._load_mm_pd_bot_configuration(self.model.config["mm_pd_bot_cfg"])
         self._validate_mm_pd_bot_configuration_content()
         self._validate_bot_creds()
 
@@ -148,7 +148,7 @@ class MmPdBotK8sCharm(CharmBase):
         """
         config = self.model.config
         pod_config = {
-            'MM_PD_BOT_CFG': config['mm_pd_bot_cfg'],
+            "MM_PD_BOT_CFG": config["mm_pd_bot_cfg"],
         }
 
         return pod_config
@@ -159,10 +159,10 @@ class MmPdBotK8sCharm(CharmBase):
         :param pod_spec: pod spec v3 as defined by juju.
         """
 
-        hostname = self.bot_config.get('httpd', 'hostname')
-        container_port = self.bot_config.getint('httpd', 'listen-port')
-        magic_uuid = self.bot_config.get('httpd', 'magic-uuid')
-        tls_secret_name = self.model.config.get('tls_secret_name', None)
+        hostname = self.bot_config.get("httpd", "hostname")
+        container_port = self.bot_config.getint("httpd", "listen-port")
+        magic_uuid = self.bot_config.get("httpd", "magic-uuid")
+        tls_secret_name = self.model.config.get("tls_secret_name", None)
 
         if tls_secret_name:
             scheme = "https"
@@ -192,52 +192,52 @@ class MmPdBotK8sCharm(CharmBase):
             },
         }
         if tls_secret_name:
-            ingress['spec']['tls'] = [
-                {'hosts': [bot_listening_url_parsed.hostname], 'secretName': tls_secret_name},
+            ingress["spec"]["tls"] = [
+                {"hosts": [bot_listening_url_parsed.hostname], "secretName": tls_secret_name},
             ]
         else:
-            annotations['nginx.ingress.kubernetes.io/ssl-redirect'] = 'false'
+            annotations["nginx.ingress.kubernetes.io/ssl-redirect"] = "false"
 
-        ingress_whitelist_source_range = self.model.config.get('ingress_whitelist_source_range', None)
+        ingress_whitelist_source_range = self.model.config.get("ingress_whitelist_source_range", None)
         if ingress_whitelist_source_range:
-            annotations['nginx.ingress.kubernetes.io/whitelist-source-range'] = ingress_whitelist_source_range
+            annotations["nginx.ingress.kubernetes.io/whitelist-source-range"] = ingress_whitelist_source_range
 
         if annotations:
-            ingress['annotations'] = annotations
+            ingress["annotations"] = annotations
 
         # Due to https://github.com/canonical/operator/issues/293 we
         # can't use pod.set_spec's k8s_resources argument.
-        resources = pod_spec.get('kubernetesResources', {})
-        resources['ingressResources'] = [ingress]
-        pod_spec['kubernetesResources'] = resources
+        resources = pod_spec.get("kubernetesResources", {})
+        resources["ingressResources"] = [ingress]
+        pod_spec["kubernetesResources"] = resources
 
     def _make_pod_spec(self) -> None:
         """Create a pod spec with some core configuration."""
 
         config = self.model.config
-        container_port = self.bot_config.getint('httpd', 'listen-port')
+        container_port = self.bot_config.getint("httpd", "listen-port")
         # TODO: The bot replies 501 on HTTP GET so until we have something to probe readiness, magic-uuid is not needed
         # here
         # magic_uuid = self.bot_config.get('httpd', 'magic-uuid')
         image_details = {
-            'imagePath': config['image_path'],
+            "imagePath": config["image_path"],
         }
-        if config.get('image_username', None):
-            image_details.update({'username': config['image_username'], 'password': config['image_password']})
+        if config.get("image_username", None):
+            image_details.update({"username": config["image_username"], "password": config["image_password"]})
         pod_config = self._make_pod_config()
 
         return {
-            'version': 3,  # otherwise resources are ignored
-            'containers': [
+            "version": 3,  # otherwise resources are ignored
+            "containers": [
                 {
-                    'name': self.app.name,
-                    'imageDetails': image_details,
+                    "name": self.app.name,
+                    "imageDetails": image_details,
                     # TODO: debatable. The idea is that if you want to force an update with the same image name, you
                     # don't need to empty kubelet cache on each node to have the right version.
                     # This implies a performance drop upon start.
-                    'imagePullPolicy': 'Always',
-                    'ports': [{'containerPort': container_port, 'protocol': 'TCP'}],
-                    'envConfig': pod_config,
+                    "imagePullPolicy": "Always",
+                    "ports": [{"containerPort": container_port, "protocol": "TCP"}],
+                    "envConfig": pod_config,
                     # TODO: Add readiness probe in the bot.
                     # 'kubernetes': {
                     #     'readinessProbe': {'httpGet': {'path': '/{0}/'.format(magic_uuid), 'port': container_port}},
@@ -261,13 +261,13 @@ class MmPdBotK8sCharm(CharmBase):
             self.unit.status = BlockedStatus(str(exc))
             return
 
-        logger.info('Assembling pod spec')
+        logger.info("Assembling pod spec")
         pod_spec = self._make_pod_spec()
         self._update_pod_spec_for_k8s_ingress(pod_spec)
-        logger.info('Setting pod spec')
+        logger.info("Setting pod spec")
         self.model.pod.set_spec(pod_spec)
         self.unit.status = ActiveStatus()
 
 
-if __name__ == '__main__':  # pragma: no cover
+if __name__ == "__main__":  # pragma: no cover
     main(MmPdBotK8sCharm, use_juju_for_storage=True)
diff --git a/tests/unit/scenario.py b/tests/unit/scenario.py
index 9c62641..4aa55b5 100644
--- a/tests/unit/scenario.py
+++ b/tests/unit/scenario.py
@@ -12,16 +12,16 @@ import yaml
 logger = logging.getLogger(__name__)
 
 BOT_CFG_FILES = {
-    'both_authentication_method': 'both_authentication_method.cfg',
-    'good_configuration': 'good_bot.cfg',
-    'missing_options': 'missing_options.cfg',
-    'missing_sections': 'missing_sections.cfg',
-    'no_valid_authentication': 'no_valid_authentication.cfg',
-    'ssl_configuration': 'ssl_bot.cfg',
+    "both_authentication_method": "both_authentication_method.cfg",
+    "good_configuration": "good_bot.cfg",
+    "missing_options": "missing_options.cfg",
+    "missing_sections": "missing_sections.cfg",
+    "no_valid_authentication": "no_valid_authentication.cfg",
+    "ssl_configuration": "ssl_bot.cfg",
 }
 
 
-def get_cfg_content(configuration='good_configuration') -> str:
+def get_cfg_content(configuration="good_configuration") -> str:
     """Get the content of the file associated with configuration.
 
     :param str configuration: Key of the BOT_CFG_FILES dictionary to get the content of the associated configuration
@@ -32,7 +32,7 @@ def get_cfg_content(configuration='good_configuration') -> str:
     files_dir = "files"
     try:
         path = Path(dir_path, files_dir, BOT_CFG_FILES[configuration])
-        with open(path, 'r') as cfg:
+        with open(path, "r") as cfg:
             return cfg.read()
     except KeyError:
         logger.error("No configuration files associated with '%s'", configuration)
@@ -43,12 +43,12 @@ def get_juju_settings_default_value() -> list:
     """Return the list of juju settings as defined in config.yaml."""
 
     dir_path = os.path.dirname(os.path.realpath(__file__))
-    config_yaml = Path(dir_path, '..', '..', 'config.yaml')
-    with open(config_yaml, 'r') as config:
+    config_yaml = Path(dir_path, "..", "..", "config.yaml")
+    with open(config_yaml, "r") as config:
         loaded_config = yaml.safe_load(config.read())
 
     # Load the dict with the default values
-    default_values = {config: value['default'] for (config, value) in loaded_config['options'].items()}
+    default_values = {config: value["default"] for (config, value) in loaded_config["options"].items()}
 
     return default_values
 
@@ -57,52 +57,52 @@ def get_juju_settings_default_value() -> list:
 JUJU_SETTINGS_DEFAULT = get_juju_settings_default_value()
 
 TEST_JUJU_SETTINGS = {
-    'empty_image_path': {
-        'config': {'mm_pd_bot_cfg': 'dummy'},
-        'logger': ["ERROR:root:Required setting empty: image_path"],
-        'expected': 'Required setting(s) empty: image_path',
+    "empty_image_path": {
+        "config": {"mm_pd_bot_cfg": "dummy"},
+        "logger": ["ERROR:root:Required setting empty: image_path"],
+        "expected": "Required setting(s) empty: image_path",
     },
-    'empty_mm_pd_bot_cfg': {
-        'config': {'image_path': 'mm_pd_bot:devel'},
-        'logger': ["ERROR:root:Required setting empty: mm_pd_bot_cfg"],
-        'expected': 'Required setting(s) empty: mm_pd_bot_cfg',
+    "empty_mm_pd_bot_cfg": {
+        "config": {"image_path": "mm_pd_bot:devel"},
+        "logger": ["ERROR:root:Required setting empty: mm_pd_bot_cfg"],
+        "expected": "Required setting(s) empty: mm_pd_bot_cfg",
     },
-    'empty_all_settings': {
-        'config': {},
-        'logger': ["ERROR:root:Required setting empty: image_path", "ERROR:root:Required setting empty: mm_pd_bot_cfg"],
-        'expected': 'Required setting(s) empty: image_path, mm_pd_bot_cfg',
+    "empty_all_settings": {
+        "config": {},
+        "logger": ["ERROR:root:Required setting empty: image_path", "ERROR:root:Required setting empty: mm_pd_bot_cfg"],
+        "expected": "Required setting(s) empty: image_path, mm_pd_bot_cfg",
     },
 }
 
 
 VALIDATE_MM_PD_BOT_CFG = {
-    'good_configuration': {
-        'config': {'image_path': 'mm_pd_bot:devel', 'mm_pd_bot_cfg': get_cfg_content("good_configuration")},
-        'expected': True,
-        'logger': [],
+    "good_configuration": {
+        "config": {"image_path": "mm_pd_bot:devel", "mm_pd_bot_cfg": get_cfg_content("good_configuration")},
+        "expected": True,
+        "logger": [],
     },
-    'bad_formatted_configuration': {
-        'config': {'image_path': 'mm_pd_bot:devel', 'mm_pd_bot_cfg': "[PagerDuty"},
-        'expected': "Error while parsing mm_pd_bot_cfg setting",
-        'logger': ["ERROR:root:Error while parsing mm_pd_bot_cfg setting"],
+    "bad_formatted_configuration": {
+        "config": {"image_path": "mm_pd_bot:devel", "mm_pd_bot_cfg": "[PagerDuty"},
+        "expected": "Error while parsing mm_pd_bot_cfg setting",
+        "logger": ["ERROR:root:Error while parsing mm_pd_bot_cfg setting"],
     },
-    'missing_sections': {
-        'config': {'image_path': 'mm_pd_bot:devel', 'mm_pd_bot_cfg': get_cfg_content('missing_sections')},
-        'expected': "Required section(s) missing in bot configuration file: "
+    "missing_sections": {
+        "config": {"image_path": "mm_pd_bot:devel", "mm_pd_bot_cfg": get_cfg_content("missing_sections")},
+        "expected": "Required section(s) missing in bot configuration file: "
         "MattermostBot, Prometheus, httpd, nickname to email",
-        'logger': [
+        "logger": [
             "ERROR:root:required section missing in bot configuration file: httpd",
             "ERROR:root:required section missing in bot configuration file: MattermostBot",
             "ERROR:root:required section missing in bot configuration file: Prometheus",
             "ERROR:root:required section missing in bot configuration file: nickname to email",
         ],
     },
-    'missing_options': {
-        'config': {'image_path': 'mm_pd_bot:devel', 'mm_pd_bot_cfg': get_cfg_content('missing_options')},
-        'expected': "Required option(s) missing in section MattermostBot: bot_team, bot_url, "
+    "missing_options": {
+        "config": {"image_path": "mm_pd_bot:devel", "mm_pd_bot_cfg": get_cfg_content("missing_options")},
+        "expected": "Required option(s) missing in section MattermostBot: bot_team, bot_url, "
         "Required option(s) missing in section PagerDuty: api-token, "
         "Required option(s) missing in section httpd: hostname, listen-ip, listen-port, magic-uuid",
-        'logger': [
+        "logger": [
             "ERROR:root:required option api-token missing in section PagerDuty",
             "ERROR:root:required option bot_team missing in section MattermostBot",
             "ERROR:root:required option bot_url missing in section MattermostBot",
@@ -112,52 +112,52 @@ VALIDATE_MM_PD_BOT_CFG = {
             "ERROR:root:required option magic-uuid missing in section httpd",
         ],
     },
-    'no_valid_authentication': {
-        'config': {'mm_pd_bot_cfg': get_cfg_content('no_valid_authentication')},
-        'expected': "bot_token or bot_login/bot_password in MattermostBot section required",
-        'logger': ["ERROR:root:bot_token or bot_login/bot_password in MattermostBot section required"],
+    "no_valid_authentication": {
+        "config": {"mm_pd_bot_cfg": get_cfg_content("no_valid_authentication")},
+        "expected": "bot_token or bot_login/bot_password in MattermostBot section required",
+        "logger": ["ERROR:root:bot_token or bot_login/bot_password in MattermostBot section required"],
     },
-    'both_authentication_method': {
-        'config': {'mm_pd_bot_cfg': get_cfg_content('both_authentication_method')},
-        'expected': "bot_token and bot_login are both set. Pick one of them",
-        'logger': ["ERROR:root:bot_token and bot_login are both set. Pick one of them"],
+    "both_authentication_method": {
+        "config": {"mm_pd_bot_cfg": get_cfg_content("both_authentication_method")},
+        "expected": "bot_token and bot_login are both set. Pick one of them",
+        "logger": ["ERROR:root:bot_token and bot_login are both set. Pick one of them"],
     },
 }
 
 VALIDATE_POD_SPEC = {
-    'basic': {
-        'config': {'image_path': 'mm_pd_bot:devel', 'mm_pd_bot_cfg': get_cfg_content('good_configuration')},
-        'pod_spec': {
-            'version': 3,  # otherwise resources are ignored
-            'containers': [
+    "basic": {
+        "config": {"image_path": "mm_pd_bot:devel", "mm_pd_bot_cfg": get_cfg_content("good_configuration")},
+        "pod_spec": {
+            "version": 3,  # otherwise resources are ignored
+            "containers": [
                 {
-                    'name': 'mm-pd-bot',
-                    'imageDetails': {
-                        'imagePath': 'mm_pd_bot:devel',
+                    "name": "mm-pd-bot",
+                    "imageDetails": {
+                        "imagePath": "mm_pd_bot:devel",
                     },
-                    'imagePullPolicy': 'Always',
-                    'ports': [{'containerPort': 2160, 'protocol': 'TCP'}],
-                    'envConfig': {'MM_PD_BOT_CFG': get_cfg_content('good_configuration')},
+                    "imagePullPolicy": "Always",
+                    "ports": [{"containerPort": 2160, "protocol": "TCP"}],
+                    "envConfig": {"MM_PD_BOT_CFG": get_cfg_content("good_configuration")},
                 }
             ],
         },
     },
-    'private_registry': {
-        'config': {
-            'image_path': 'mm_pd_bot:devel',
-            'image_username': 'rockity',
-            'image_password': 'rock',
-            'mm_pd_bot_cfg': get_cfg_content('good_configuration'),
+    "private_registry": {
+        "config": {
+            "image_path": "mm_pd_bot:devel",
+            "image_username": "rockity",
+            "image_password": "rock",
+            "mm_pd_bot_cfg": get_cfg_content("good_configuration"),
         },
-        'pod_spec': {
-            'version': 3,  # otherwise resources are ignored
-            'containers': [
+        "pod_spec": {
+            "version": 3,  # otherwise resources are ignored
+            "containers": [
                 {
-                    'name': 'mm-pd-bot',
-                    'imageDetails': {'imagePath': 'mm_pd_bot:devel', 'username': 'rockity', 'password': 'rock'},
-                    'imagePullPolicy': 'Always',
-                    'ports': [{'containerPort': 2160, 'protocol': 'TCP'}],
-                    'envConfig': {'MM_PD_BOT_CFG': get_cfg_content('good_configuration')},
+                    "name": "mm-pd-bot",
+                    "imageDetails": {"imagePath": "mm_pd_bot:devel", "username": "rockity", "password": "rock"},
+                    "imagePullPolicy": "Always",
+                    "ports": [{"containerPort": 2160, "protocol": "TCP"}],
+                    "envConfig": {"MM_PD_BOT_CFG": get_cfg_content("good_configuration")},
                 }
             ],
         },
@@ -166,124 +166,124 @@ VALIDATE_POD_SPEC = {
 
 
 VALIDATE_POD_SPEC_AND_INGRESS = {
-    'basic': {
-        'config': {'image_path': 'mm_pd_bot:devel', 'mm_pd_bot_cfg': get_cfg_content('good_configuration')},
-        'pod_spec': {
-            'version': 3,  # otherwise resources are ignored
-            'containers': [
+    "basic": {
+        "config": {"image_path": "mm_pd_bot:devel", "mm_pd_bot_cfg": get_cfg_content("good_configuration")},
+        "pod_spec": {
+            "version": 3,  # otherwise resources are ignored
+            "containers": [
                 {
-                    'name': 'mm-pd-bot',
-                    'imageDetails': {'imagePath': 'mm_pd_bot:devel'},
-                    'imagePullPolicy': 'Always',
-                    'ports': [{'containerPort': 2160, 'protocol': 'TCP'}],
-                    'envConfig': {'MM_PD_BOT_CFG': get_cfg_content('good_configuration')},
+                    "name": "mm-pd-bot",
+                    "imageDetails": {"imagePath": "mm_pd_bot:devel"},
+                    "imagePullPolicy": "Always",
+                    "ports": [{"containerPort": 2160, "protocol": "TCP"}],
+                    "envConfig": {"MM_PD_BOT_CFG": get_cfg_content("good_configuration")},
                 }
             ],
-            'kubernetesResources': {
-                'ingressResources': [
+            "kubernetesResources": {
+                "ingressResources": [
                     {
-                        'name': 'mm-pd-bot-ingress',
-                        'spec': {
-                            'rules': [
+                        "name": "mm-pd-bot-ingress",
+                        "spec": {
+                            "rules": [
                                 {
-                                    'host': 'mm-pd-bot.example.com',
-                                    'http': {
-                                        'paths': [
+                                    "host": "mm-pd-bot.example.com",
+                                    "http": {
+                                        "paths": [
                                             {
-                                                'path': '/bdddcacb-ab42-40ac-9106-4275c1db1519/',
-                                                'backend': {'serviceName': 'mm-pd-bot', 'servicePort': 2160},
+                                                "path": "/bdddcacb-ab42-40ac-9106-4275c1db1519/",
+                                                "backend": {"serviceName": "mm-pd-bot", "servicePort": 2160},
                                             },
                                         ],
                                     },
                                 },
                             ],
                         },
-                        'annotations': {'nginx.ingress.kubernetes.io/ssl-redirect': 'false'},
+                        "annotations": {"nginx.ingress.kubernetes.io/ssl-redirect": "false"},
                     },
                 ],
             },
         },
     },
-    'ssl': {
-        'config': {
-            'image_path': 'mm_pd_bot:devel',
-            'mm_pd_bot_cfg': get_cfg_content('ssl_configuration'),
-            'ingress_whitelist_source_range': '10.0.69.0/24',
-            'tls_secret_name': 'mm_pd_bot_secret',
+    "ssl": {
+        "config": {
+            "image_path": "mm_pd_bot:devel",
+            "mm_pd_bot_cfg": get_cfg_content("ssl_configuration"),
+            "ingress_whitelist_source_range": "10.0.69.0/24",
+            "tls_secret_name": "mm_pd_bot_secret",
         },
-        'pod_spec': {
-            'version': 3,  # otherwise resources are ignored
-            'containers': [
+        "pod_spec": {
+            "version": 3,  # otherwise resources are ignored
+            "containers": [
                 {
-                    'name': 'mm-pd-bot',
-                    'imageDetails': {'imagePath': 'mm_pd_bot:devel'},
-                    'imagePullPolicy': 'Always',
-                    'ports': [{'containerPort': 2160, 'protocol': 'TCP'}],
-                    'envConfig': {'MM_PD_BOT_CFG': get_cfg_content('ssl_configuration')},
+                    "name": "mm-pd-bot",
+                    "imageDetails": {"imagePath": "mm_pd_bot:devel"},
+                    "imagePullPolicy": "Always",
+                    "ports": [{"containerPort": 2160, "protocol": "TCP"}],
+                    "envConfig": {"MM_PD_BOT_CFG": get_cfg_content("ssl_configuration")},
                 }
             ],
-            'kubernetesResources': {
-                'ingressResources': [
+            "kubernetesResources": {
+                "ingressResources": [
                     {
-                        'name': 'mm-pd-bot-ingress',
-                        'spec': {
-                            'rules': [
+                        "name": "mm-pd-bot-ingress",
+                        "spec": {
+                            "rules": [
                                 {
-                                    'host': 'mm-pd-bot.example.com',
-                                    'http': {
-                                        'paths': [
+                                    "host": "mm-pd-bot.example.com",
+                                    "http": {
+                                        "paths": [
                                             {
-                                                'path': '/bdddcacb-ab42-40ac-9106-4275c1db1519/',
-                                                'backend': {'serviceName': 'mm-pd-bot', 'servicePort': 2160},
+                                                "path": "/bdddcacb-ab42-40ac-9106-4275c1db1519/",
+                                                "backend": {"serviceName": "mm-pd-bot", "servicePort": 2160},
                                             },
                                         ],
                                     },
                                 },
                             ],
-                            'tls': [{'hosts': ['mm-pd-bot.example.com'], 'secretName': 'mm_pd_bot_secret'}],
+                            "tls": [{"hosts": ["mm-pd-bot.example.com"], "secretName": "mm_pd_bot_secret"}],
                         },
-                        'annotations': {'nginx.ingress.kubernetes.io/whitelist-source-range': '10.0.69.0/24'},
+                        "annotations": {"nginx.ingress.kubernetes.io/whitelist-source-range": "10.0.69.0/24"},
                     },
                 ],
             },
         },
     },
-    'ssl_without_whitelist': {
-        'config': {
-            'image_path': 'mm_pd_bot:devel',
-            'mm_pd_bot_cfg': get_cfg_content('ssl_configuration'),
-            'tls_secret_name': 'mm_pd_bot_secret',
+    "ssl_without_whitelist": {
+        "config": {
+            "image_path": "mm_pd_bot:devel",
+            "mm_pd_bot_cfg": get_cfg_content("ssl_configuration"),
+            "tls_secret_name": "mm_pd_bot_secret",
         },
-        'pod_spec': {
-            'version': 3,  # otherwise resources are ignored
-            'containers': [
+        "pod_spec": {
+            "version": 3,  # otherwise resources are ignored
+            "containers": [
                 {
-                    'name': 'mm-pd-bot',
-                    'imageDetails': {'imagePath': 'mm_pd_bot:devel'},
-                    'imagePullPolicy': 'Always',
-                    'ports': [{'containerPort': 2160, 'protocol': 'TCP'}],
-                    'envConfig': {'MM_PD_BOT_CFG': get_cfg_content('ssl_configuration')},
+                    "name": "mm-pd-bot",
+                    "imageDetails": {"imagePath": "mm_pd_bot:devel"},
+                    "imagePullPolicy": "Always",
+                    "ports": [{"containerPort": 2160, "protocol": "TCP"}],
+                    "envConfig": {"MM_PD_BOT_CFG": get_cfg_content("ssl_configuration")},
                 }
             ],
-            'kubernetesResources': {
-                'ingressResources': [
+            "kubernetesResources": {
+                "ingressResources": [
                     {
-                        'name': 'mm-pd-bot-ingress',
-                        'spec': {
-                            'rules': [
+                        "name": "mm-pd-bot-ingress",
+                        "spec": {
+                            "rules": [
                                 {
-                                    'host': 'mm-pd-bot.example.com',
-                                    'http': {
-                                        'paths': [
+                                    "host": "mm-pd-bot.example.com",
+                                    "http": {
+                                        "paths": [
                                             {
-                                                'path': '/bdddcacb-ab42-40ac-9106-4275c1db1519/',
-                                                'backend': {'serviceName': 'mm-pd-bot', 'servicePort': 2160},
+                                                "path": "/bdddcacb-ab42-40ac-9106-4275c1db1519/",
+                                                "backend": {"serviceName": "mm-pd-bot", "servicePort": 2160},
                                             },
                                         ],
                                     },
                                 },
                             ],
-                            'tls': [{'hosts': ['mm-pd-bot.example.com'], 'secretName': 'mm_pd_bot_secret'}],
+                            "tls": [{"hosts": ["mm-pd-bot.example.com"], "secretName": "mm_pd_bot_secret"}],
                         },
                     },
                 ],
diff --git a/tests/unit/test_charm.py b/tests/unit/test_charm.py
index 27bbaf3..d56fcde 100755
--- a/tests/unit/test_charm.py
+++ b/tests/unit/test_charm.py
@@ -39,8 +39,8 @@ class TestMmPdBotK8sCharmBlockedStatus(unittest.TestCase):
         """Cleanup the harness."""
         self.harness.cleanup()
 
-    @patch('charm.MmPdBotK8sCharm._validate_mm_pd_bot_configuration')
-    @patch('charm.MmPdBotK8sCharm._check_juju_settings')
+    @patch("charm.MmPdBotK8sCharm._validate_mm_pd_bot_configuration")
+    @patch("charm.MmPdBotK8sCharm._check_juju_settings")
     def test_check_for_config_problems(self, mock_juju, mock_mm_pd_bot):
         """Check the calls for config problems."""
         self.harness.charm._check_for_config_problems()
@@ -51,47 +51,47 @@ class TestMmPdBotK8sCharmBlockedStatus(unittest.TestCase):
         """Check the required juju settings."""
         for scenario, values in TEST_JUJU_SETTINGS.items():
             with self.subTest(scenario=scenario):
-                with self.assertLogs(level='ERROR') as logger:
+                with self.assertLogs(level="ERROR") as logger:
                     with self.assertRaises(MmPdBotK8sCharmConfigError) as exc:
                         self.harness.update_config(JUJU_SETTINGS_DEFAULT)  # Load the default values
-                        self.harness.update_config(values['config'])
+                        self.harness.update_config(values["config"])
                         self.harness.charm._check_juju_settings()
-                self.assertEqual(sorted(logger.output), sorted(values['logger']))
-                self.assertEqual(str(exc.exception), values['expected'])
+                self.assertEqual(sorted(logger.output), sorted(values["logger"]))
+                self.assertEqual(str(exc.exception), values["expected"])
             self.harness.update_config(JUJU_SETTINGS_DEFAULT)  # You need to clean the config after each run
 
     def test_validate_mm_pd_bot_configuration(self):
         """Check the MM_PD_BOT_CFG string is a valid INI."""
         for scenario, values in VALIDATE_MM_PD_BOT_CFG.items():
             with self.subTest(scenario=scenario):
-                self.harness.update_config(values['config'])
-                if values['expected'] is True:
+                self.harness.update_config(values["config"])
+                if values["expected"] is True:
                     self.harness.charm._validate_mm_pd_bot_configuration()
                 else:
-                    with self.assertLogs(level='ERROR') as logger:
+                    with self.assertLogs(level="ERROR") as logger:
                         with self.assertRaises(MmPdBotK8sCharmConfigError) as exc:
                             self.harness.charm._validate_mm_pd_bot_configuration()
-                    self.assertEqual(str(exc.exception), values['expected'])
-                    self.assertEqual(sorted(logger.output), sorted(values['logger']))
+                    self.assertEqual(str(exc.exception), values["expected"])
+                    self.assertEqual(sorted(logger.output), sorted(values["logger"]))
             self.harness.update_config(JUJU_SETTINGS_DEFAULT)  # You need to clean the config after each run
 
     def test_load_mm_pd_bot_configuration(self):
         """Test the loading of the configuration for the bot."""
-        config = VALIDATE_MM_PD_BOT_CFG['good_configuration']['config']['mm_pd_bot_cfg']
+        config = VALIDATE_MM_PD_BOT_CFG["good_configuration"]["config"]["mm_pd_bot_cfg"]
         loaded_config = self.harness.charm._load_mm_pd_bot_configuration(config)
         self.assertIsInstance(loaded_config, configparser.ConfigParser)
 
-        config = VALIDATE_MM_PD_BOT_CFG['bad_formatted_configuration']['config']['mm_pd_bot_cfg']
+        config = VALIDATE_MM_PD_BOT_CFG["bad_formatted_configuration"]["config"]["mm_pd_bot_cfg"]
         with self.assertRaises(MmPdBotK8sCharmConfigError) as exc:
             loaded_config = self.harness.charm._load_mm_pd_bot_configuration(config)
-        self.assertEqual(str(exc.exception), VALIDATE_MM_PD_BOT_CFG['bad_formatted_configuration']['expected'])
+        self.assertEqual(str(exc.exception), VALIDATE_MM_PD_BOT_CFG["bad_formatted_configuration"]["expected"])
 
     def test_configure_pod(self):
         """Test the pod configuration."""
         mock_event = MagicMock()
 
         # Good configuration but not leader
-        self.harness.update_config(VALIDATE_MM_PD_BOT_CFG['good_configuration']['config'])
+        self.harness.update_config(VALIDATE_MM_PD_BOT_CFG["good_configuration"]["config"])
         self.harness.set_leader(False)
         self.harness.charm.unit.status = BlockedStatus("Testing")
         self.harness.charm.configure_pod(mock_event)
@@ -99,11 +99,11 @@ class TestMmPdBotK8sCharmBlockedStatus(unittest.TestCase):
         self.harness.update_config(JUJU_SETTINGS_DEFAULT)  # You need to clean the config after each run
 
         # Good configuration and leader
-        self.harness.update_config(VALIDATE_MM_PD_BOT_CFG['good_configuration']['config'])
+        self.harness.update_config(VALIDATE_MM_PD_BOT_CFG["good_configuration"]["config"])
         self.harness.set_leader(True)
         self.harness.charm.unit.status = BlockedStatus("Testing")
         self.harness.charm.configure_pod(mock_event)
-        with self.assertLogs(level='INFO') as logger:
+        with self.assertLogs(level="INFO") as logger:
             self.harness.charm.configure_pod(mock_event)
         self.assertEqual(
             sorted(logger.output),
@@ -113,8 +113,8 @@ class TestMmPdBotK8sCharmBlockedStatus(unittest.TestCase):
         self.harness.update_config(JUJU_SETTINGS_DEFAULT)  # You need to clean the config after each run
 
         # Bad configuration and leader
-        config = VALIDATE_MM_PD_BOT_CFG['bad_formatted_configuration']['config']
-        expected = VALIDATE_MM_PD_BOT_CFG['bad_formatted_configuration']['expected']
+        config = VALIDATE_MM_PD_BOT_CFG["bad_formatted_configuration"]["config"]
+        expected = VALIDATE_MM_PD_BOT_CFG["bad_formatted_configuration"]["expected"]
         self.harness.update_config(config)
         self.harness.set_leader(True)
         self.harness.charm.configure_pod(mock_event)
@@ -125,26 +125,26 @@ class TestMmPdBotK8sCharmBlockedStatus(unittest.TestCase):
         """Check the crafting of the pod spec."""
         for scenario, values in VALIDATE_POD_SPEC.items():
             with self.subTest(scenario=scenario):
-                self.harness.update_config(values['config'])
+                self.harness.update_config(values["config"])
                 self.harness.charm.bot_config = self.harness.charm._load_mm_pd_bot_configuration(
-                    values['config']['mm_pd_bot_cfg']
+                    values["config"]["mm_pd_bot_cfg"]
                 )
-                self.assertEqual(self.harness.charm._make_pod_spec(), values['pod_spec'])
+                self.assertEqual(self.harness.charm._make_pod_spec(), values["pod_spec"])
                 self.harness.update_config(JUJU_SETTINGS_DEFAULT)  # You need to clean the config after each run
 
     def test_update_pod_spec_for_k8s_ingress(self):
         """Check the crafting of the ingress part of the pod spec."""
         for scenario, values in VALIDATE_POD_SPEC_AND_INGRESS.items():
             with self.subTest(scenario=scenario):
-                self.harness.update_config(values['config'])
+                self.harness.update_config(values["config"])
                 self.harness.charm.bot_config = self.harness.charm._load_mm_pd_bot_configuration(
-                    values['config']['mm_pd_bot_cfg']
+                    values["config"]["mm_pd_bot_cfg"]
                 )
                 pod_spec = self.harness.charm._make_pod_spec()
                 self.harness.charm._update_pod_spec_for_k8s_ingress(pod_spec)
-                self.assertEqual(pod_spec, values['pod_spec'])
+                self.assertEqual(pod_spec, values["pod_spec"])
                 self.harness.update_config(JUJU_SETTINGS_DEFAULT)  # You need to clean the config after each run
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     unittest.main()