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

GCE: bump COS image version to cos-stable-63-10032-71-0 #57204

Merged
merged 1 commit into from
Dec 17, 2017
Merged

GCE: bump COS image version to cos-stable-63-10032-71-0 #57204

merged 1 commit into from
Dec 17, 2017

Conversation

yujuhong
Copy link
Contributor 

GCE: bump COS image version to cos-stable-63-10032-71-0

@k8s-ci-robot k8s-ci-robot added release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Dec 14, 2017
@yujuhong
Copy link
Contributor Author

/cc @yguo0905 @dashpole

@abgworrall
Copy link
Contributor

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 14, 2017
@dashpole
Copy link
Contributor

Is there a corresponding change in test-infra?

@dchen1107
Copy link
Member

/lgtm

@yujuhong
Copy link
Contributor Author

yujuhong commented Dec 15, 2017 •
edited
Loading

Once this PR lands, all GCE tests on the master branch will be testing overlay2 + live-restore.
This does not affect GKE, which we will determine how to roll out later.

/approve no-issue

Is there a corresponding change in test-infra?

Updated the node e2e configuration in kubernetes/test-infra#5965

@yujuhong yujuhong added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 15, 2017
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: abgworrall, dchen1107, yujuhong

No associated issue. Requirement bypassed by manually added approval.

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

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@yujuhong
Copy link
Contributor Author

Manually applied approved based on @dchen1107's LGTM

@BenTheElder BenTheElder mentioned this pull request Dec 15, 2017
Merged
@k8s-github-robot
Copy link

Automatic merge from submit-queue (batch tested with PRs 56386, 57204, 55692, 57107, 57177). If you want to cherry-pick this change to another branch, please follow the instructions here.

@k8s-github-robot k8s-github-robot merged commit d117e86 into kubernetes:master Dec 17, 2017
@yujuhong yujuhong deleted the bump-cos-version branch January 26, 2018 18:58
@bsdnet bsdnet mentioned this pull request May 26, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dashpole dashpole Awaiting requested review from dashpole

@yguo0905 yguo0905 Awaiting requested review from yguo0905

Assignees

@dchen1107 dchen1107

@abgworrall abgworrall

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. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@yujuhong @abgworrall @dashpole @dchen1107 @k8s-ci-robot @k8s-github-robot