Merge pull request #55 from projectsyn/allow-setting-ns-metadata

Allow setting namespace metadata using `.namespaceMetadata`

Author: bastjan

class/defaults.yml

@@ -3,6 +3,7 @@ parameters:
     =_metadata:
       multi_instance: true
     namespace: syn-${_instance}
+    namespaceMetadata: {}
 
     host_service_cidr: null
 

component/main.jsonnet

@@ -10,6 +10,8 @@ local instance = inv.parameters._instance;
 
 // Define outputs below
 {
-  '00_namespace': kube.Namespace(params.namespace),
+  '00_namespace': kube.Namespace(params.namespace) {
+    metadata+: com.makeMergeable(params.namespaceMetadata),
+  },
   '10_cluster': cluster.Cluster(instance, params),
 }

docs/modules/ROOT/pages/references/parameters.adoc

@@ -14,6 +14,24 @@ The namespace in which to deploy this component.
 
 Deploying multiple vclusters in the same namespace isn't supported.
 
+
+== `namespaceMetadata`
+
+[horizontal]
+type:: dictionary
+default:: `{}`
+example::
++
+[source,yaml]
+----
+namespace:
+  annotations:
+    prometheus: infra
+----
+
+Additional metadata for the namespace.
+
+
 == `host_service_cidr`
 
 [horizontal]

tests/golden/openshift/openshift/openshift/00_namespace.yaml

@@ -1,7 +1,8 @@
 apiVersion: v1
 kind: Namespace
 metadata:
-  annotations: {}
+  annotations:
+    openshift.io/cluster-monitoring: 'true'
   labels:
     name: syn-openshift
   name: syn-openshift

tests/openshift.yml

@@ -3,6 +3,9 @@ parameters:
     host_service_cidr: 172.30.0.0/16
     ocp_route:
       host: test.apps.local
+    namespaceMetadata:
+      annotations:
+        openshift.io/cluster-monitoring: "true"
 
   facts:
     distribution: openshift4