Skip to content

Improve atomic delete#3293

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
crosbymichael:atomic-delete
May 21, 2019
Merged

Improve atomic delete#3293
dmcgowan merged 1 commit intocontainerd:masterfrom
crosbymichael:atomic-delete

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

skip hidden directories in load task, and return soon if path not exist
in atomicDelete

carry of #3233

Closes #3233

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

skip hidden directories in load task, and return soon if path not exist
in atomicDelete

carry of containerd#3233

Closes containerd#3233

Signed-off-by: Ace-Tang <[email protected]>
Signed-off-by: Michael Crosby <[email protected]>
@crosbymichael
Copy link
Copy Markdown
Member Author

LGTM

carry updating the error handling from code review

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented May 20, 2019

Build succeeded.

@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #3293   +/-   ##
======================================
  Coverage    44.4%   44.4%           
======================================
  Files         113     113           
  Lines       12231   12231           
======================================
  Hits         5431    5431           
  Misses       5966    5966           
  Partials      834     834
Flag Coverage Δ
#linux 48.33% <ø> (ø) ⬆️
#windows 39.66% <ø> (ø) ⬆️

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 cafda1c...5b7a327. Read the comment docs.

@dmcgowan
Copy link
Copy Markdown
Member

LGTM

@dmcgowan dmcgowan merged commit 30082ab into containerd:master May 21, 2019
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