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

bugfix: check the duplicate mount point #1185

Merged
merged 1 commit into from
Apr 23, 2018

Conversation

rudyfly
Copy link
Collaborator

@rudyfly rudyfly commented Apr 23, 2018

Ⅰ. Describe what this PR did

Check the duplicate mount point for each volumes or binds.

Ⅱ. Does this pull request fix one issue?

NONE

Ⅲ. Describe how you did it

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

Signed-off-by: Rudy Zhang rudyflyzhang@gmail.com

@pouchrobot pouchrobot added kind/bug This is bug report for project size/L labels Apr 23, 2018
Check the duplicate mount point for each volumes or binds.

Signed-off-by: Rudy Zhang <rudyflyzhang@gmail.com>
@codecov-io
Copy link

Codecov Report

Merging #1185 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1185      +/-   ##
==========================================
- Coverage   16.03%   16.01%   -0.03%     
==========================================
  Files         176      176              
  Lines       10152    10166      +14     
==========================================
  Hits         1628     1628              
- Misses       8404     8418      +14     
  Partials      120      120
Impacted Files Coverage Δ
apis/opts/mountpoint.go 78.57% <0%> (-6.05%) ⬇️
daemon/mgr/container.go 0% <0%> (ø) ⬆️

@yyb196
Copy link
Collaborator

yyb196 commented Apr 23, 2018

LGTM, will merge soon

@pouchrobot pouchrobot added the LGTM one maintainer or community participant agrees to merge the pull reuqest. label Apr 23, 2018
@yyb196 yyb196 merged commit b1dd6bc into AliyunContainerService:master Apr 23, 2018
@rudyfly rudyfly deleted the volumesFrom branch October 29, 2018 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug This is bug report for project LGTM one maintainer or community participant agrees to merge the pull reuqest. size/XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants