Skip to content

[release-4.22] AGENT-1544: Backport NoRegistryClusterInstall feature to release-4.22#2930

Open
pawanpinjarkar wants to merge 5 commits into
openshift:release-4.22from
pawanpinjarkar:backport-2880-to-release-4.22
Open

[release-4.22] AGENT-1544: Backport NoRegistryClusterInstall feature to release-4.22#2930
pawanpinjarkar wants to merge 5 commits into
openshift:release-4.22from
pawanpinjarkar:backport-2880-to-release-4.22

Conversation

@pawanpinjarkar

@pawanpinjarkar pawanpinjarkar commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

This patch is part of the NoRegistryClusterInstall feature backport to 4.22. It contains the following PR:

#2880

sadasu and others added 5 commits July 13, 2026 17:05
- Created machineconfiguration/v1/types_internalreleaseimage.go with v1 API
- Updated compatibility level from 4 (alpha) to 1 (stable/GA)
- Registered InternalReleaseImage types in v1 scheme
- Created v1 integration test suite
- Updated payload CRD script to use v1 instead of v1alpha1
- Generated all CRD manifests, deepcopy, and swagger docs

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Add manual CRD override to include v1alpha1 schema alongside v1 schema
for InternalReleaseImage API. This allows both API versions to coexist
during the graduation period, enabling gradual migration from v1alpha1
to v1 across dependent repositories.

The manual override contains the complete v1alpha1 schema from the
featuregated manifest, which will be merged with the v1 schema during
code generation to produce a CRD that serves both versions.

This addresses the review feedback to support both schemas simultaneously
rather than requiring all consumers to update at once.

Related: openshift#2880 (comment)
During code generation, the manifest-merge tool combines this override with the v1
schema generated from Go types to produce a final CRD that serves both versions:

- v1 (storage: true): New stable API with MaxItems=5 for Conditions
- v1alpha1 (storage: false): Original alpha API with MaxItems=20 for Conditions

Both versions are served, but v1 is the storage version. Resources submitted
via v1alpha1 are automatically converted to v1 for storage in etcd, then
converted back to v1alpha1 when read, ensuring backward compatibility without
requiring simultaneous updates across all consuming repositories.
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: LGTM mode

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jul 13, 2026
@openshift-ci-robot

openshift-ci-robot commented Jul 13, 2026

Copy link
Copy Markdown

@pawanpinjarkar: This pull request references AGENT-1500 which is a valid jira issue.

Details

In response to this:

Backport #2880 to 4.22

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 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Hello @pawanpinjarkar! 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.

@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: a6f477ed-4d32-4305-8dda-99e42dff016f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@openshift-ci openshift-ci Bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jul 13, 2026
@openshift-ci openshift-ci Bot requested review from everettraven and jkyros July 13, 2026 21:39
@pawanpinjarkar

Copy link
Copy Markdown
Contributor Author

/cc @andfasano @bfournie

@openshift-ci openshift-ci Bot requested review from andfasano and bfournie July 13, 2026 21:57
@pawanpinjarkar pawanpinjarkar changed the title AGENT-1500: Backport 2880 to release 4.22 AGENT-1500: NoRegistryClusterInstall Feature Promotion (4.22 backport) Jul 13, 2026
@andfasano

andfasano commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

/retitle [release-4.22] AGENT-1544: Backport NoRegistryClusterInstall feature to release-4.22

@openshift-ci openshift-ci Bot changed the title AGENT-1500: NoRegistryClusterInstall Feature Promotion (4.22 backport) [release-4.22] AGENT-1544: Backport NoRegistryClusterInstall feature to release-4.22 Jul 14, 2026
@openshift-ci-robot

openshift-ci-robot commented Jul 14, 2026

Copy link
Copy Markdown

@pawanpinjarkar: This pull request references AGENT-1544 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.22.0" version, but no target version was set.

Details

In response to this:

Cherrypick #2880 to 4.22

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.

@andfasano

Copy link
Copy Markdown
Contributor

/hold

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 14, 2026
@andfasano

Copy link
Copy Markdown
Contributor

/hold cancel

@openshift-ci openshift-ci Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 14, 2026
@pawanpinjarkar

Copy link
Copy Markdown
Contributor Author

/test verify

@pawanpinjarkar

Copy link
Copy Markdown
Contributor Author

/assign @yuqi-zhang

@andfasano

Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 14, 2026
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

Scheduling tests matching the pipeline_run_if_changed or not excluded by pipeline_skip_if_only_changed parameters:
/test e2e-aws-ovn
/test e2e-aws-ovn-hypershift
/test e2e-aws-ovn-hypershift-conformance
/test e2e-aws-ovn-techpreview
/test e2e-aws-serial-1of2
/test e2e-aws-serial-2of2
/test e2e-aws-serial-techpreview-1of2
/test e2e-aws-serial-techpreview-2of2
/test e2e-azure
/test e2e-gcp
/test e2e-upgrade
/test e2e-upgrade-out-of-change

@openshift-ci

openshift-ci Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: andfasano
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.

Details 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

@pawanpinjarkar

Copy link
Copy Markdown
Contributor Author

/jira refresh

@openshift-ci-robot

openshift-ci-robot commented Jul 14, 2026

Copy link
Copy Markdown

@pawanpinjarkar: This pull request references AGENT-1544 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.22.0" version, but no target version was set.

Details

In response to this:

/jira refresh

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.

@yuqi-zhang

Copy link
Copy Markdown
Contributor

In 5.0 it was a bit safer since we were doing the switch on an unreleased version, but here in 4.22 I think we would need to be a bit more careful. Deferring to @JoelSpeed on what the best path forward is (this looks like it's directly switching the payload CRD, I guess that's fine since it won't be active until we GA the featuregate?)

@openshift-ci

openshift-ci Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

@pawanpinjarkar: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.

@pawanpinjarkar

Copy link
Copy Markdown
Contributor Author

/label backport-risk-assessed

@openshift-ci

openshift-ci Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

@pawanpinjarkar: The label(s) backport-risk-assessed cannot be applied or removed, because you are not in one of the allowed teams and are not an allowed user. Must be a member of one of these teams: openshift-patch-managers, openshift-release-oversight, openshift-staff-engineers, openshift-sustaining-engineers

Details

In response to this:

/label backport-risk-assessed

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.

@andfasano

Copy link
Copy Markdown
Contributor

In 5.0 it was a bit safer since we were doing the switch on an unreleased version, but here in 4.22 I think we would need to be a bit more careful. Deferring to @JoelSpeed on what the best path forward is (this looks like it's directly switching the payload CRD, I guess that's fine since it won't be active until we GA the featuregate?)

@yuqi-zhang I think it should be safe to have both v1alpha1 and v1 defined in the CRD, anyhow I understand that from a point of view of the 4.22 branch is not something strictly required.

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. lgtm Indicates that a PR is ready to be merged. 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.

6 participants