Fixing kafka_port validation in RO charm

Change-Id: I5d7ea4ecd3f45e901d0da431e386580f5c49611c
Signed-off-by: sousaedu <eduardo.sousa@canonical.com>

installers/charm/ro/src/pod_spec.py

@@ -65,7 +65,7 @@ def _validate_data(
         "kafka_host": lambda value, _: (isinstance(value, str) and len(value) > 0)
         if config_data.get("enable_ng_ro", True)
         else True,
-        "kafka_port": lambda value, _: (isinstance(value, int) and value > 0)
+        "kafka_port": lambda value, _: (isinstance(value, str) and len(value) > 0)
         if config_data.get("enable_ng_ro", True)
         else True,
         "mongodb_connection_string": lambda value, _: (

installers/charm/ro/tests/test_charm.py

@@ -91,7 +91,7 @@ class TestCharm(unittest.TestCase):
                         "OSMRO_LOG_LEVEL": "INFO",
                         "OSMRO_MESSAGE_DRIVER": "kafka",
                         "OSMRO_MESSAGE_HOST": "kafka",
-                        "OSMRO_MESSAGE_PORT": 9090,
+                        "OSMRO_MESSAGE_PORT": "9090",
                         "OSMRO_DATABASE_DRIVER": "mongo",
                         "OSMRO_DATABASE_URI": "mongodb://mongo",
                         "OSMRO_DATABASE_COMMONKEY": "osm",
@@ -139,7 +139,7 @@ class TestCharm(unittest.TestCase):
             "kafka/0",
             {
                 "host": "kafka",
-                "port": 9090,
+                "port": "9090",
             },
         )
 

installers/charm/ro/tests/test_pod_spec.py

@@ -54,7 +54,7 @@ class TestPodSpec(unittest.TestCase):
         }
         relation_state = {
             "kafka_host": "kafka",
-            "kafka_port": 9090,
+            "kafka_port": "9090",
             "mongodb_connection_string": "mongodb://mongo",
         }
 
@@ -171,7 +171,7 @@ class TestPodSpec(unittest.TestCase):
         }
         relation_state = {
             "kafka_host": "kafka",
-            "kafka_port": 9090,
+            "kafka_port": "9090",
             "mongodb_connection_string": "mongodb://mongo",
         }
         app_name = "ro"