Use juju provided ip instead of resolved one (#558)

Author: paulomach

src/hostname_resolution.py

@@ -50,14 +50,7 @@ def _update_host_details_in_databag(self, _) -> None:
         hostname = socket.gethostname()
         fqdn = socket.getfqdn()
 
-        s = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
-        s.settimeout(0)
-        try:
-            s.connect(("10.10.10.10", 1))
-            ip = s.getsockname()[0]
-        except Exception:
-            logger.exception("Unable to get local IP address")
-            ip = "127.0.0.1"
+        ip = str(self.model.get_binding(PEER).network.bind_address)
 
         host_details = {"names": [hostname, fqdn], "address": ip}
 

tests/unit/test_config.py

@@ -10,6 +10,7 @@
 from ops.testing import Harness
 
 from charm import MySQLOperatorCharm
+from constants import PEER
 
 CONFIG = str(yaml.safe_load(Path("./config.yaml").read_text()))
 ACTIONS = str(yaml.safe_load(Path("./actions.yaml").read_text()))
@@ -22,6 +23,7 @@
 def harness():
     harness = Harness(MySQLOperatorCharm, meta=METADATA, config=CONFIG, actions=ACTIONS)
     harness.add_relation("restart", "mysql")
+    harness.add_relation(PEER, "mysql")
     harness.begin()
     return harness