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

mirror/diff: Ignore errors during objectDifference. #2200

Merged
merged 1 commit into from
Jul 6, 2017

Conversation

harshavardhana
Copy link
Member

Files which are broken in symlink, not accessible
and unsupported should be ignored. We should proceed
to next object.

Fixes #2199

Files which are broken in symlink, not accessible
and unsupported should be ignored. We should proceed
to next object.

Fixes minio#2199
@codecov-io
Copy link

codecov-io commented Jul 2, 2017

Codecov Report

Merging #2200 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #2200   +/-   ##
======================================
  Coverage    9.12%   9.12%           
======================================
  Files          93      93           
  Lines        6906    6906           
======================================
  Hits          630     630           
  Misses       6146    6146           
  Partials      130     130
Impacted Files Coverage Δ
cmd/diff-main.go 0% <0%> (ø) ⬆️
cmd/mirror-url.go 0% <0%> (ø) ⬆️

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 05f588f...eea75fc. Read the comment docs.

@harshavardhana
Copy link
Member Author

why is this blocked @deekoder ?

@deekoder deekoder requested review from vadmeste and balamurugana and removed request for vadmeste July 3, 2017 05:08
@harshavardhana
Copy link
Member Author

Any reviews?

@deekoder deekoder requested review from krisis and removed request for vadmeste July 3, 2017 16:49
Copy link
Member

@krisis krisis 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
Member

@vadmeste vadmeste left a comment

Choose a reason for hiding this comment

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

LGTM

@deekoder deekoder merged commit 8391a69 into minio:master Jul 6, 2017
@harshavardhana harshavardhana deleted the ignore branch July 6, 2017 16:32
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.

5 participants