Skip to content
This repository has been archived by the owner on Aug 12, 2024. It is now read-only.

bump go 1.22 and k8s deps 0.30.0 #890

Merged

Conversation

everettraven
Copy link
Contributor

No description provided.

@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 May 14, 2024
go.mod Outdated Show resolved Hide resolved
@everettraven everettraven force-pushed the chore/bump-k8s-1.30 branch from 7d7e747 to d7880c2 Compare May 16, 2024 19:55
@everettraven everettraven changed the title wip: bump go 1.22 and k8s deps 0.30.0 bump go 1.22 and k8s deps 0.30.0 May 16, 2024
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 18, 2024
@everettraven everettraven force-pushed the chore/bump-k8s-1.30 branch from 18b79e1 to bade292 Compare May 20, 2024 19:39
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 20, 2024
Copy link

codecov bot commented May 20, 2024

Codecov Report

Attention: Patch coverage is 53.84615% with 12 lines in your changes are missing coverage. Please review.

Project coverage is 15.06%. Comparing base (e38bb7e) to head (bade292).

Files Patch % Lines
...l/controllers/bundledeployment/bundledeployment.go 0.00% 12 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #890       +/-   ##
===========================================
- Coverage   37.95%   15.06%   -22.90%     
===========================================
  Files           9       40       +31     
  Lines         843     2443     +1600     
===========================================
+ Hits          320      368       +48     
- Misses        481     2018     +1537     
- Partials       42       57       +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

github.com/operator-framework/helm-operator-plugins v0.2.1
github.com/operator-framework/operator-registry v1.40.0
github.com/operator-framework/api v0.24.0
github.com/operator-framework/helm-operator-plugins v0.2.2-0.20240520180534-f463c36fedf9
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Only reason I am pinning on a commit is because I wasn't sure if we wanted to cut a release of helm-operator-plugins since the Kubebuilder dependency was not updated to one that supports go 1.22 and k8s 1.30 for scaffolding.

Happy to discuss this further (and cut a release if desired), but if we don't intend to cut a release for helm-operator-plugins in the very near future I'd prefer to get this in as is to unblock the CRD Upgrade Safety preflight check work

Copy link
Member

Choose a reason for hiding this comment

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

Is helm-operator-plugins scaffolding still working right now with your changes that just merged? If so, I see no reason not to go ahead and tag it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe so. I can go ahead and tag it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As found in https://kubernetes.slack.com/archives/C0181L6JYQ2/p1716234751101299 the scaffolding is broken and should be fixed in helm-operator-plugins before being tagged. To unblock the in-flight CRD upgrade safety preflight check work we will capture fixing that in an issue to be addressed before tagging and will be keeping this pinned on a commit in the meantime

@everettraven everettraven marked this pull request as ready for review May 20, 2024 19:49
@everettraven everettraven requested a review from a team as a code owner May 20, 2024 19:49
@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 May 20, 2024
@everettraven everettraven requested a review from joelanford May 20, 2024 20:34
Copy link
Member

@rashmigottipati rashmigottipati left a comment

Choose a reason for hiding this comment

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

Changes look good to me! Will leave final approval to @joelanford

@everettraven everettraven added this pull request to the merge queue May 21, 2024
Merged via the queue into operator-framework:main with commit 5830540 May 21, 2024
8 of 10 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants