Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Preserve ACL when copying or renaming blob.#2409

Merged
daspecster merged 3 commits into
googleapis:mastergoogleapis/google-cloud-python:masterfrom
daspecster:add-storage-preserve-acldaspecster/google-cloud-python:add-storage-preserve-aclCopy head branch name to clipboard
Sep 28, 2016
Merged

Preserve ACL when copying or renaming blob.#2409
daspecster merged 3 commits into
googleapis:mastergoogleapis/google-cloud-python:masterfrom
daspecster:add-storage-preserve-acldaspecster/google-cloud-python:add-storage-preserve-aclCopy head branch name to clipboard

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
Comment thread google/cloud/storage/bucket.py Outdated

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.

Comment thread google/cloud/storage/bucket.py Outdated
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.

Comment thread unit_tests/storage/test_bucket.py Outdated
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.

@tseaver tseaver left a comment

Copy link
Copy Markdown
Contributor

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.