Skip to content

Add metrics type alias#3666

Merged
fuweid merged 1 commit intocontainerd:masterfrom
crosbymichael:metrics
Sep 20, 2019
Merged

Add metrics type alias#3666
fuweid merged 1 commit intocontainerd:masterfrom
crosbymichael:metrics

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

This will help to decouple the import in CRI from the cgroups package
directly by importing the type alias in containerd repo.

Signed-off-by: Michael Crosby [email protected]

This will help to decouple the import in CRI from the cgroups package
directly by importing the type alias in containerd repo.

Signed-off-by: Michael Crosby <[email protected]>
Copy link
Copy Markdown
Contributor

@jterry75 jterry75 left a comment

Choose a reason for hiding this comment

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

Oh great thanks for doing this side! LGTM

@crosbymichael
Copy link
Copy Markdown
Member Author

@jterry75 i'll have to update CRI with this change then I can bump the cgroups dep in each project. I'm not 100% that this will work but I think it should.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 19, 2019

Build succeeded.

@jterry75
Copy link
Copy Markdown
Contributor

@jterry75 i'll have to update CRI with this change then I can bump the cgroups dep in each project. I'm not 100% that this will work but I think it should.

Yea I'm not sure either. But the theory seems good :)

@crosbymichael
Copy link
Copy Markdown
Member Author

Travis passed, it's just not updated on github

Copy link
Copy Markdown
Member

@mxpv mxpv left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

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

LGTM

@fuweid fuweid merged commit ed16170 into containerd:master Sep 20, 2019
@crosbymichael crosbymichael deleted the metrics branch September 20, 2019 14:03
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.

4 participants