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

Switch k8s on GCE to use GCI by default #33353

Merged
merged 1 commit into from
Sep 24, 2016
Merged

Switch k8s on GCE to use GCI by default #33353

merged 1 commit into from
Sep 24, 2016

Conversation

vishh
Copy link
Contributor

@vishh vishh commented Sep 23, 2016 •
edited by k8s-oncall
Loading

This change is Reviewable

@vishh
Switch k8s on GCE to use GCI by default
ef49584 
Signed-off-by: Vishnu kannan <vishnuk@google.com>
@k8s-github-robot k8s-github-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. release-note-label-needed labels Sep 23, 2016
@vishh
Copy link
Contributor Author

vishh commented Sep 23, 2016

master branch (v1.5) was never meant to run container-vm by default. We are ready for wider testing of GCI and running it as the default OS. There are some known issues and they are being tracked and resolved.
Workaround is to use container-vm

@vishh vishh added release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed release-note-label-needed labels Sep 23, 2016
@vishh vishh assigned mtaufen and unassigned eparis Sep 23, 2016
@vishh vishh added this to the v1.5 milestone Sep 23, 2016
@k8s-github-robot
Copy link

@k8s-ci-robot
You must link to the test flake issue which caused you to request this manual re-test.
Re-test requests should be in the form of: k8s-bot test this issue: #<number>
Here is the list of open test flakes.

@vishh
Copy link
Contributor Author

vishh commented Sep 23, 2016

@k8s-bot gci test this

@mtaufen
Copy link
Contributor

mtaufen commented Sep 23, 2016

LGTM

@vishh vishh added the priority/backlog Higher priority than priority/awaiting-more-evidence. label Sep 23, 2016
@vishh
Copy link
Contributor Author

vishh commented Sep 23, 2016

P2 since this is critical for stabilizing GCI and this change was supposed to not even be necessary in the first place.

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

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 5583047 into kubernetes:master Sep 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mtaufen mtaufen

Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/backlog Higher priority than priority/awaiting-more-evidence. 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
v1.5
Development

Successfully merging this pull request may close these issues.
5 participants
@vishh @k8s-github-robot @mtaufen @googlebot @eparis