Skip to content

import retry module - #2049

Merged
andrewpollock merged 1 commit into
google:masterfrom
cuixq:fix-retry
Mar 12, 2024
Merged

andrewpollock merged 1 commit into
google:masterfrom
cuixq:fix-retry

Conversation

@cuixq

@cuixq cuixq commented Mar 12, 2024

Copy link
Copy Markdown
Contributor

Fixes the issue AttributeError: module 'google.cloud.storage' has no attribute brought by #2046

@cuixq
cuixq marked this pull request as draft March 12, 2024 05:29
@cuixq
cuixq marked this pull request as ready for review March 12, 2024 05:39

@andrewpollock andrewpollock 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.

Sorry I didn't pick up on this in the previous PR :-(

@andrewpollock
andrewpollock merged commit dcb6a15 into google:master Mar 12, 2024
@cuixq

cuixq commented Mar 12, 2024

Copy link
Copy Markdown
Contributor Author

No worries! This is a good learning experience for me as well! :)

@cuixq
cuixq deleted the fix-retry branch March 12, 2024 05:45
CharlyReux pushed a commit to CharlyReux/osv.dev that referenced this pull request May 1, 2024
Fixes the issue `AttributeError: module 'google.cloud.storage' has no
attribute` brought by google#2046
andrewpollock added a commit to andrewpollock/osv.dev that referenced this pull request May 9, 2024
In the same spirit of what was done in google#2046 and google#2049

Recently saw a transient 502 error in Production that would have
benefited from this.
andrewpollock added a commit that referenced this pull request May 9, 2024
In the same spirit of what was done in #2046 and #2049

Recently saw a transient 502 error in Production from the importer that
would have benefited from this, and decided to look more broadly for any
other situations where this could crop up in the future...
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.

3 participants