Skip to content

Conversation

@lovesegfault
Copy link
Member

@lovesegfault lovesegfault commented Oct 27, 2025

Motivation

Implement abortMultipartUpload() for cleaning up incomplete multipart
uploads on error:

  • Constructs URL with ?uploadId=ID query parameter
  • Issues DELETE request to abort the multipart upload

Context

Part-Of: #14330
Depends-On: #14377


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@github-actions github-actions bot added the store Issues and pull requests concerning the Nix store label Oct 27, 2025
@lovesegfault lovesegfault force-pushed the nix-s3-abort-multipart branch 3 times, most recently from 416b02d to 7c4555e Compare October 27, 2025 18:48
@dpulls
Copy link

dpulls bot commented Oct 27, 2025

🎉 All dependencies have been resolved !

Implement `abortMultipartUpload()` for cleaning up incomplete multipart
uploads on error:

- Constructs URL with `?uploadId=ID` query parameter
- Issues `DELETE` request to abort the multipart upload
@lovesegfault lovesegfault force-pushed the nix-s3-abort-multipart branch from 7c4555e to 3915b3a Compare October 27, 2025 18:57
@Ericson2314 Ericson2314 enabled auto-merge October 27, 2025 19:01
@Ericson2314 Ericson2314 added this pull request to the merge queue Oct 27, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Oct 27, 2025
github-merge-queue bot pushed a commit that referenced this pull request Oct 27, 2025
feat(libstore/s3-binary-cache-store): implement `abortMultipartUpload()`
@github-merge-queue github-merge-queue bot closed this pull request by merging all changes into NixOS:master in dd0d006 Oct 27, 2025
Ericson2314 added a commit to obsidiansystems/nix that referenced this pull request Oct 28, 2025
feat(libstore/s3-binary-cache-store): implement `abortMultipartUpload()`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

store Issues and pull requests concerning the Nix store

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants