python-docs-samples icon indicating copy to clipboard operation
python-docs-samples copied to clipboard

datacatalog.quickstart.quickstart_test: test_quickstart failed

Open flaky-bot[bot] opened this issue 9 months ago • 9 comments

Note: #10216 was also for this test, but it was closed more than 10 days ago. So, I didn't mark it flaky.


commit: c8b11a689bc5aacad120b5ee7890835fdd0f0367 buildURL: Build Status, Sponge status: failed

flaky-bot[bot] avatar May 03 '24 09:05 flaky-bot[bot]

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

flaky-bot[bot] avatar May 03 '24 13:05 flaky-bot[bot]

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

flaky-bot[bot] avatar May 04 '24 09:05 flaky-bot[bot]

Looks like this issue is flaky. :worried:

I'm going to leave this open and stop commenting.

A human should fix and close this.


commit: 2b3e64e00574cef9b879761c223ec5bceea1f64d buildURL: Build Status, Sponge status: failed

flaky-bot[bot] avatar May 06 '24 09:05 flaky-bot[bot]

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 avatar May 09 '24 19:05 davidcavazos

Most likely a transient issue, closing for now

davidcavazos avatar May 13 '24 22:05 davidcavazos

Oops! Looks like this issue is still flaky. It failed again. :grimacing:

I reopened the issue, but a human will need to close it again.


commit: ecabbe688cd259260049377f5c3357f9edd88e2c buildURL: Build Status, Sponge status: failed

flaky-bot[bot] avatar May 14 '24 09:05 flaky-bot[bot]

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 avatar May 21 '24 15:05 davidcavazos

Failure happens on datacatalog_client.lookup_entry

davidcavazos avatar May 21 '24 15:05 davidcavazos

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.

davidcavazos avatar May 21 '24 15:05 davidcavazos