Skip to content
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

Open
flaky-bot bot opened this issue May 3, 2024 · 9 comments
Open

datacatalog.quickstart.quickstart_test: test_quickstart failed #11616

flaky-bot bot opened this issue May 3, 2024 · 9 comments
Assignees
Labels
api: datacatalog Issues related to the Data Catalog API. flakybot: flaky Tells the Flaky Bot not to close or comment on this issue. flakybot: issue An issue filed by the Flaky Bot. Should not be added manually. priority: p2 Moderately-important priority. Fix may not be included in next release. samples Issues that are directly related to samples. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@flaky-bot
Copy link

flaky-bot bot commented May 3, 2024

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

@flaky-bot flaky-bot bot added flakybot: issue An issue filed by the Flaky Bot. Should not be added manually. priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels May 3, 2024
@product-auto-label product-auto-label bot added api: datacatalog Issues related to the Data Catalog API. samples Issues that are directly related to samples. labels May 3, 2024
Copy link
Author

flaky-bot bot commented May 3, 2024

commit: 977110a
buildURL: Build Status, Sponge
status: failed

Copy link
Author

flaky-bot bot commented May 4, 2024

Test passed for commit 892c56a (Build Status, Sponge)! Closing this issue.

@flaky-bot flaky-bot bot closed this as completed May 4, 2024
@flaky-bot flaky-bot bot reopened this May 6, 2024
@flaky-bot flaky-bot bot added the flakybot: flaky Tells the Flaky Bot not to close or comment on this issue. label May 6, 2024
Copy link
Author

flaky-bot bot commented May 6, 2024

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
buildURL: Build Status, Sponge
status: failed

@davidcavazos
Copy link
Contributor

Error:
google.api_core.exceptions.PermissionDenied: 403 No permission to lookup Entry "//bigquery.googleapis.com/projects/python-docs-samples-tests/datasets/python_data_catalog_sample_20240506093127_bddbb5ee/tables/python_data_catalog_sample_20240506093128_dfb2c81f" or it does not exist.

@davidcavazos
Copy link
Contributor

Most likely a transient issue, closing for now

@flaky-bot flaky-bot bot reopened this May 14, 2024
Copy link
Author

flaky-bot bot commented May 14, 2024

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
buildURL: Build Status, Sponge
status: failed

@davidcavazos
Copy link
Contributor

google.api_core.exceptions.PermissionDenied: 403 No permission to lookup Entry "//bigquery.googleapis.com/projects/python-docs-samples-tests/datasets/python_data_catalog_sample_20240514092932_ff1d2d9b/tables/python_data_catalog_sample_20240514092933_b8c54596" or it does not exist.

Same error as before. Definitely not consistent, but I'm not sure why this one is flaky. This will need looking in more detail.

@davidcavazos
Copy link
Contributor

Failure happens on datacatalog_client.lookup_entry

@davidcavazos
Copy link
Contributor

davidcavazos commented May 21, 2024

The test looks pretty simple, using some fixtures for the resources. Those fixtures are defined in the conftest.py file.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: datacatalog Issues related to the Data Catalog API. flakybot: flaky Tells the Flaky Bot not to close or comment on this issue. flakybot: issue An issue filed by the Flaky Bot. Should not be added manually. priority: p2 Moderately-important priority. Fix may not be included in next release. samples Issues that are directly related to samples. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

No branches or pull requests

1 participant