Remove NoVolumeNodeConflict from 3.9+

[sdodson]

No description provided.

[sdodson]

/assign aveshagarwal

[aveshagarwal]

@sdodson lgtm except as we discussed or IRC that CheckVolumeBinding is missing.

[sdodson]

@michaelgugino should we get rid of roles/openshift_master_facts/lookup_plugins/openshift_master_facts_default_predicates.py at this point? Seems wholely duplicated from roles/lib_utils/lookup_plugins/openshift_master_facts_default_predicates.py ?

[michaelgugino]

@sdodson that sounds fine to me. I might have missed that one when I was doing the refactoring.

I left the addon (logging, metrics) roles' plugins somewhat intact to try to keep those somewhat portable. If it's part of core openshift, it should most likely be in lib_utils.

[sdodson]

@aveshagarwal added CheckVolumeBinding
/retest

[aveshagarwal]

@sdodson thanks. lgtm for predicates. Not sure I can provide lgtm here as I have not reviewed other changes.

[sdodson]

/retest

[openshift-merge-robot]

/test all [submit-queue is verifying that this PR is safe to merge]

[michaelgugino]

bot, retest this please

[sdodson]

/test system-containers

[openshift-merge-robot]

/test all [submit-queue is verifying that this PR is safe to merge]

[openshift-ci-robot]

@sdodson: The following tests failed, say /retest to rerun them all:

Test name	Commit	Details	Rerun command
ci/openshift-jenkins/logging	df793e9	link	/test logging
ci/openshift-jenkins/install	df793e9	link	/test install
ci/openshift-jenkins/system-containers	df793e9	link	/test system-containers
ci/openshift-jenkins/extended_conformance_install_crio	df793e9	link	/test crio

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.