Skip to content

OCPNODE-3225,OCPNODE-2557: features: set user namespace features on by default #2305

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

haircommander
Copy link
Member

now that we are approaching kube 1.33 rebase (and are moving away from 1.32), we are safe from the risks that caused us not to turn this on by default (needing a n-3 version skew to 1.30 to make sure a user namespace is created or a pod is denied)

Copy link
Contributor

openshift-ci bot commented May 2, 2025

Hello @haircommander! Some important instructions when contributing to openshift/api:
API design plays an important part in the user experience of OpenShift and as such API PRs are subject to a high level of scrutiny to ensure they follow our best practices. If you haven't already done so, please review the OpenShift API Conventions and ensure that your proposed changes are compliant. Following these conventions will help expedite the api review process for your PR.

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label May 2, 2025
@openshift-ci openshift-ci bot requested review from deads2k and JoelSpeed May 2, 2025 13:10
Copy link
Contributor

openshift-ci bot commented May 2, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: haircommander
Once this PR has been reviewed and has the lgtm label, please assign joelspeed for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@haircommander haircommander changed the title features: set user namespace features on by default OCPNODE-3225,OCPNODE-2557: features: set user namespace features on by default May 2, 2025
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label May 2, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented May 2, 2025

@haircommander: This pull request references OCPNODE-3225 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.20.0" version, but no target version was set.

This pull request references OCPNODE-2557 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.20.0" version, but no target version was set.

In response to this:

now that we are approaching kube 1.33 rebase (and are moving away from 1.32), we are safe from the risks that caused us not to turn this on by default (needing a n-3 version skew to 1.30 to make sure a user namespace is created or a pod is denied)

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.

@openshift-ci openshift-ci bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 2, 2025
@openshift-ci openshift-ci bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 2, 2025
@@ -784,7 +784,7 @@ spec:
/<datacenter>/network/<portgroup>.
items:
type: string
maxItems: 10
maxItems: 1
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'm unsure why this snuck in

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make sure you're rebased on the latest tip of master and try the generation again 🤔

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this happened to me on master too, but I just dropped manually

@bertinatto
Copy link
Member

/retest

@haircommander haircommander force-pushed the userns-default branch 2 times, most recently from 3e545e7 to f08ce86 Compare May 6, 2025 14:11
@haircommander
Copy link
Member Author

/retest

I think the remaining verify failures are expected. We have this weird deadlock where upstream userns tests will fail because the feature is turned off, but they're only enabled in tech preview clusters so we don't have any feedback that they're working. If needed, I could duplicate the upstream tests in o/origin to mimic the upstream behaviors, but we still would need to wait a bit to get test readiness. OR we can override.

I ask because this is blocking @bertinatto 's rebase and needs timely consideration. WDYT @JoelSpeed

Copy link
Contributor

openshift-ci bot commented May 7, 2025

@haircommander: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/okd-scos-e2e-aws-ovn 0f9a8e8 link false /test okd-scos-e2e-aws-ovn
ci/prow/verify-feature-promotion 0f9a8e8 link true /test verify-feature-promotion
ci/prow/e2e-aws-serial 0f9a8e8 link true /test e2e-aws-serial
ci/prow/verify-crd-schema 0f9a8e8 link true /test verify-crd-schema

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-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants