Skip to content

Fix GAX logging system tests - #1952

Merged
tseaver merged 6 commits into
googleapis:masterfrom
tseaver:logging-fix_gax_system_tests
Jul 6, 2016
Merged

Fix GAX logging system tests#1952
tseaver merged 6 commits into
googleapis:masterfrom
tseaver:logging-fix_gax_system_tests

Conversation

@tseaver

Copy link
Copy Markdown
Contributor

Logging system tests now pass both withGCLOUD_ENABLE_GAXand without it.

The _retry_backoff function added in 3a3613e may help toward #1619.

@tseavertseaver added testing api: logging Issues related to the Cloud Logging API. labels Jul 1, 2016
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 1, 2016


def _retry_backoff(result_predicate, meth, *args, **kw):
from grpc.beta.interfaces import StatusCode

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@tseaver

Copy link
Copy Markdown
ContributorAuthor

@daspecster I added the missing continue in 6ffe88d. Any remaining issues?

@daspecster

Copy link
Copy Markdown
Contributor

LGTM!

@tseaver
tseaver merged commit 771bc7a into googleapis:masterJul 6, 2016
@tseaver
tseaver deleted the logging-fix_gax_system_tests branch July 6, 2016 19:09
This was referenced Aug 3, 2016
parthea pushed a commit that referenced this pull request Mar 6, 2026
* feat: support load job option ColumnNameCharacterMap
* add unit test
parthea pushed a commit that referenced this pull request Apr 1, 2026
…1943)" (#1952)
This reverts commit a511e09e6924d2e8302af2eb4a602c6b9e5d2d72.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: loggingIssues related to the Cloud Logging API.cla: yesThis human has signed the Contributor License Agreement.testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tseaver@daspecster@googlebot