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

Fix bug with new minio behavior #4725

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

davisp
Copy link
Contributor

@davisp davisp commented Feb 13, 2024

New behavior doesn't delete masked objects. So check after the first deletion if we need to remove anything that was uncovered.


TYPE: BUG
DESC: Fix bug with new minio behavior

Copy link

This pull request has been linked to Shortcut Story #40963: Fix S3 errors on macOS S3 CI runners.

@KiterLuc KiterLuc merged commit e11fe82 into dev Feb 14, 2024
64 checks passed
@KiterLuc KiterLuc deleted the pd/sc-40963/account-for-new-minio-behavior branch February 14, 2024 06:25
abigalekim pushed a commit that referenced this pull request Feb 16, 2024
New behavior doesn't delete masked objects. So check after the first
deletion if we need to remove anything that was uncovered.

---
TYPE: BUG
DESC: Fix bug with new minio behavior
KiterLuc pushed a commit that referenced this pull request Feb 21, 2024
New behavior doesn't delete masked objects. So check after the first
deletion if we need to remove anything that was uncovered.

---
TYPE: BUG
DESC: Fix bug with new minio behavior

(cherry picked from commit e11fe82)
KiterLuc pushed a commit that referenced this pull request Feb 21, 2024
New behavior doesn't delete masked objects. So check after the first
deletion if we need to remove anything that was uncovered.

---
TYPE: BUG
DESC: Fix bug with new minio behavior

(cherry picked from commit e11fe82)
KiterLuc pushed a commit that referenced this pull request Feb 21, 2024
New behavior doesn't delete masked objects. So check after the first
deletion if we need to remove anything that was uncovered.

---
TYPE: BUG
DESC: Fix bug with new minio behavior

(cherry picked from commit e11fe82)
@teo-tsirpanis
Copy link
Member

For the record the minio change happened in minio/minio#19034.

KiterLuc pushed a commit that referenced this pull request Feb 21, 2024
New behavior doesn't delete masked objects. So check after the first
deletion if we need to remove anything that was uncovered.

---
TYPE: BUG
DESC: Fix bug with new minio behavior

(cherry picked from commit e11fe82)
KiterLuc added a commit that referenced this pull request Feb 22, 2024
Backport
e11fe82
from #4725.

---
TYPE: BUG
DESC: Fix bug with new minio behavior.

Co-authored-by: Paul J. Davis <[email protected]>
KiterLuc added a commit that referenced this pull request Feb 22, 2024
Backport
e11fe82
from #4725.

---
TYPE: BUG
DESC: Fix bug with new minio behavior.

Co-authored-by: Paul J. Davis <[email protected]>
Shelnutt2 pushed a commit that referenced this pull request Apr 21, 2024
New behavior doesn't delete masked objects. So check after the first
deletion if we need to remove anything that was uncovered.

---
TYPE: BUG
DESC: Fix bug with new minio behavior

(cherry picked from commit e11fe82)
Shelnutt2 added a commit that referenced this pull request Apr 21, 2024
The new MinIO behavior was accounted for in dev/2.20 and newer.
Instead of backporting the new S3 behavior we are pinning MinIO.

The mentioned update took place in
#4725
Shelnutt2 added a commit that referenced this pull request Apr 21, 2024
The new MinIO behavior was accounted for in dev/2.20 and newer.
Instead of backporting the new S3 behavior we are pinning MinIO.

The mentioned update took place in
#4725
Shelnutt2 added a commit that referenced this pull request Apr 21, 2024
…4899)

The [new MinIO behavior](minio/minio#19034) was
accounted for in dev/2.20 and newer. Instead of backporting the new S3
behavior we are pinning MinIO.

The mentioned update took place in
#4725


---
TYPE: NO_HISTORY
DESC: Change CI to use a pinned version of MinIO before breaking
behavior on deletes was introduced.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants