Skip to content
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

Update cAdvisor dependency. #7897

Merged
merged 1 commit into from May 7, 2015
Merged

Update cAdvisor dependency. #7897

merged 1 commit into from May 7, 2015

Conversation

vmarmol
Copy link
Contributor

@vmarmol vmarmol commented May 7, 2015

Brings with it:

  • Lower logging
  • Better event storage and management
  • Support for container labels

/cc @vishh @rjnagal @dchen1107

@vmarmol
Update cAdvisor dependency.
2af756f 
Brings with it:
- Lower logging
- Better event storage and management
- Support for container labels
@vmarmol
Copy link
Contributor Author

vmarmol commented May 7, 2015

Running e2e, all is clean so far.

@vmarmol
Copy link
Contributor Author

vmarmol commented May 7, 2015

All e2e tests pass.

@rjnagal
Copy link
Contributor

rjnagal commented May 7, 2015

LGTM

@rjnagal rjnagal self-assigned this May 7, 2015
@dchen1107
Copy link
Member

LGTM. Thanks for fixing logging. :-)

dchen1107 added a commit that referenced this pull request May 7, 2015
@dchen1107
Merge pull request #7897 from vmarmol/cadvisor
5f6d54b 
Update cAdvisor dependency.
@dchen1107 dchen1107 merged commit 5f6d54b into kubernetes:master May 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@rjnagal rjnagal

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@vmarmol @rjnagal @dchen1107 @googlebot