Skip to content
This repository has been archived by the owner on Jun 26, 2023. It is now read-only.

Update controller-runtime to v0.7.0 #1367

Merged
merged 1 commit into from
Jan 29, 2021

Conversation

adrianludwin
Copy link
Contributor

@adrianludwin adrianludwin commented Jan 29, 2021

Also updates to the latest (post-v0.1.0) version of cert-controller,
which has a minor API change and supports controller-runtime v0.7.0.

This commit doesn't include any changes except to react to API changes
in controller-runtime and cert-controller.

Tested: e2e tests on GKE 1.19

Fixes #1327
Fixes #1270

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jan 29, 2021
@adrianludwin adrianludwin added this to the hnc-v0.8 milestone Jan 29, 2021
@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jan 29, 2021
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 29, 2021
Also updates to the latest (post-v0.1.0) version of cert-controller,
which has a minor API change and supports controller-runtime v0.7.0.

This commit doesn't include any changes except to react to API changes
in controller-runtime and cert-controller.

Tested: e2e tests on GKE 1.19
@adrianludwin
Copy link
Contributor Author

/assign @GinnyJI
/assign @rjbez17
/cc @yiqigao217

@rjbez17
Copy link

rjbez17 commented Jan 29, 2021

/lgtm
/approve

/hold

@k8s-ci-robot k8s-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm Indicates that a PR is ready to be merged. labels Jan 29, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adrianludwin, rjbez17

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:
  • OWNERS [adrianludwin,rjbez17]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@GinnyJI
Copy link
Contributor

GinnyJI commented Jan 29, 2021

/hold cancel

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 29, 2021
@k8s-ci-robot k8s-ci-robot merged commit c22e3ec into kubernetes-retired:master Jan 29, 2021
@adrianludwin adrianludwin deleted the cr070 branch April 30, 2021 16:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HNC: use a later version of controller-tools HNC: use v1 for CRDs and webhooks
4 participants