-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
ignore typed errors correctly in list cache layer #10879
Conversation
bonus write bucket metadata cache with enough quorum Possible fix for minio#10868
d9e8681
to
4362f55
Compare
Mint Automation
10879-4362f55/mint-gateway-azure.sh.log:
Deleting image on docker hub |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually most of this will not work with wrapped errors.
Also, where is the bonus fix?
Wrapped errors won't happen since we look at replies directly from object layer. That is why using toObjectErr.
Bonus fix is removing setting RRS @klauspost |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, let's try
Description
ignore typed errors correctly in the list cache layer
Motivation and Context
bonus write bucket metadata cache with enough quorum
Possible fix for #10868
How to test this PR?
Hard to reproduce errors from #10868, most probably
if the quorum is lost on the objects it's possible to see
these errors.
Types of changes
Checklist:
commit-id
orPR #
here)