Skip to content

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented Oct 9, 2018

Also, make logger names unique between test runs, in an attempt to avoid race condition in deletion of a logger from a previous test run, vs. current logger.

Closes #5303.

Attempt to avoid race condition in deletion of a logger from a previous
test run, vs. current logger.
@tseaver tseaver requested review from crwilcox and theacodes October 9, 2018 17:10
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 9, 2018
@tseaver tseaver added testing api: logging Issues related to the Cloud Logging API. flaky labels Oct 9, 2018
@tseaver tseaver merged commit 47843a0 into master Oct 9, 2018
@tseaver tseaver deleted the 5303-logging-moar_backoff_mashing branch October 9, 2018 17:35
@tseaver
Copy link
Contributor Author

tseaver commented Oct 9, 2018

Merged without review, because this is a testing-only change which turns the logging systests green for the first time in weeks (months?).

parthea pushed a commit that referenced this pull request Nov 24, 2025
Also, make logger names unique between test runs.

Closes #5303.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: logging Issues related to the Cloud Logging API. cla: yes This human has signed the Contributor License Agreement. flaky testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants