Fix master services list for etcd migration #6428
Conversation
openshift-ci-robot
added
the
size/XS
|
/assign mtnbikenc |
|
I've not made this change on master and I wonder if we should consider removing the etcd migration playbooks there. We require that the migration be completed prior to upgrading to 3.7 so they're no longer necessary. |
|
/test all [submit-queue is verifying that this PR is safe to merge] |
|
@sdodson: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
Automatic merge from submit-queue. |
We require that folks migrate to etcd v3 storage before upgrading to 3.7 and the 3.7 upgrade documentation also calls this out. So there's an incredibly strong chance that the 3.7 playbooks will be used to migrate from etcd v2 to v3. However in 3.6 if you had a single master it likely did not use the ha services. So check for that, this is slightly simplified logic copied from release-3.6 branch.