diff options
| author | Jeff Cantrill <jcantril@redhat.com> | 2017-03-10 15:18:12 -0500 |
|---|---|---|
| committer | Jeff Cantrill <jcantril@redhat.com> | 2017-03-10 15:18:12 -0500 |
| commit | 3c586308f6abe4451b022833291068900fe102fb (patch) | |
| tree | 7345922089e8eea82dfe6f568e36cc9bd5c75472 | |
| parent | 646195dd74f22c8f26da0f80b12ebd1955e9f511 (diff) | |
| download | openshift-3c586308f6abe4451b022833291068900fe102fb.tar.gz openshift-3c586308f6abe4451b022833291068900fe102fb.tar.bz2 openshift-3c586308f6abe4451b022833291068900fe102fb.tar.xz openshift-3c586308f6abe4451b022833291068900fe102fb.zip | |
bug 1430661. Update masterConfig metricsPublicURL on install
| -rw-r--r-- | roles/openshift_metrics/handlers/main.yml | 26 | ||||
| -rw-r--r-- | roles/openshift_metrics/tasks/install_metrics.yaml | 2 | ||||
| -rw-r--r-- | roles/openshift_metrics/tasks/update_master_config.yaml | 9 |
3 files changed, 37 insertions, 0 deletions
diff --git a/roles/openshift_metrics/handlers/main.yml b/roles/openshift_metrics/handlers/main.yml new file mode 100644 index 000000000..ffb812271 --- /dev/null +++ b/roles/openshift_metrics/handlers/main.yml @@ -0,0 +1,26 @@ +--- +- name: restart master + systemd: name={{ openshift.common.service_type }}-master state=restarted + when: (openshift.master.ha is not defined or not openshift.master.ha | bool) and (not (master_service_status_changed | default(false) | bool)) + notify: Verify API Server + +- name: Verify API Server + # Using curl here since the uri module requires python-httplib2 and + # wait_for port doesn't provide health information. + command: > + curl --silent --tlsv1.2 + {% if openshift.common.version_gte_3_2_or_1_2 | bool %} + --cacert {{ openshift.common.config_base }}/master/ca-bundle.crt + {% else %} + --cacert {{ openshift.common.config_base }}/master/ca.crt + {% endif %} + {{ openshift.master.api_url }}/healthz/ready + args: + # Disables the following warning: + # Consider using get_url or uri module rather than running curl + warn: no + register: api_available_output + until: api_available_output.stdout == 'ok' + retries: 120 + delay: 1 + changed_when: false diff --git a/roles/openshift_metrics/tasks/install_metrics.yaml b/roles/openshift_metrics/tasks/install_metrics.yaml index 66a3abdbd..ffe6f63a2 100644 --- a/roles/openshift_metrics/tasks/install_metrics.yaml +++ b/roles/openshift_metrics/tasks/install_metrics.yaml @@ -34,6 +34,8 @@ file_content: "{{ item.content | b64decode | from_yaml }}" with_items: "{{ object_defs.results }}" +- include: update_master_config.yaml + - command: > {{openshift.common.client_binary}} --config={{mktemp.stdout}}/admin.kubeconfig diff --git a/roles/openshift_metrics/tasks/update_master_config.yaml b/roles/openshift_metrics/tasks/update_master_config.yaml new file mode 100644 index 000000000..93054b99b --- /dev/null +++ b/roles/openshift_metrics/tasks/update_master_config.yaml @@ -0,0 +1,9 @@ +--- +- name: Adding metrics route information to metricsPublicURL + modify_yaml: + dest: "{{ openshift.common.config_base }}/master/master-config.yaml" + yaml_key: assetConfig.metricsPublicURL + yaml_value: "https://{{ openshift_metrics_hawkular_hostname}}" + notify: restart master + tags: + - update_master_config |