api icon indicating copy to clipboard operation
api copied to clipboard

OCPNODE-3029: features: add requiredMinimumComponentVersion

Open haircommander opened this issue 8 months ago • 7 comments
trafficstars

this is an optional field that allows a feature author to declare the minimum version a kubelet on an openshift node can be to use said feature.

This PR also sets UserNamespace related features to set their requiredMinimumKubeletVersion to 1.30.0.

haircommander avatar Mar 10 '25 18:03 haircommander

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[bot] avatar Mar 10 '25 18:03 openshift-ci[bot]

Please run the API linter over this and fix the reported issues make lint

there's a number of errors not related to the PR, should I fix these too?

haircommander avatar Mar 11 '25 17:03 haircommander

there's a number of errors not related to the PR, should I fix these too?

Make sure your local master branch is up to date, and it should only show changes since the master branch

JoelSpeed avatar Mar 11 '25 17:03 JoelSpeed

/retest

haircommander avatar Mar 11 '25 19:03 haircommander

@haircommander: This pull request references OCPNODE-3029 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.19.0" version, but no target version was set.

In response to this:

this is an optional field that allows a feature author to declare the minimum version a kubelet on an openshift node can be to use said feature.

This PR also sets UserNamespace related features to set their requiredMinimumKubeletVersion to 1.30.0.

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-robot avatar Mar 11 '25 19:03 openshift-ci-robot

[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 deads2k 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

openshift-ci[bot] avatar Mar 20 '25 17:03 openshift-ci[bot]

PR needs rebase.

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.

openshift-merge-robot avatar May 15 '25 09:05 openshift-merge-robot

@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/e2e-azure 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link false /test e2e-azure
ci/prow/e2e-gcp 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link false /test e2e-gcp
ci/prow/okd-scos-e2e-aws-ovn 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link false /test okd-scos-e2e-aws-ovn
ci/prow/e2e-upgrade-out-of-change 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-upgrade-out-of-change
ci/prow/e2e-aws-ovn-techpreview 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-aws-ovn-techpreview
ci/prow/e2e-aws-ovn 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-aws-ovn
ci/prow/e2e-aws-serial-techpreview 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-aws-serial-techpreview
ci/prow/verify-crd-schema 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test verify-crd-schema
ci/prow/e2e-aws-serial 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-aws-serial
ci/prow/minor-e2e-upgrade-minor 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test minor-e2e-upgrade-minor
ci/prow/e2e-upgrade 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-upgrade
ci/prow/e2e-aws-ovn-hypershift 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-aws-ovn-hypershift
ci/prow/verify-client-go 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test verify-client-go
ci/prow/e2e-aws-serial-techpreview-2of2 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-aws-serial-techpreview-2of2
ci/prow/e2e-aws-serial-techpreview-1of2 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-aws-serial-techpreview-1of2
ci/prow/e2e-aws-serial-2of2 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-aws-serial-2of2
ci/prow/e2e-aws-serial-1of2 0e36791986b1c0aee02cd3bfe3c4889e7c873163 link true /test e2e-aws-serial-1of2

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.

openshift-ci[bot] avatar May 15 '25 09:05 openshift-ci[bot]

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close. Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

openshift-bot avatar Aug 14 '25 01:08 openshift-bot