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

Fix dashboard version in labels and remove version from RC name #625

Merged
merged 2 commits into from May 8, 2017
Merged

Fix dashboard version in labels and remove version from RC name #625

merged 2 commits into from May 8, 2017

Conversation

Vrtak-CZ
Copy link
Contributor

@Vrtak-CZ Vrtak-CZ commented May 4, 2017

No description provided.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 4, 2017
@Vrtak-CZ
Copy link
Contributor Author

Vrtak-CZ commented May 4, 2017

Btw @mumoshu is there any reason for version in RC name for dashboard?

@codecov-io
Copy link

codecov-io commented May 4, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #625   +/-   ##
=======================================
  Coverage   38.26%   38.26%           
=======================================
  Files          51       51           
  Lines        3316     3316           
=======================================
  Hits         1269     1269           
  Misses       1845     1845           
  Partials      202      202

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 9336b3a...d9dd400. Read the comment docs.

@mumoshu
Copy link
Contributor

mumoshu commented May 8, 2017

@Vrtak-CZ Nothing specific on my side - Even the corresponding kube-dashboard manifest in the upstream repo doesn't contain version numbers in its names.
I guess we can safely remove the version numbers if you'd (also) like to do so.

@Vrtak-CZ
Copy link
Contributor Author

Vrtak-CZ commented May 8, 2017

@mumoshu rebased and removed :-)

@mumoshu
Copy link
Contributor

mumoshu commented May 8, 2017

@Vrtak-CZ LGTM. Thanks for your support 👍

@mumoshu mumoshu merged commit 7bfd50c into kubernetes-retired:master May 8, 2017
@mumoshu mumoshu changed the title Fix dashboard version in labels and RC name Fix dashboard version in labels and remove version from RC name May 8, 2017
@mumoshu mumoshu added this to the v0.9.7-rc.1 milestone May 8, 2017
camilb added a commit to camilb/kube-aws that referenced this pull request May 18, 2017
* kubernetes-incubator/master:
  RBAC setup improvements (kubernetes-retired#618)
  kube-dns improvements (kubernetes-retired#615)
  Fix dashboard version in labels and remove version from RC name (kubernetes-retired#625)
  Improve taints validation (kubernetes-retired#616)
  Fix IamFleetRole syntax (kubernetes-retired#619)
  Update default dashboard to 1.6.0
  Replace custom 'minus' templating function with sprig-version and remove unused 'sha1' function
  Add sprig templating functions
kylehodgetts pushed a commit to HotelsDotCom/kube-aws that referenced this pull request Mar 27, 2018
…rnetes-retired#625)

* Fix dashboard version in labels

* Remove version from dashboard RC name
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants