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

datalabeling.import_data_test: test_import_data failed #3439

Closed
flaky-bot bot opened this issue Apr 20, 2020 · 4 comments · Fixed by #3443
Closed

datalabeling.import_data_test: test_import_data failed #3439

flaky-bot bot opened this issue Apr 20, 2020 · 4 comments · Fixed by #3443
Assignees
Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. 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 Apr 20, 2020

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


commit: 8ea9cf4
buildURL: Build Status, Sponge
status: failed

@flaky-bot flaky-bot bot added buildcop: issue priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Apr 20, 2020
@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 20, 2020

The error is basically the API call is getting DeadlineExceeded: 504 Deadline Exceeded.
The test code look ok. Maybe we can mark the test as flaky? I'll try.

@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 20, 2020

The test code has the following:

    # If provided, use a provided test endpoint - this will prevent tests on                                                                                       
    # this snippet from triggering any action by a real human                                                                                                      
    if 'DATALABELING_ENDPOINT' in os.environ:
        opts = ClientOptions(api_endpoint=os.getenv('DATALABELING_ENDPOINT'))
        client = datalabeling.DataLabelingServiceClient(client_options=opts)

So be careful if anyone wants to run the tests locally. We're setting the envvar in

export DATALABELING_ENDPOINT="test-datalabeling.sandbox.googleapis.com:443"

@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 20, 2020

When setting the ENDPOINT, I'm getting an error 403 Data Labeling API (Test) has not been used in project 902283572421.

The link in the error message doesn't work, and I don't know how to enable that API. For now I'm just marking the tests as flaky and see the results tomorrow.

@flaky-bot
Copy link
Author

flaky-bot bot commented Apr 21, 2020

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

@flaky-bot flaky-bot bot closed this as completed Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants