Skip to content

Preserve ACL when copying or renaming blob.#2409

Merged
daspecster merged 3 commits intogoogleapis:masterfrom
daspecster:add-storage-preserve-acl
Sep 28, 2016
Merged

Preserve ACL when copying or renaming blob.#2409
daspecster merged 3 commits intogoogleapis:masterfrom
daspecster:add-storage-preserve-acl

Conversation

@daspecster
Copy link
Copy Markdown
Contributor

See: #2389

ACL settings are not held when copying a blob or renaming a blob.

Adds preserve_acl to Bucket.copy_blob and Bucket.rename_blob.

@daspecster daspecster added the api: storage Issues related to the Cloud Storage API. label Sep 23, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 23, 2016

def copy_blob(self, blob, destination_bucket, new_name=None,
client=None):
client=None, preserve_acl=False):

This comment was marked as spam.

This comment was marked as spam.

return new_blob

def rename_blob(self, blob, new_name, client=None):
def rename_blob(self, blob, new_name, client=None, preserve_acl=False):

This comment was marked as spam.

This comment was marked as spam.

self.assertEqual(new_blob.name, NEW_NAME)
self.assertIsInstance(new_blob.acl, ObjectACL)
self.assertFalse(new_blob.acl.loaded)
self.assertEqual(new_blob.acl.entities, {})

This comment was marked as spam.

@daspecster daspecster force-pushed the add-storage-preserve-acl branch from 8e85462 to 04ea4f7 Compare September 27, 2016 21:54
@daspecster
Copy link
Copy Markdown
Contributor Author

I finally circled back around.
@tseaver I think I found the issue! I had the wrong order of operations.
I also added assertions for the expected requests.

Copy link
Copy Markdown
Contributor

@tseaver tseaver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@daspecster daspecster merged commit a7cb215 into googleapis:master Sep 28, 2016
@daspecster daspecster deleted the add-storage-preserve-acl branch September 28, 2016 09:23
parthea pushed a commit that referenced this pull request Apr 1, 2026
PR created by the Librarian CLI to initialize a release. Merging this PR
will auto trigger a release.

Librarian Version: v0.7.0
Language Image:
us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator@sha256:1a2a85ab507aea26d787c06cc7979decb117164c81dd78a745982dfda80d4f68
<details><summary>bigframes: 2.36.0</summary>

##
[2.36.0](googleapis/python-bigquery-dataframes@v2.35.0...v2.36.0)
(2026-02-17)

### Features

* Initial support for biglake iceberg tables (#2409)
([ae35a989](googleapis/python-bigquery-dataframes@ae35a989))

* add bigquery.ai.generate_table function (#2453)
([b925aa24](googleapis/python-bigquery-dataframes@b925aa24))

### Documentation

* fix generate_text and generate_table input docs (#2455)
([078bd32e](googleapis/python-bigquery-dataframes@078bd32e))

* update multimodal dataframe notebook to use public APIs (#2456)
([342fa723](googleapis/python-bigquery-dataframes@342fa723))

* use direct API for pdf chunk and pdf extract (#2452)
([543ce52c](googleapis/python-bigquery-dataframes@543ce52c))

* use direct API for audio transcription (#2447)
([59cbc5db](googleapis/python-bigquery-dataframes@59cbc5db))

* Add EXIF metadata extraction example to multimodal notebook (#2429)
([84c6f883](googleapis/python-bigquery-dataframes@84c6f883))

* Update multimodal notebook to use public runtime helpers (#2451)
([e36dd8b4](googleapis/python-bigquery-dataframes@e36dd8b4))

</details>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants