Merge ~barryprice/charm-k8s-wordpress/+git/charm-k8s-wordpress:master into charm-k8s-wordpress:master

[Barry Price]

Add an Ingress resource to our pod spec, hard-coded port for now

3ab42f7... by Barry Price on 2020-05-21

Typos

[Barry Price (barryprice)]

n.b. the current target cluster doesn't recognise "startupProbe" so please do not merge until I've got (at least) that issue figured out :)

cd483ec... by Barry Price on 2020-05-22

Drop probes temporarily as we're seeing parsing issues, add 'version: 2' to activate ingress resource

[Barry Price (barryprice)]

This now deploys cleanly, but defines no probes.

It can be merged as-is, as long as re-adding probes is taken care of at some point.

Unmerged commits

cd483ec... by Barry Price on 2020-05-22

Drop probes temporarily as we're seeing parsing issues, add 'version: 2' to activate ingress resource

3ab42f7... by Barry Price on 2020-05-21

Typos

e0627a7... by Barry Price on 2020-05-21

Add an Ingress resource to our pod spec, hard-coded port for now

diff --git a/src/charm.py b/src/charm.py
index 69c317d..147eda4 100755
--- a/src/charm.py
+++ b/src/charm.py
@@ -144,11 +144,12 @@ class WordpressK8sCharm(CharmBase):
         # Only the leader can set_spec().
         if self.model.unit.is_leader():
             spec = self.make_pod_spec()
+            resources = self.make_pod_resources()
 
             msg = "Configuring pod"
             logger.info(msg)
             self.model.unit.status = MaintenanceStatus(msg)
-            self.model.pod.set_spec(spec)
+            self.model.pod.set_spec(spec, resources)
 
             msg = "Pod configured"
             logger.info(msg)
@@ -167,15 +168,15 @@ class WordpressK8sCharm(CharmBase):
         ]
 
         spec = {
-            "containers": [
-                {
-                    "name": self.app.name,
-                    "imageDetails": {"imagePath": config["image"]},
-                    "ports": ports,
-                    "config": secure_pod_config,
-                    "readinessProbe": {"exec": {"command": ["/bin/cat", "/srv/wordpress-helpers/.ready"]}},
-                }
-            ]
+            "version": 2,
+            "containers": [{
+                "name": self.app.name,
+                "imageDetails": {
+                    "imagePath": config["image"]
+                },
+                "ports": ports,
+                "config": secure_pod_config,
+            }]
         }
 
         out = io.StringIO()
@@ -190,6 +191,34 @@ class WordpressK8sCharm(CharmBase):
 
         return spec
 
+    def make_pod_resources(self):
+        resources = {
+            "kubernetesResources": {
+                "ingressResources": [{
+                    "name": "{}-ingress".format(self.app.name),
+                    "spec": {
+                        "rules": [{
+                            "http": {
+                                "paths": [{
+                                    "path": "/",
+                                    "backend": {
+                                        "serviceName": self.app.name,
+                                        "servicePort": 80
+                                    }
+                                }]
+                            }
+                        }]
+                    }
+                }]
+            }
+        }
+
+        out = io.StringIO()
+        pprint(resources, out)
+        logger.info("This is the Kubernetes Pod resources <<EOM\n{}\nEOM".format(out.getvalue()))
+
+        return resources
+
     def is_valid_config(self):
         is_valid = True
         config = self.model.config