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.

Sign up for GitHub

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

Jump to bottom

Update with Kubernetes v1.22 #273

Merged
merged 10 commits into from
Sep 23, 2021
Merged

Update with Kubernetes v1.22 #273

merged 10 commits into from
Sep 23, 2021

Conversation

cici37
Copy link
Contributor

@cici37 cici37 commented Sep 15, 2021 •
edited

  1. Update Kubernetes release tar with v1.22
  2. Update kubectl
  3. Update kube git version
  4. Update //cluster dir with kubernetes 1.22
  5. Update CCM manifest to address changes.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 15, 2021
@cici37 cici37 changed the title Update with Kubernetes v1.22 [WIP]Update with Kubernetes v1.22 Sep 15, 2021
@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 15, 2021
@cici37
Copy link
Contributor Author

cici37 commented Sep 21, 2021

After kubernetes/test-infra#23689 is merged, the pr should be ready to go.

@cici37 cici37 changed the title [WIP]Update with Kubernetes v1.22 Update with Kubernetes v1.22 Sep 21, 2021
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 21, 2021
@cici37
Copy link
Contributor Author

cici37 commented Sep 21, 2021

/test cloud-provider-gcp-e2e-full

@cici37
Copy link
Contributor Author

cici37 commented Sep 21, 2021

/assign @cheftako @jpbetz
Would you mind to take a look when have time? This will block other changes from merging^^. Thank you

jpbetz
jpbetz requested changes Sep 22, 2021
View reviewed changes
cluster/OWNERS Outdated Show resolved Hide resolved
cluster/README.md Outdated Show resolved Hide resolved
cluster/gce/gci/OWNERS Outdated Show resolved Hide resolved
cluster/images/conformance/Dockerfile Outdated Show resolved Hide resolved
cluster/images/etcd-version-monitor/Dockerfile Outdated Show resolved Hide resolved
cluster/images/etcd/Dockerfile Outdated Show resolved Hide resolved
cluster/images/OWNERS Outdated Show resolved Hide resolved
@jpbetz
Copy link
Contributor

jpbetz commented Sep 22, 2021

/lgtm

I believe all my concerns have been addressed.

@cheftako would you be willing to review for approval?

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 22, 2021
cheftako
cheftako reviewed Sep 22, 2021
View reviewed changes
WORKSPACE Outdated Show resolved Hide resolved
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 22, 2021
This was referenced Sep 23, 2021
Merged
Merged
@mikedanese
Copy link
Member

Reapplying lgtms:

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 23, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cici37, jpbetz, mikedanese

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 23, 2021
@k8s-ci-robot k8s-ci-robot merged commit 62e51ee into kubernetes:master Sep 23, 2021
@cici37 cici37 deleted the fix branch September 23, 2021 18:09
@jpbetz jpbetz mentioned this pull request Sep 29, 2021
Merged
@cici37 cici37 added the sig/cloud-provider Categorizes an issue or PR as relevant to SIG Cloud Provider. label Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cheftako cheftako cheftako left review comments

@jpbetz jpbetz jpbetz approved these changes

@yujuhong yujuhong Awaiting requested review from yujuhong

Assignees

@jpbetz jpbetz

@cheftako cheftako

@mikedanese mikedanese

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 "Looks good to me", indicates that a PR is ready to be merged. sig/cloud-provider Categorizes an issue or PR as relevant to SIG Cloud Provider. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@cici37 @jpbetz @mikedanese @k8s-ci-robot @cheftako