Skip to content

feat: GCP copy, move, rename bucket#3810

Merged
yaron2 merged 6 commits into
dapr:mainfrom
nelson-parente:feat/gcp-cpmvrn
Jun 5, 2025
Merged

feat: GCP copy, move, rename bucket#3810
yaron2 merged 6 commits into
dapr:mainfrom
nelson-parente:feat/gcp-cpmvrn

Conversation

@nelson-parente

@nelson-parente nelson-parente commented May 4, 2025

Copy link
Copy Markdown
Contributor

Description

This PR adds GCP bucket binding copy, move and rename methods.

GCP provides copy, rename and move functionality which is currently not supported by dapr.
It would be good if we could provide these functionality as they are common across all major storage providers.

https://cloud.google.com/storage/docs/copying-renaming-moving-objects#permissions-rest

dapr/docs#4644

Issue reference

#3157

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

@nelson-parente nelson-parente changed the title feat: copy, move, rename bucket feat: GCP copy, move, rename bucket May 4, 2025
@nelson-parente
nelson-parente force-pushed the feat/gcp-cpmvrn branch 2 times, most recently from 1e16b64 to 5034364 Compare May 4, 2025 16:10
@nelson-parente
nelson-parente marked this pull request as ready for review May 5, 2025 15:18
@nelson-parente
nelson-parente requested review from a team as code owners May 5, 2025 15:18
yaron2
yaron2 previously approved these changes May 16, 2025
@yaron2
yaron2 added this pull request to the merge queue May 16, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 16, 2025
@nelson-parente
nelson-parente added this pull request to the merge queue May 19, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 19, 2025
@nelson-parente
nelson-parente added this pull request to the merge queue May 19, 2025
github-merge-queue Bot pushed a commit that referenced this pull request May 19, 2025
Signed-off-by: nelson.parente <[email protected]>
Co-authored-by: Yaron Schneider <[email protected]>
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 19, 2025
@nelson-parente

Copy link
Copy Markdown
Contributor Author

Merge queue failing 😞

@nelson-parente
nelson-parente added this pull request to the merge queue May 19, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 19, 2025
@nelson-parente
nelson-parente added this pull request to the merge queue May 22, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch May 22, 2025
@cicoyle

cicoyle commented May 22, 2025

Copy link
Copy Markdown
Contributor

@nelson-parente - mind fixing the conflicts?

Signed-off-by: nelson.parente <[email protected]>
Signed-off-by: nelson.parente <[email protected]>
@nelson-parente

Copy link
Copy Markdown
Contributor Author

@nelson-parente - mind fixing the conflicts?

done!

@nelson-parente
nelson-parente requested a review from yaron2 May 22, 2025 20:08
@nelson-parente

Copy link
Copy Markdown
Contributor Author

@yaron2 can we merge this?

@yaron2
yaron2 added this pull request to the merge queue Jun 3, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 3, 2025
@yaron2
yaron2 merged commit 4a50840 into dapr:main Jun 5, 2025
passuied pushed a commit to passuied/components-contrib that referenced this pull request Feb 14, 2026
Signed-off-by: nelson.parente <[email protected]>
Co-authored-by: Yaron Schneider <[email protected]>
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.

3 participants