Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

idoshamun
Copy link
Contributor

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you tested this change? If you were to make this change on the heapster repo here, it will get tested automatically by CI!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually it might be a good idea to revert this change.
I made it especially for my cluster.
And everything is tested of course.

@j3ffml
Copy link
Contributor

j3ffml commented May 1, 2015

Also need to make sure the updated image version is pushed to gcr.io

@vishh
Copy link
Contributor

vishh commented May 1, 2015

I did push the images to gcr.io. So that should not be an issue!

On Fri, May 1, 2015 at 10:50 AM, Jeff Lowdermilk [email protected]
wrote:

Also need to make sure the updated image version is pushed to gcr.io


Reply to this email directly or view it on GitHub
#7626 (comment)
.

@j3ffml j3ffml assigned vishh and unassigned satnam6502 May 1, 2015
@satnam6502 satnam6502 added lgtm "Looks good to me", indicates that a PR is ready to be merged. and removed lgtm "Looks good to me", indicates that a PR is ready to be merged. labels May 1, 2015
@vishh
Copy link
Contributor

vishh commented May 1, 2015

LGTM

vishh added a commit that referenced this pull request May 1, 2015
bump heapster to v0.11.0 and grafana to v0.7.0
@vishh vishh merged commit 3cc5312 into kubernetes:master May 1, 2015
@idoshamun idoshamun deleted the heapster-0.11.0 branch May 22, 2015 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants