Implementation of ops requires side of aws interface

ops/ops/interface_aws/requires.py

@@ -130,14 +130,22 @@ def _joined(self, _):
             self.instance_id,
             self.region,
         )
-        self._to_publish["instance-id"] = self.instance_id
-        self._to_publish["region"] = self.region
+        self._request({"instance-id": self.instance_id, "region": self.region})
 
     @property
     def is_ready(self):
         completed = json.loads(self._received.get("completed", "{}"))
         response_hash = completed.get(self.instance_id)
-        return response_hash == self._expected_hash
+        ready = response_hash == self._expected_hash
+        if not response_hash:
+            log.warning("Remote end is yet to calculate a response")
+        elif not ready:
+            log.warning(
+                "Waiting for response_hash=%s to be self._expected_hash=%s",
+                response_hash,
+                self._expected_hash,
+            )
+        return ready
 
     def evaluate_relation(self, event) -> Optional[str]:
         """Determine if relation is ready."""
@@ -168,9 +176,14 @@ def region(self):
 
     @property
     def _expected_hash(self):
-        return sha256(
-            json.dumps(dict(self._to_publish), sort_keys=True).encode("utf8")
-        ).hexdigest()
+        def from_json(s: str):
+            try:
+                return json.loads(s)
+            except json.decoder.JSONDecodeError:
+                return s
+
+        to_sha = {key: from_json(val) for key, val in self._to_publish.items()}
+        return sha256(json.dumps(to_sha, sort_keys=True).encode()).hexdigest()
 
     def _request(self, keyvals):
         kwds = {key: json.dumps(val) for key, val in keyvals.items()}

ops/tests/data/aws_recv.yaml

@@ -1 +1 @@
-completed: '{"i-abcdefghijklmnopq": "1fca8013323a5aaa67a9a816ab1b910a2402165c198f8502c69d7c0b2a0546cf"}'
+completed: '{"i-abcdefghijklmnopq": "3577342edb3a0a9a4b6861b7d2f580f35db2709e92f3ddd17bb027eb85a2f670"}'