openshift / openshift-ansible / 8363

51%

push

travis-ci

Merge pull request #5392 from ingvagabund/pull-openshift_master-deps-out-into-a-play

Automatic merge from submit-queue

Pull openshift_master deps out into a play

The `openshift_master` role is called only in a single play. Thus, we can pull out all its dependencies without duplicating all dependency role invocations. Both `lib_openshift` and `lib_os_firewall` are required deps as they defined ansible modules used inside the `openshift_master` role.

I have also rearranged definition of variables so variable used only inside a single role are part of the `include_role` statement.

Atm, we can't use `include_role` due to https://github.com/ansible/ansible/issues/21890

2239 of 5748 branches covered (38.95%)

7179 of 13256 relevant lines covered (54.16%)

1.07 hits per line