Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

Allow toggling Metrics Server installation #1068

Merged
merged 1 commit into from
Dec 8, 2017

Conversation

mumoshu
Copy link
Contributor

@mumoshu mumoshu commented Dec 8, 2017

Resolves #1054

@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 Dec 8, 2017
@mumoshu mumoshu changed the title Allow toggling Metrics Server installation Resolves #1054 Allow toggling Metrics Server installation Dec 8, 2017
@mumoshu mumoshu force-pushed the toggle-metrics-server branch 2 times, most recently from e7a641b to 6ac795b Compare December 8, 2017 12:13
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Dec 8, 2017
@mumoshu mumoshu added this to the v0.9.9 milestone Dec 8, 2017
@codecov-io
Copy link

Codecov Report

Merging #1068 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1068   +/-   ##
=======================================
  Coverage   35.13%   35.13%           
=======================================
  Files          59       59           
  Lines        3393     3393           
=======================================
  Hits         1192     1192           
  Misses       2039     2039           
  Partials      162      162

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2fb8d4...d9fe738. Read the comment docs.

@mumoshu
Copy link
Contributor Author

mumoshu commented Dec 8, 2017

Manually verified this to work by updating my cluster toggling metrics-server several times.

@mumoshu mumoshu merged commit 2267305 into kubernetes-retired:master Dec 8, 2017
@mumoshu mumoshu deleted the toggle-metrics-server branch December 8, 2017 13:47
kylehodgetts pushed a commit to HotelsDotCom/kube-aws that referenced this pull request Mar 27, 2018
…s-server

Allow toggling Metrics Server installation
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants