-
Notifications
You must be signed in to change notification settings - Fork 4.7k
NetworkCheck diagnostic: create projects with empty nodeselector #14686
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NetworkCheck diagnostic: create projects with empty nodeselector #14686
Conversation
@pravisankar any concerns? I wasn't sure if specifying the one annotation was going to get in the way of whatever default annotations were supposed to be, but it doesn't look like it's a problem in my tests. |
fix bug 1431588 https://bugzilla.redhat.com/show_bug.cgi?id=1431588 fix bug 1459241 https://bugzilla.redhat.com/show_bug.cgi?id=1459241 When creating the projects in which the test pods will deploy, create them with an empty node selector so that they can run on any nodes, not just nodes with the cluster default node selector.
7760b10
to
4ad8129
Compare
@sosiouxme No issues. Default node selector is only applied to the project if it doesn't have its own node selector. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
[merge][severity: bug] |
Evaluated for origin merge up to 4ad8129 |
[Test]ing while waiting on the merge queue |
Evaluated for origin test up to 4ad8129 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/2302/) (Base Commit: a68db68) (PR Branch Commit: 4ad8129) |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/1030/) (Base Commit: 7b0131b) (PR Branch Commit: 4ad8129) (Extended Tests: bug) (Image: devenv-rhel7_6374) |
fix bug 1431588 https://bugzilla.redhat.com/show_bug.cgi?id=1431588
fix bug 1459241 https://bugzilla.redhat.com/show_bug.cgi?id=1459241
When creating the projects in which the test pods will deploy, create
them with an empty node selector so that they can run on any nodes, not
just nodes with the cluster default node selector.