Browse Source

Merge pull request #6143 from dabelenda/add_hostname_routes

Allow to set the hostname for routes to prometheus and alertmanager
Michael Gugino 7 years ago
parent
commit
9dbcf7bfda

+ 4 - 0
roles/openshift_prometheus/README.md

@@ -25,6 +25,10 @@ For default values, see [`defaults/main.yaml`](defaults/main.yaml).
 
 - `openshift_prometheus_args`: Modify or add arguments for prometheus application
 
+- `openshift_prometheus_hostname`: specify the hostname for the route to prometheus `prometheus-{{openshift_prometheus_namespace}}.{{openshift_master_default_subdomain}}`
+
+- `openshift_prometheus_alerts_hostname`: specify the hostname for the route to prometheus-alerts `prometheus_alerts-{{openshift_prometheus_namespace}}.{{openshift_master_default_subdomain}}`
+
 e.g
 ```
 openshift_prometheus_args=['--storage.tsdb.retention=6h', '--storage.tsdb.min-block-duration=5s', '--storage.tsdb.max-block-duration=6m']

+ 4 - 0
roles/openshift_prometheus/defaults/main.yaml

@@ -4,6 +4,10 @@ openshift_prometheus_state: present
 
 openshift_prometheus_namespace: openshift-metrics
 
+# defaults hosts for routes
+openshift_prometheus_hostname: prometheus-{{openshift_prometheus_namespace}}.{{openshift_master_default_subdomain}}
+openshift_prometheus_alerts_hostname: alerts-{{openshift_prometheus_namespace}}.{{openshift_master_default_subdomain}}
+
 openshift_prometheus_node_selector: {"region":"infra"}
 
 # additional prometheus rules file

+ 3 - 0
roles/openshift_prometheus/tasks/install_prometheus.yaml

@@ -113,12 +113,15 @@
   oc_route:
     state: present
     name: "{{ item.name }}"
+    host: "{{ item.host }}"
     namespace: "{{ openshift_prometheus_namespace }}"
     service_name: "{{ item.name }}"
     tls_termination: reencrypt
   with_items:
     - name: prometheus
+      host: "{{ openshift_prometheus_hostname }}"
     - name: alerts
+      host: "{{ openshift_prometheus_alerts_hostname }}"
 
 # Storage
 - name: create prometheus pvc