Skip to content

Adding retries for failed copy_blob() calls in system tests. - #2290

Merged
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:fix-2289
Sep 9, 2016
Merged

Adding retries for failed copy_blob() calls in system tests.#2290
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:fix-2289

Conversation

@dhermes

Copy link
Copy Markdown
Contributor

Fixes#2289.


I ran it locally until I could force the error:

(system-tests) $ .tox/system-tests/bin/nosetests system_tests/storage.py -vv --nocapture
nose.config: INFO: Ignoring files matching ['^\\.', '^_', '^setup\\.py$']
test_create_bucket (storage.TestStorageBuckets) ... ok
test_list_buckets (storage.TestStorageBuckets) ... ok
test_list_files (storage.TestStorageListFiles) ... ok
test_paginate_files (storage.TestStorageListFiles) ... ok
400 No file found in request. (POST https://www.googleapis.com/storage/v1/b/new_1473450503532/o/file01.txt/copyTo/b/new_1473450503532/o/parent%2Fchild%2Ffile21.txt), Trying again in 1 seconds...
400 No file found in request. (POST https://www.googleapis.com/storage/v1/b/new_1473450503532/o/file01.txt/copyTo/b/new_1473450503532/o/parent%2Fchild%2Ffile21.txt), Trying again in 2 seconds...
400 No file found in request. (POST https://www.googleapis.com/storage/v1/b/new_1473450503532/o/file01.txt/copyTo/b/new_1473450503532/o/parent%2Fchild%2Fgrand%2Ffile31.txt), Trying again in 1 seconds...
test_first_level (storage.TestStoragePseudoHierarchy) ... ok
test_root_level_w_delimiter (storage.TestStoragePseudoHierarchy) ... ok
test_second_level (storage.TestStoragePseudoHierarchy) ... ok
test_third_level (storage.TestStoragePseudoHierarchy) ... ok
test_create_signed_delete_url (storage.TestStorageSignURLs) ... ok
test_create_signed_read_url (storage.TestStorageSignURLs) ... ok
test_copy_existing_file (storage.TestStorageWriteFiles) ... ok
test_direct_write_and_read_into_file (storage.TestStorageWriteFiles) ... ok
test_large_file_write_from_stream (storage.TestStorageWriteFiles) ... ok
test_small_file_write_from_filename (storage.TestStorageWriteFiles) ... ok
test_write_metadata (storage.TestStorageWriteFiles) ... ok
----------------------------------------------------------------------
Ran 15 tests in 26.949s
OK

@dhermesdhermes added api: storage Issues related to the Cloud Storage API. testing flaky labels Sep 9, 2016
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 9, 2016
Comment threadsystem_tests/storage.py Outdated


def setUpModule():
_helpers.PROJECT = TESTS_PROJECT

This comment was marked as spam.

This comment was marked as spam.

@tseaver

Copy link
Copy Markdown
Contributor

LGTM

@dhermes
dhermes merged commit 3072014 into googleapis:masterSep 9, 2016
@dhermes
dhermes deleted the fix-2289 branch September 9, 2016 21:52
@dhermesdhermes mentioned this pull request Sep 19, 2016
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:c8612d3fffb3f6a32353b2d1abd16b61e87811866f7ec9d65b59b02eb452a620
<details><summary>bigframes: 2.30.0</summary>
##
[2.30.0](google/bigframes@v2.29.0...v2.30.0)
(2025-12-03)
### Features
* Support mixed scalar-analytic expressions (#2239)
([20ab469d](google/bigframes@20ab469d))
* Allow drop_duplicates over unordered dataframe (#2303)
([52665fa5](google/bigframes@52665fa5))
* Preserve source names better for more readable sql (#2243)
([64995d65](google/bigframes@64995d65))
* use end user credentials for `bigframes.bigquery.ai` functions when
`connection_id` is not present (#2272)
([7c062a68](google/bigframes@7c062a68))
* pivot_table supports fill_value arg (#2257)
([8f490e68](google/bigframes@8f490e68))
* Support builtins funcs for df.agg (#2256)
([956a5b00](google/bigframes@956a5b00))
* add bigquery.json_keys (#2286)
([b487cf1f](google/bigframes@b487cf1f))
* Add agg/aggregate methods to windows (#2288)
([c4cb39dc](google/bigframes@c4cb39dc))
* Add bigframes.pandas.crosstab (#2231)
([c62e5535](google/bigframes@c62e5535))
* Implement single-column sorting for interactive table widget (#2255)
([d1ecc61b](google/bigframes@d1ecc61b))
### Bug Fixes
* Pass credentials properly for read api instantiation (#2280)
([3e3fe259](google/bigframes@3e3fe259))
* Update max_instances default to reflect actual value (#2302)
([4489687e](google/bigframes@4489687e))
* Improve Anywidget pagination and display for unknown row counts
(#2258)
([508deae5](google/bigframes@508deae5))
* Fix issue with stream upload batch size upload limit (#2290)
([6cdf64b0](google/bigframes@6cdf64b0))
* calling info() on empty dataframes no longer leads to errors (#2267)
([95a83f77](google/bigframes@95a83f77))
* do not warn with DefaultIndexWarning in partial ordering mode (#2230)
([cc2dbae6](google/bigframes@cc2dbae6))
### Documentation
* update docs and tests for Gemini 2.5 models (#2279)
([08c0c0c8](google/bigframes@08c0c0c8))
* Add Google Analytics configuration to conf.py (#2301)
([0b266da1](google/bigframes@0b266da1))
* fix LogisticRegression docs rendering (#2295)
([32e53134](google/bigframes@32e53134))
* update API reference to new `dataframes.bigquery.dev` location (#2293)
([da064397](google/bigframes@da064397))
* use autosummary to split documentation pages (#2251)
([f7fd2d20](google/bigframes@f7fd2d20))
</details>
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.testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@dhermes@tseaver@daspecster@googlebot