Skip to content

cAdvisor 0.6.0 #310

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
Merged
merged 1 commit into from
Nov 17, 2014
Merged

cAdvisor 0.6.0 #310

merged 1 commit into from
Nov 17, 2014

Conversation

@vmarmol
Copy link
Contributor

@vmarmol vmarmol commented Nov 17, 2014

No description provided.

@vmarmol
Copy link
Contributor Author

vmarmol commented Nov 17, 2014

Ping @vishh @rjnagal

@vmarmol
Copy link
Contributor Author

vmarmol commented Nov 17, 2014

Wait for the merge :) I have a couple of fixes.

vishh
vishh reviewed Nov 17, 2014
View reviewed changes
CHANGELOG.md Outdated
Copy link
Contributor

@vishh vishh Nov 17, 2014

Choose a reason for hiding this comment

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

Mention ContainerStats API update.

Copy link
Contributor Author

@vmarmol vmarmol Nov 17, 2014

Choose a reason for hiding this comment

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

What update? The flattening? Since that doesn't affect the wire format, should we mention it?

Copy link
Contributor

@vishh vishh Nov 17, 2014

Choose a reason for hiding this comment

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

I think we should mention it.

On Mon, Nov 17, 2014 at 9:54 AM, Victor Marmol notifications@github.com
wrote:

In CHANGELOG.md:

@@ -1,5 +1,12 @@

Changelog

+## 0.6.0 (2014-11-17)
+- Adding /docker UI endpoint for Docker containers.
+- Fixes around handling Docker containers.
+- Performance enhancements.
+- Embed all external dependencies.
+- Misc bugfixes and cleanups.

What update? The flattening? Since that doesn't affect the wire format,
should we mention it?

—
Reply to this email directly or view it on GitHub
https://github.com/google/cadvisor/pull/310/files#r20451543.

Copy link
Contributor Author

@vmarmol vmarmol Nov 17, 2014

Choose a reason for hiding this comment

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

Done.

@vmarmol
Copy link
Contributor Author

vmarmol commented Nov 17, 2014

This is now ready for merging :)

@vishh
Copy link
Contributor

vishh commented Nov 17, 2014

LGTM

vmarmol added a commit that referenced this pull request Nov 17, 2014
@vmarmol
Merge pull request #310 from vmarmol/release
1e98602 
cAdvisor 0.6.0
@vmarmol vmarmol merged commit 1e98602 into google:master Nov 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vmarmol @vishh