Skip to content

Update go dependencies for 0.19 release#283

Merged
hasanawad94 merged 1 commit intoshipwright-io:mainfrom
hasanawad94:deps-v19
Mar 10, 2026
Merged

Update go dependencies for 0.19 release#283
hasanawad94 merged 1 commit intoshipwright-io:mainfrom
hasanawad94:deps-v19

Conversation

@hasanawad94
Copy link
Contributor

@hasanawad94 hasanawad94 commented Mar 8, 2026

Changes

Update go modules:

  • k8s.io/client-go → v0.34.4 (was v0.33.6)
  • k8s.io/api, k8s.io/apimachinery, k8s.io/apiextensions-apiserver → v0.34.4
  • openshift/api, openshift/client-go, openshift/apiserver-library-go → latest versions
  • controller-runtime → v0.22.5 (matches k8s 1.34)

Submitter Checklist

  • Includes tests if functionality changed/was added
  • Includes docs if changes are user-facing
  • Set a kind label on this PR
  • Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.

Release Notes

Replaced `gcr.io/kubebuilder/kube-rbac-proxy` image with `quay.io/brancz/kube-rbac-proxy`

@hasanawad94 hasanawad94 added this to the release-v0.19.0 milestone Mar 8, 2026
@hasanawad94 hasanawad94 added the dependencies Pull requests that update a dependency file label Mar 8, 2026
@openshift-ci openshift-ci bot requested review from adambkaplan and otaviof March 8, 2026 09:45
@hasanawad94 hasanawad94 changed the title bump build,k8,go and tekton deps Update go dependencies for 1.19 relase Mar 8, 2026
@hasanawad94 hasanawad94 changed the title Update go dependencies for 1.19 relase Update go dependencies for 1.19 release Mar 8, 2026
@hasanawad94 hasanawad94 changed the title Update go dependencies for 1.19 release Update go dependencies for 0.19 release Mar 8, 2026
@hasanawad94 hasanawad94 marked this pull request as draft March 8, 2026 09:50
@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 Mar 8, 2026
@hasanawad94 hasanawad94 force-pushed the deps-v19 branch 11 times, most recently from 9b65f96 to ea16b83 Compare March 9, 2026 09:44
@hasanawad94 hasanawad94 marked this pull request as ready for review March 9, 2026 12:25
@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 Mar 9, 2026
@openshift-ci openshift-ci bot requested a review from sayan-biswas March 9, 2026 12:25
- name: Create KinD cluster
uses: helm/kind-action@v1
with:
version: v0.31.0
Copy link
Member

Choose a reason for hiding this comment

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

That looks risky. Without it, I guess it takes latest, but what if latest stops being compatible with the node image we are using ?

Copy link
Member

Choose a reason for hiding this comment

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

I agree here. Version pinning also makes it easier to reproduce a particular CI run.

@SaschaSchwarze0 SaschaSchwarze0 added kind/dependency-change Categorizes issue or PR as related to changing dependencies and removed dependencies Pull requests that update a dependency file labels Mar 9, 2026
@adambkaplan adambkaplan moved this to Todo in Issues Mar 9, 2026
@adambkaplan adambkaplan moved this from Todo to In Progress in Issues Mar 9, 2026
Copy link
Member

@adambkaplan adambkaplan left a comment

Choose a reason for hiding this comment

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

/approve

Green-lighting this to go in. However @hasanawad94 please address the issue with the kind-action workflow change. Once that is addressed I think this PR is able to be merged.

- name: Create KinD cluster
uses: helm/kind-action@v1
with:
version: v0.31.0
Copy link
Member

Choose a reason for hiding this comment

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

I agree here. Version pinning also makes it easier to reproduce a particular CI run.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 9, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adambkaplan

The full list of commands accepted by this bot can be found here.

The pull request process is described 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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 9, 2026
@adambkaplan
Copy link
Member

@hasanawad94 please also include a release note in the PR description.

Update Go version to 1.25.6
golint fix: golangci-lint v2 requires the version: "2" field in the config — without it, it refuses to run.
gcr.io/kubebuilder/kube-rbac-proxy image was moved to a different registry based on: https://github.com/kubernetes-sigs/kubebuilder/blob/master/designs/discontinue_usage_of_kube_rbac_proxy.md#summary
Use `ObjectTracker` as a workaround with the new `controller-runtime` update in controllers/shipwrightbuild_controller_test.go

Signed-off-by: Hasan Awad <hasan.m.awad94@gmail.com>
@hasanawad94 hasanawad94 merged commit 507a756 into shipwright-io:main Mar 10, 2026
4 of 5 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/dependency-change Categorizes issue or PR as related to changing dependencies release-note

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants