[Bz 1468113] Configure the rest of the masters with the correct URL. #4701
Conversation
|
@sdodson, Running a test on this currently. |
| @@ -5,3 +5,12 @@ | |||
| hosts: oo_first_master | |||
| roles: | |||
| - openshift_metrics | |||
|
|
|||
| - name: OpenShift Metrics | |||
| hosts: oo_masters:!oo_first_master | |||
There was a problem hiding this comment.
serial: 1 to ensure it only hits one master at a time since it will be restarting the master.
There was a problem hiding this comment.
thanks, will update
kwoodson
force-pushed
the
master_metrics_url
branch
from
6f6e410 to
18fd0ab
Compare
|
@sdodson, ran a test and verified that it does serially call the other masters. |
|
aos-ci-test |
|
@sdodson, The travis tests failed because I had a trailing newline in my yaml file. I fixed that and pushed this back up earlier. |
|
aos-ci-test |
|
[merge][severity:blocker] |
|
Evaluated for openshift ansible merge up to 25a35d4 |
|
[test] |
|
Evaluated for openshift ansible test up to 25a35d4 |
|
continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_openshift_ansible/683/) (Base Commit: a974791) (PR Branch Commit: 25a35d4) (Extended Tests: blocker) |
|
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_openshift_ansible/331/) (Base Commit: cc3e1b8) (PR Branch Commit: 25a35d4) |
|
flake8 openshift/origin#10162 |
|
@sdodson, this flaked but rest looks ok. Do we want to run it through the gauntlet again? |
|
the test job passed, merging |
No description provided.