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

update olm bindata #5684

Merged

Conversation

everettraven
Copy link
Contributor

Description of the change:
operator-sdk was behind on a few OLM versions. updated the supported versions and bindata.

Motivation for the change:
resolves #5680

Checklist

If the pull request includes user-facing changes, extra documentation is required:

operator-sdk was behind on a few OLM versions. updated the supported versions and bindata.

resolves operator-framework#5680

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@everettraven
Copy link
Contributor Author

Does this need a changelog update?

@everettraven
Copy link
Contributor Author

Also, this updates to use the 3 latest minor version releases (0.18.3, 0.19.1, 0.20.0). Do we need to instead update it to use the 3 latest releases (0.19.0, 0.19.1, 0.20.0)?

@everettraven
Copy link
Contributor Author

/cc @varshaprasad96

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Copy link
Member

@varshaprasad96 varshaprasad96 left a comment

Choose a reason for hiding this comment

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

@everettraven Thanks for the PR! Yes, let's modify it to support the latest 3 versions (0.19.0, 0.19.1, 0.20.0). Also, you may have to update the available olm versions here:

var availableVersions = map[string]struct{}{
for the e2e integration test suite to pass.

@varshaprasad96
Copy link
Member

Yes, let's add changelog too.

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Copy link
Member

@varshaprasad96 varshaprasad96 left a comment

Choose a reason for hiding this comment

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

/lgtm

@@ -92,7 +92,7 @@ By project type (replace `${IMAGE_VERSION}` with base image version in your proj

Operator SDK officially supports the latest 3 versions of OLM present at the time of a given Operator SDK release. These versions of OLM manifests are packaged with the SDK binary in the form of `bindata` to support low-latency installations of OLM with [`operator-sdk olm install`][olm-install-cmd]. Any other version installed with this command may work but is not tested nor officially supported.

Currently, the officially supported OLM Versions are: 0.16.1, 0.17.0 and 0.18.2.
Currently, the officially supported OLM Versions are: 0.19.0, 0.19.1, and 0.20.0
Copy link
Member

Choose a reason for hiding this comment

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

Just nit:

Suggested change
Currently, the officially supported OLM Versions are: 0.19.0, 0.19.1, and 0.20.0
Currently, the officially supported OLM Versions are: 0.19.0, 0.19.1, and 0.20.0.

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 27, 2022
@theishshah theishshah added this to the v1.20.0 milestone Apr 27, 2022
Copy link
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

I don't really see the value of someone installing an earlier version of OLM that already has a point release fix out. i.e. why would I want 0.19.0 instead of 0.19.1.

Makefile Outdated Show resolved Hide resolved
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Apr 27, 2022
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Copy link
Member

@laxmikantbpandhare laxmikantbpandhare left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 28, 2022
@asmacdo
Copy link
Member

asmacdo commented Apr 28, 2022

/lgtm

@everettraven everettraven merged commit 61132e1 into operator-framework:master Apr 28, 2022
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.

SDK does not officially support the latest OLM version
6 participants