Skip to content

Commit

Permalink
Merge pull request #7222 from enj/patch-2
Browse files Browse the repository at this point in the history
Automatic merge from submit-queue.

Update API healthz check to use uri module

This should prevent issues with curl getting upset with certs that have the same serial number and name.

See openshift/origin#18405

/assign @sdodson
  • Loading branch information
openshift-merge-robot committed Feb 20, 2018
2 parents 63dbe26 + 79e283a commit 0b072f5
Showing 1 changed file with 5 additions and 7 deletions.
12 changes: 5 additions & 7 deletions roles/openshift_service_catalog/tasks/start_api_server.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,12 @@

# wait to see that the apiserver is available
- name: wait for api server to be ready
command: >
curl --noproxy '*' -k https://apiserver.kube-service-catalog.svc/healthz
args:
# Disables the following warning:
# Consider using get_url or uri module rather than running curl
warn: no
uri:
url: https://apiserver.kube-service-catalog.svc/healthz
validate_certs: no
return_content: yes
register: api_health
until: api_health.stdout == 'ok'
until: "'ok' in api_health.content"
retries: 60
delay: 5
changed_when: false

0 comments on commit 0b072f5

Please sign in to comment.