chore: bump CAPI to v1.5.0 #985
Conversation
k8s-ci-robot
added
release-note
|
@richardcase: The label(s) In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
k8s-ci-robot
added
the
cncf-cla: yes
k8s-ci-robot
added
approved
|
/kind feature |
k8s-ci-robot
added
the
kind/feature
richardcase
force-pushed
the
bump_capi_150rc
branch
4 times, most recently
from
94d9c05
to
dbd7117
Compare
k8s-ci-robot
added
size/XL
richardcase
force-pushed
the
bump_capi_150rc
branch
from
dbd7117
to
da4b97a
Compare
|
/override pull-cluster-api-provider-gcp-apidiff |
|
@richardcase: Overrode contexts on behalf of richardcase: pull-cluster-api-provider-gcp-apidiff In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
richardcase
force-pushed
the
bump_capi_150rc
branch
from
da4b97a
to
ff20c46
Compare
|
/override pull-cluster-api-provider-gcp-apidiff |
|
@richardcase: Overrode contexts on behalf of richardcase: pull-cluster-api-provider-gcp-apidiff In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
richardcase
changed the title
k8s-ci-robot
removed
the
do-not-merge/work-in-progress
| with: | ||
| go-version: 1.19 | ||
| check-latest: true |
Dockerfile
Outdated
| @@ -13,7 +13,7 @@ | |||
| # limitations under the License. | |||
|
|
|||
| # Build the manager binary | |||
| FROM golang:1.19.10@sha256:83f9f840072d05ad4d90ce4ac7cb2427632d6b89d5ffc558f18f9577ec8188c0 as builder | |||
| FROM public.ecr.aws/docker/library/golang:1.20.6 as builder | |||
There was a problem hiding this comment.
what is the advantage of using that? also can we add the digest as well?
There was a problem hiding this comment.
I got confused....changing to pull from ecr instead of gcr was for something else, doh!
I have changed it back to docker hub and also added the hash.
k8s-ci-robot
added
the
needs-rebase
This change bumps CAPI to v1.5.0 and also controller runtime. Signed-off-by: Richard Case <[email protected]>
richardcase
force-pushed
the
bump_capi_150rc
branch
from
ff20c46
to
f524e1a
Compare
k8s-ci-robot
removed
the
needs-rebase
|
/retitle chore: bump CAPI to v1.5.0 |
k8s-ci-robot
changed the title
|
/override pull-cluster-api-provider-gcp-apidiff |
|
@richardcase: Overrode contexts on behalf of richardcase: pull-cluster-api-provider-gcp-apidiff In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
k8s-ci-robot
added
the
lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cpanato, richardcase 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 |
What type of PR is this?
/kind other
What this PR does / why we need it:
This bumps main to use v1.5.0-rc1 of CAPI. It also bumps to use Go v1.20 and controller-runtime v0.15.0.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs:
Release note: