Skip to content
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

Fix CSV for 1.5.3 #401

Merged
merged 2 commits into from Apr 10, 2024
Merged

Fix CSV for 1.5.3 #401

merged 2 commits into from Apr 10, 2024

Conversation

gkurz
Copy link
Member

@gkurz gkurz commented Apr 10, 2024

- Description of the problem which is fixed/What is the use case

This brings changes of #398 and #399 to 1.5.3.

Fixes: KATA-2876

- What I did

git cherry-pick -sx 696aef2b492a1f3ab78b33c1a365b904dac80e51 and adpated version to 1.5.3
git cherry-pick -sx 02086163095a8e97df209f3bbbd3d5d8293842e2 and fixed a trivial context conflict

- How to verify it

  • Checkout the PR
  • Verify that CSV was converted from operators.openshift.io/infrastructure-features to features.operators.openshift.io
  • Verify that CSV has the replaces: sandboxed-containers-operator.v1.5.2 required by olm.skipRange: '>=1.1.0 <1.5.3'

gkurz and others added 2 commits April 10, 2024 15:03
This is now mandated by CVP if the CSV uses the `skipRange` field.

Fixes: KATA-2894
Fixes: KATA-2876

Signed-off-by: Greg Kurz <groug@kaod.org>
(cherry picked from commit 696aef2,
 fixed version for 1.5.3)
Signed-off-by: Greg Kurz <groug@kaod.org>
The `features.operators.openshift.io` infrastructure feature annotations,
introduced in OpenShift Container Platform 4.14, deprecate the
`operators.openshift.io/infrastructure-features` annotations used in
earlier versions of OpenShift Container Platform. See [1]

Drop the `operators.openshift.io/infrastructure-features` line.

[1] https://docs.openshift.com/container-platform/4.14/operators/operator_sdk/osdk-generating-csvs.html#osdk-csv-manual-annotations-deprecated_osdk-generating-csvs

Fixes: KATA-2895
Fixes: KATA-2876

Signed-off-by: Cameron Meadors <cmeadors@redhat.com>
Signed-off-by: Greg Kurz <groug@kaod.org>
(cherry picked from commit 0208616,
 fixed trivial context conflict)
Signed-off-by: Greg Kurz <groug@kaod.org>
@openshift-ci openshift-ci bot requested a review from jensfr April 10, 2024 13:05
@gkurz gkurz marked this pull request as draft April 10, 2024 13:06
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 10, 2024
@gkurz
Copy link
Member Author

gkurz commented Apr 10, 2024

Make it a draft until we know if https://issues.redhat.com/browse/KATA-2523 calls for an extra change.

@gkurz gkurz marked this pull request as ready for review April 10, 2024 15:18
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 10, 2024
Copy link
Contributor

@bpradipt bpradipt left a comment

Choose a reason for hiding this comment

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

/lgtm
Verified that disconnected is true for the original PR.
And replaces replaces 1.5.2 since this is for 1.5.3

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 10, 2024
Copy link
Contributor

@littlejawa littlejawa left a comment

Choose a reason for hiding this comment

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

lgtm

Compared with the changes from referenced PRs, and consistent with the content we expect in a 1.5.3 release.

Thanks @gkurz

@gkurz gkurz merged commit 398d12e into openshift:release-1.5 Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants