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.

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

feature: add blkio related parameters in swagger #643

Merged
merged 1 commit into from
Jan 26, 2018
Merged

feature: add blkio related parameters in swagger #643

merged 1 commit into from
Jan 26, 2018

Conversation

Ace-Tang
Copy link
Contributor

Signed-off-by: Ace-Tang aceapril@126.com

1.Describe what this PR did

2.Does this pull request fix one issue?

3.Describe how you did it

4.Describe how to verify it

5.Special notes for reviews

@Ace-Tang Ace-Tang changed the title add blkio related parameters in swagger feature: add blkio related parameters in swagger Jan 25, 2018
@allencloud
Copy link
Collaborator

I think we need you help to take a look at the flaky test in CI result: https://travis-ci.org/alibaba/pouch/builds/333265280?utm_source=github_status&utm_medium=notification.
@Letty5411 @sunyuan3

@allencloud allencloud added this to the v0.2-milestone milestone Jan 25, 2018
Signed-off-by: Ace-Tang <aceapril@126.com>
@Ace-Tang
Copy link
Contributor Author

updated, @allencloud

@allencloud allencloud added this to Container in Pouch-v0.2 Jan 26, 2018
@codecov-io
Copy link

Codecov Report

Merging #643 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #643   +/-   ##
=======================================
  Coverage   14.21%   14.21%           
=======================================
  Files          65       65           
  Lines        3736     3736           
=======================================
  Hits          531      531           
  Misses       3159     3159           
  Partials       46       46

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a983b63...a4a3144. Read the comment docs.

@allencloud
Copy link
Collaborator

LGTM, thanks a lot for your work. @Ace-Tang

@pouchrobot pouchrobot added the LGTM one maintainer or community participant agrees to merge the pull reuqest. label Jan 26, 2018
@allencloud allencloud merged commit cde4bff into AliyunContainerService:master Jan 26, 2018
@Ace-Tang Ace-Tang deleted the blkio branch January 26, 2018 06:13
@allencloud allencloud moved this from Container to done in Pouch-v0.2 Jan 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature LGTM one maintainer or community participant agrees to merge the pull reuqest. size/XL
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants