openshift / openshift-ansible / 12290

51%

push

travis-ci

Merge pull request #6687 from jpeeler/sc-cert-fix

Automatic merge from submit-queue.

Update deployment and apiserver with new certs

Since new certificates are generated for every run, the apiservice
caBundle needs updating in order to have the on disk CA match what is in
Kubernetes.

Because the secrets are updated, the daemonset needs to do a rolling
update for the apiserver to pick up the new certs. Implemented here is
an added annotation to the api server such that the update occurs
automatically when the CA is changed.

---

There may be a better way to make the rolling update occur without adding an annotation, such as within ansible itself (I just didn't know how to do that). Also, I think that probably the controller needs to be updated too in order to ensure staying in sync with the api server.
Edit: I did not think that doing "oc apply" would cause the daemonset to pick up a change since the yaml may be exactly the same, but if it does then the annotation part can definitely be changed.

2322 of 5830 branches covered (39.83%)

7390 of 13437 relevant lines covered (55.0%)

1.09 hits per line