Skip to content

Release v1.24.0#6071

Merged
jberkhahn merged 1 commit into
operator-framework:masterfrom
jberkhahn:release-v1.24.0
Oct 10, 2022
Merged

Release v1.24.0#6071
jberkhahn merged 1 commit into
operator-framework:masterfrom
jberkhahn:release-v1.24.0

Conversation

@jberkhahn

Copy link
Copy Markdown
Contributor

Hope this works

@jberkhahn jberkhahn temporarily deployed to deploy October 7, 2022 23:53 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 7, 2022 23:53 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 7, 2022 23:53 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 7, 2022 23:53 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 7, 2022 23:53 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 7, 2022 23:53 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 7, 2022 23:53 Inactive

@laxmikantbpandhare laxmikantbpandhare left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We updated OLM versions in docs in another PR but looks like we missed updating at a few places.

Please check Bryce's PR for the 1.22 release - https://github.com/operator-framework/operator-sdk/pull/5876/files

Missing places:

internal/bindata/olm/versions.go
internal/testutils/olm.go

@laxmikantbpandhare laxmikantbpandhare left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also, run make test-sanity locally, and if you found any changes in the files then push it. It will solve sanity failures.

@jmrodri

jmrodri commented Oct 10, 2022

Copy link
Copy Markdown
Member

Also, run make test-sanity locally, and if you found any changes in the files then push it. It will solve sanity failures.

I have 2 patches to fix sanity. You have to run it with go 1.18 for this to work. If you run it with 1.19 you will get what's in this PR.

@jmrodri

jmrodri commented Oct 10, 2022

Copy link
Copy Markdown
Member

Okay it was only one patch. That is run with go 1.18.

0001-Fix-test-sanity-formatting.txt

@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:52 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:52 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:52 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:52 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:52 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:52 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:52 Inactive
Signed-off-by: Jonathan Berkhahn <jaberkha@us.ibm.com>
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:59 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:59 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 16:59 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 17:01 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 17:01 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 17:01 Inactive
@jberkhahn jberkhahn temporarily deployed to deploy October 10, 2022 17:02 Inactive
@laxmikantbpandhare laxmikantbpandhare self-requested a review October 10, 2022 18:03

@laxmikantbpandhare laxmikantbpandhare left a comment

Copy link
Copy Markdown
Member

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 Oct 10, 2022
@jberkhahn jberkhahn merged commit de6a14d into operator-framework:master Oct 10, 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.

3 participants