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
NO-JIRA: Clean carry patches #1880
Conversation
@soltysh: This pull request explicitly references no jira issue. In response to this: 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 openshift-eng/jira-lifecycle-plugin repository. |
/test verify |
@soltysh: the contents of this pull request could not be automatically validated. The following commits could not be validated and must be approved by a top-level approver:
Comment |
Skipping CI for Draft Pull Request. |
b5b3725
to
0f57a95
Compare
@soltysh: the contents of this pull request could not be automatically validated. The following commits could not be validated and must be approved by a top-level approver:
Comment |
/remove-label backports/unvalidated-commits |
@soltysh: Those labels are not set on the issue: In response to this:
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. |
openshift-hack/e2e/namespace.go
Outdated
@@ -33,6 +33,9 @@ func CreateTestingNS(ctx context.Context, baseName string, c kclientset.Interfac | |||
labels = map[string]string{} | |||
} | |||
labels["security.openshift.io/disable-securitycontextconstraints"] = "true" | |||
// turn off the OpenShift label syncer so that it does not attempt to sync |
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.
when is "isKubeNamespace" false?
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.
In origin we have it set here: https://github.com/openshift/origin/blob/master/test/extended/util/test_setup.go#L89-L91, in k8s it's always true https://github.com/openshift/kubernetes/blob/master/openshift-hack/cmd/k8s-tests/provider.go#L103-L105
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.
make it apply for all namespaces regardless of their name, please
openshift-hack/e2e/namespace.go
Outdated
@@ -33,6 +33,9 @@ func CreateTestingNS(ctx context.Context, baseName string, c kclientset.Interfac | |||
labels = map[string]string{} | |||
} | |||
labels["security.openshift.io/disable-securitycontextconstraints"] = "true" | |||
// turn off the OpenShift label syncer so that it does not attempt to sync |
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.
make it apply for all namespaces regardless of their name, please
0f57a95
to
aa46a65
Compare
@soltysh: the contents of this pull request could not be automatically validated. The following commits could not be validated and must be approved by a top-level approver:
Comment |
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 auth-wise.
Leaving review to @dinhxuanvu for the API things.
/remove-label backports/unvalidated-commits |
@soltysh: Those labels are not set on the issue: In response to this:
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. |
/label backports/validated-commits |
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dinhxuanvu, soltysh The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
We have sufficient proof in openshift/origin#28588 |
@soltysh: all tests passed! Full PR test history. Your PR dashboard. 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. |
[ART PR BUILD NOTIFIER] This PR has been included in build openshift-enterprise-pod-container-v4.16.0-202402152340.p0.gedc2c12.assembly.stream.el9 for distgit openshift-enterprise-pod. |
No description provided.