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.

Sign up for GitHub

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

Jump to bottom

S3 Mountpoint to support renaming existing files on S3 bucket #1174

Open
prabaksa opened this issue Nov 19, 2024 · 5 comments
Open

S3 Mountpoint to support renaming existing files on S3 bucket #1174

prabaksa opened this issue Nov 19, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@prabaksa
Copy link

/feature

Is your feature request related to a problem? Please describe.
S3 Mountpoint doesn't support renaming files that are already existing on General purpose S3 bucket.

Describe the solution you'd like in detail
S3 Mountpoint should support renaming files that are already existing on General purpose S3 buckets.

Describe alternatives you've considered
The Alternate solution to copy the file with a different name and delete the old file.

Additional context
Add any other context or screenshots about the feature request here.
@unexge unexge added the enhancement New feature or request label Nov 22, 2024
@muddyfish
Copy link
Contributor

I'm moving this to the Mountpoint repository as this a Mountpoint feature request.

Move and rename operations on the filesystem cannot be implemented efficiently against S3 general purpose buckets. Mountpoint's approach is not to support this behaviour, but I'll leave this open for now to gauge interest in case general purpose S3 buckets were to start supporting this feature.

@muddyfish muddyfish transferred this issue from awslabs/mountpoint-s3-csi-driver Nov 28, 2024
@mdnadm
Copy link

mdnadm commented Dec 1, 2024

+1

1 similar comment
@dbouclier
Copy link

+1

@dbouclier dbouclier mentioned this issue Dec 2, 2024
Open
@fatihmete
Copy link

+1

@unexge
Copy link
Contributor

unexge commented Dec 4, 2024

Hey folks, please consider upvoting (👍) the original issue to show your interest rather than commenting “+1” as that's how we gauge the interest.

github-merge-queue bot pushed a commit that referenced this issue Dec 10, 2024
@passaro
Update CRT submodules to latest releases (#1195)
688ec17 
Update the CRT libraries to the latest releases. In particular, include:
* Support full object checksum and crc64nvme.
([awslabs/aws-c-s3#468](awslabs/aws-c-s3#468))

<details>
  <summary>Full CRT changelog:</summary>
  
```
Submodule mountpoint-s3-crt-sys/crt/aws-c-common be8ed873..fadfef49:
  > Support relative paths when prebuilding dependencies with CMake (#1174)
  > Switch CI to use roles (#1173)
Submodule mountpoint-s3-crt-sys/crt/aws-c-s3 45894ed3..337155f6:
  > Support full object checksum (#468)
  > [meta request]: assign shutdown_callback inside critical region (#470)
  > Switch CI to use roles (#463)
```
</details>

### Does this change impact existing behavior?

No.

### Does this change need a changelog entry?

No.

---

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license and I agree to the terms of
the [Developer Certificate of Origin
(DCO)](https://developercertificate.org/).

Signed-off-by: Alessandro Passaro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@muddyfish @fatihmete @unexge @mdnadm @prabaksa @dbouclier