Skip to content

Add retries to verification in _test_gs_cache(). #2372

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 26, 2019

Conversation

DailyDreaming
Copy link
Contributor

@DailyDreaming DailyDreaming commented Aug 24, 2019

Should address the flaky test in #2276 (though I haven't had this particular test fail on me).

@DailyDreaming DailyDreaming requested a review from chmreid August 24, 2019 01:20
Copy link
Contributor

@chmreid chmreid left a comment

Choose a reason for hiding this comment

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

LGTM!

@DailyDreaming DailyDreaming merged commit fc8d411 into master Aug 26, 2019
@DailyDreaming DailyDreaming deleted the lon-fix-flaky-caching-test branch August 26, 2019 18:40
chmreid added a commit that referenced this pull request Aug 28, 2019
* master:
  Revert "Bump httpie from 1.0.2 to 1.0.3 (#2397)" (#2401)
  Bump httpie from 1.0.2 to 1.0.3 (#2397)
  Fix flaky test: TestFileApi.test_file_get_checkout . (#2396)
  Add retries to verification in _test_gs_cache(). (#2372)
  Added details of notify intervals (#2368)
chmreid added a commit that referenced this pull request Sep 11, 2019
* master: (36 commits)
  Adding a document to explain the action and resource relationship for… (#2395)
  Requirements Update (#2410)
  Increase dss notify workers. (#2408)
  Revert "Bump httpie from 1.0.2 to 1.0.3 (#2397)" (#2401)
  Bump httpie from 1.0.2 to 1.0.3 (#2397)
  Fix flaky test: TestFileApi.test_file_get_checkout . (#2396)
  Add retries to verification in _test_gs_cache(). (#2372)
  Added details of notify intervals (#2368)
  Fix failing collections tests
  Add pagination response headers (closes #2287)
  Fix improper markdown (#2367)
  Fix 500 on delete nonexistent bundle, again (fixes #1918) (#2351)
  Add ttl to async db items. (#2363)
  DSS Operations: Checkout (#2366)
  Revert "DSS/Operations: Checkout  (#2264)" (#2365)
  DSS/Operations: Checkout  (#2264)
  remove hard-coded stage name in secrets tests (#2364)
  This pins pyyaml <= 5.1 for aws cli compatibility (#2348)
  Update like to security policy (#2361)
  Relax sync size limit (#2354)
  ...
DailyDreaming added a commit that referenced this pull request Sep 17, 2019
Add swagger changes for an associated bundle argument.

Address review comments.

Syntax error.

Swagger changes.

Swagger changes.

Linting.

Address review comments.

Address comments.

Update how content-disposition is passed in.

Update swagger description.

Add retries to verification in _test_gs_cache(). (#2372)

Fix flaky test: TestFileApi.test_file_get_checkout . (#2396)

Bump httpie from 1.0.2 to 1.0.3 (#2397)

Bumps [httpie](https://github.com/jakubroztocil/httpie) from 1.0.2 to 1.0.3.
- [Release notes](https://github.com/jakubroztocil/httpie/releases)
- [Changelog](https://github.com/jakubroztocil/httpie/blob/master/CHANGELOG.rst)
- [Commits](httpie/cli@1.0.2...1.0.3)

Signed-off-by: dependabot[bot] <support@github.com>

Revert "Bump httpie from 1.0.2 to 1.0.3 (#2397)" (#2401)

This reverts commit ef3476b.

Increase dss notify workers. (#2408)

Requirements Update (#2410)

Adding a document to explain the action and resource relationship for… (#2395)

* Adding a document to explain the action and resource relationship for defining authorization rules.

* Adding DSS Roles

update terraform in docker file (#2424)

[easy] Add file-type and file-size for ttfb fetching. (#2425)

Bundle: enumeration (#2318)

DSS SmokeTest: Test Order (#2428)

forgot to fix this endpoint here too (#2430)

Bump httpie from 1.0.2 to 1.0.3 (closes #2404)

Bump cloud-blobstore version to 3.2.0. (#2431)
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.

2 participants