-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
datacatalog.quickstart.quickstart_test: test_quickstart failed #11616
Comments
commit: 977110a |
Test passed for commit 892c56a (Build Status, Sponge)! Closing this issue. |
Looks like this issue is flaky. 😟 I'm going to leave this open and stop commenting. A human should fix and close this. commit: 2b3e64e |
Error: |
Most likely a transient issue, closing for now |
Oops! Looks like this issue is still flaky. It failed again. 😬 I reopened the issue, but a human will need to close it again. commit: ecabbe6 |
Same error as before. Definitely not consistent, but I'm not sure why this one is flaky. This will need looking in more detail. |
Failure happens on |
The test looks pretty simple, using some fixtures for the resources. Those fixtures are defined in the The resource setup seems correct. The permissions setup is a bit convoluted but it's just taking the default credentials, which seems fine, and it works sometimes. This makes me think the code itself works, but maybe the Data Catalog entries take a bit of time to propagate from the time the table was created. Maybe just adding a retry with exponential backoff would fix this? From the sample code, it looks like simply retrying the whole test would be safe to do. |
Note: #10216 was also for this test, but it was closed more than 10 days ago. So, I didn't mark it flaky.
commit: c8b11a6
buildURL: Build Status, Sponge
status: failed
The text was updated successfully, but these errors were encountered: