Skip to content

Pass mutated properties through to 'bucket.create' API call. - #1579

Merged
tseaver merged 1 commit into
googleapis:masterfrom
tseaver:1575-bucket_location_at_create
Mar 9, 2016
Merged

Pass mutated properties through to 'bucket.create' API call.#1579
tseaver merged 1 commit into
googleapis:masterfrom
tseaver:1575-bucket_location_at_create

Conversation

@tseaver

Copy link
Copy Markdown
Contributor

Fixes: #1575.

@tseavertseaver added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: storage Issues related to the Cloud Storage API. labels Mar 7, 2016
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 7, 2016
client = self._require_client(client)
query_params = {'project': client.project}
properties = dict(
(key, self._properties[key]) for key in self._changes)

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@dhermes

Copy link
Copy Markdown
Contributor

LGTM

tseaver added a commit that referenced this pull request Mar 9, 2016
Pass mutated properties through to 'bucket.create' API call.
@tseaver
tseaver merged commit cf581cb into googleapis:masterMar 9, 2016
@tseaver
tseaver deleted the 1575-bucket_location_at_create branch March 9, 2016 01:10
@dhermesdhermes mentioned this pull request Apr 1, 2016
parthea pushed a commit that referenced this pull request Nov 24, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 26, 2025
* chore: initial setup for async auth sessions api (#1571)
* chore: initial setup for async auth sessions api
* fix whitespace
* add init file
* update file names to aiohttp
* update import statement
* feat: Implement asynchronous timeout context manager (#1569)
* feat: implement async timeout guard
* add docstring
* clean whitespace
* update import file name
* add missing return statement
* update test cases
* update test cases
* include underlying timeout exception in trace
* avoid the cost of actual time
* feat: Implement asynchronous `AuthorizedSession` api response class (#1575)
* feat: implement asynchronous response class for AuthorizedSessions API
* check if aiohttp is installed and avoid tests dependency
* update content to be async
* update docstring to be specific to aiohttp
* add type checking and avoid leaking underlying API responses
* add test case for iterating chunks
* add read method to response interface
* address PR comments
* fix lint issues
* feat: Implement asynchronous `AuthorizedSession` api request class (#1579)
* feat: implement request class for asynchoronous AuthorizedSession API
* add type checking and address TODOs
* remove default values from interface methods
* aiohttp reponse close method must not be awaited
* cleanup
* update Request class docstring
* feat: Implement asynchronous `AuthorizedSession` class (#1580)
* feat: Implement Asynchronous AuthorizedSession class
* add comment for implementing locks within refresh
* move timeout guard to sessions
* add unit tests and code cleanup
* implement async exponential backoff iterator
* cleanup
* add testing for http methods and cleanup
* update number of retries to 3
* refactor test cases
* fix linter and mypy issues
* fix pytest code coverage
* fix: avoid leaking api error for closed session
* add error handling for response
* cleanup default values and add test coverage
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* cleanup: minor code cleanup (#1589)
* chore: Add aiohttp requirements test constraint. (#1566)
See https://github.com/googleapis/google-auth-library-python/issues/1565
for more information.
* chore(main): release 2.33.0 (#1560)
* chore(main): release 2.33.0
* fix: retry token request on retryable status code (#1563)
* fix: retry token request on retryable status code
* feat(auth): Update get_client_ssl_credentials to support X.509 workload certs (#1558)
* feat(auth): Update get_client_ssl_credentials to support X.509 workload certs
* feat(auth): Update has_default_client_cert_source
* feat(auth): Fix formatting
* feat(auth): Fix test__mtls_helper.py
* feat(auth): Fix function name in tests
* chore: Refresh system test creds.
* feat(auth): Fix style
* feat(auth): Fix casing
* feat(auth): Fix linter issue
* feat(auth): Fix coverage issue
---------
Co-authored-by: Carl Lundin <clundin@google.com>
Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>
* chore: Update ECP deps. (#1583)
* chore(main): release 2.34.0 (#1574)
* cleanup: minor code cleanup
* fix lint issues
---------
Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Andy Zhao <andyzhao@google.com>
Co-authored-by: Carl Lundin <clundin@google.com>
* update secrets from forked repo
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: arithmetic1728 <58957152+arithmetic1728@users.noreply.github.com>
Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Andy Zhao <andyzhao@google.com>
Co-authored-by: Carl Lundin <clundin@google.com>
parthea pushed a commit that referenced this pull request Mar 2, 2026
Thank you for opening a Pull Request! Before submitting your PR, there
are a few things you can do to make sure it goes smoothly:
- [x] Make sure to open an issue as a
[bug/issue](https://github.com/googleapis/python-pubsub/issues/new/choose)
before writing your code! That way we can discuss the change, evaluate
designs, and agree on the general idea
- [x] Ensure the tests and linter pass
- [x] Code coverage does not decrease (if any source code was changed)
- [x] Appropriate docs were updated (if necessary)
Fixes#1578 🦕
parthea pushed a commit that referenced this pull request Mar 6, 2026
…p/templates/python_library/.kokoro (#1579)
Source-Link: googleapis/synthtool@d0f51a0
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:240b5bcc2bafd450912d2da2be15e62bc6de2cf839823ae4bf94d4f392b451dc
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Apr 1, 2026
* chore: add a `scratch/` directory to gitignore
Used for manual testing
* revert owlbot changes
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storageIssues related to the Cloud Storage API.cla: yesThis human has signed the Contributor License Agreement.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tseaver@dhermes@googlebot