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

Bump dependencies to align with Build v0.12.0 #168

Merged

Conversation

adambkaplan
Copy link
Member

@adambkaplan adambkaplan commented Nov 1, 2023

Changes

Update the dependencies so we can bring in shipwright-io/build v0.12.0 as a new dependency. This is needed to support #155

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

Update k8s libraries to v0.26.4, controller-runtime to v0.14.6, and tektoncd/operator to v0.67.0. ACTION REQUIRED: minimum kubernetes version is now v1.24.0

@openshift-ci openshift-ci bot added release-note do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Nov 1, 2023
@adambkaplan adambkaplan added the kind/dependency-change Categorizes issue or PR as related to changing dependencies label Nov 1, 2023
@adambkaplan adambkaplan force-pushed the bump-controller-runtime-v0.14.6 branch from cfd6de0 to 77553e9 Compare November 1, 2023 22:27
@adambkaplan
Copy link
Member Author

/hold

This should wait for #164 to be merged first, as it builds on top of its commits.

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 2, 2023
@adambkaplan adambkaplan changed the title WIP - Bump dependencies to align with Build v0.12.0 Bump dependencies to align with Build v0.12.0 Nov 2, 2023
@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 Nov 2, 2023
@adambkaplan
Copy link
Member Author

/approve

Self-approving

Copy link
Contributor

openshift-ci bot commented Nov 2, 2023

[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

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 Nov 2, 2023
Copy link
Member

@SaschaSchwarze0 SaschaSchwarze0 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 Nov 2, 2023
In preparation of bringing in shipwright-io/build as a dependency, the
following dependencies needed to be updated concurrently due to
transitive dependencies on k8s.io libraries:

- controller-runtime: update to v0.14.6 to align with shipwright-io/build
- tektoncd/operator: update to v0.67.0 to align with controller-runtime
- k8s.io/* to v0.26.4 (a common patch version)
- onsi/ginkgo to v2 (from controller-runtime)
- Utilize ginkgo/v2 SpecContext to handle graceful termination. A
  separate context was created to isolate EnvTest from any
  "client" contexts used during tests and avoid deadlocks.
- Use v1.26 for EnvTest-driven tests.
- Re-generate manifests and bundle
Update KinD to latest released version (v0.20.0) and use the supported
v1.24 k8s node version from the corresponding release note. The current
Tekton operator on OperatorHub does not support k8s 1.25 or higher.
This also sets the minKubeVersion to 1.24.0 on the operator CSV so the
version we test with and the min supported version are aligned.

See tektoncd/operator#1493.
@adambkaplan adambkaplan force-pushed the bump-controller-runtime-v0.14.6 branch from 77553e9 to eb706b7 Compare November 2, 2023 20:28
@openshift-ci openshift-ci bot added release-note-action-required and removed lgtm Indicates that a PR is ready to be merged. release-note labels Nov 2, 2023
@adambkaplan
Copy link
Member Author

Rebased. Added ACTION REQUIRED release note because we are bumping the minimum k8s version (granted, to a version that is still EOL upstream).

@adambkaplan
Copy link
Member Author

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 2, 2023
Copy link
Member

@SaschaSchwarze0 SaschaSchwarze0 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 Nov 2, 2023
@SaschaSchwarze0 SaschaSchwarze0 added this to the release-v0.12.0 milestone Nov 2, 2023
@openshift-ci openshift-ci bot merged commit 260b7c1 into shipwright-io:main Nov 2, 2023
4 checks passed
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 lgtm Indicates that a PR is ready to be merged. release-note-action-required
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants