-
Notifications
You must be signed in to change notification settings - Fork 295
Conversation
Codecov Report
@@ Coverage Diff @@
## master #420 +/- ##
=======================================
Coverage 38.53% 38.53%
=======================================
Files 30 30
Lines 2351 2351
=======================================
Hits 906 906
Misses 1324 1324
Partials 121 121
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you've missed this reference.
c4c6e2d
to
552667e
Compare
@danielfm thanks. fixed. |
Also, if I may suggest, instead of keeping the hard-coded version in the template, like this: metadata:
name: heapster-v1.2.0 ... why not use the tag from the config, like this: metadata:
name: heapster-{{.HeapsterImage.Tag}} I did not test this though, but it seems possible to get rid of this duplication here. |
Thanks for the help @AlmogBaku @danielfm! Btw, @danielfm's suggested improvement sounds great. Would you mind submitting an another github issue for that? |
* kubernetes-incubator/master: (29 commits) Emit errors when kube-aws sees unexpected keys in cluster.yaml Resolves kubernetes-retired#404 Tag controller nodes appropriately with `kubernetes.io/role`. Resolves kubernetes-retired#370 Make Container Linux AMI fetching a bit more reliable Stop locksmithd errors on etcd nodes Upgrade heapster to version 1.3.0 (kubernetes-retired#420) Auth token file support (kubernetes-retired#418) Update README.md Update README accordingly to the new git repo AWS China region support (kubernetes-retired#390) Conform as a Kubernetes Incubator Project Fixed typo in template upgrade aws-sdk to latest version Fix kubernetes-retired#388 Upgrade Kubernetes version to v1.5.4 Fix assumed public hostnames for EC2 instances in us-east-1 Fix assumed public hostnames for EC2 instances in us-east-1 typo fix: etcdDataVolumeEncrypted not creating encrypted volumes fixes kubernetes-retired#383 Allow disabling wait signals fixes kubernetes-retired#371 Update file paths in readme Fix an issue with glue security group documentation ...
Upgrade heapster to version 1.3.0