Skip to content

dialogflow.cloud-client.delete_knowledge_base_test: test_delete_knowledge_base failed #3113

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.

Sign up for GitHub

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

Jump to bottom
Closed
flaky-bot bot opened this issue Mar 18, 2020 · 7 comments
Closed

dialogflow.cloud-client.delete_knowledge_base_test: test_delete_knowledge_base failed #3113

flaky-bot bot opened this issue Mar 18, 2020 · 7 comments
Assignees
@nnegrey
Labels
api: dialogflow Issues related to the Dialogflow API. 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.

Comments

@flaky-bot
Copy link

flaky-bot bot commented Mar 18, 2020

This test failed!

To configure my behavior, see the Build Cop Bot documentation.

If I'm commenting on this issue too often, add the buildcop: quiet label and
I will stop commenting.

commit: d4014b0
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 Mar 18, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Mar 18, 2020

Looks like this issue is flaky. 😟

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

A human should fix and close this.

commit: d4014b0
buildURL: Build Status, Sponge
status: passed

@nnegrey nnegrey assigned nnegrey and unassigned dmahugh Mar 19, 2020
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Mar 24, 2020
@JustinBeckwith JustinBeckwith added the api: dialogflow Issues related to the Dialogflow API. label Apr 2, 2020
@nnegrey nnegrey added priority: p2 Moderately-important priority. Fix may not be included in next release. and removed priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Apr 10, 2020
@nnegrey
Copy link
Contributor

nnegrey commented Apr 13, 2020

Waiting on a PR to remove a bunch of Dialogflow samples, then will clean up the tests

@yoshi-automation yoshi-automation removed the 🚨 This issue needs some love. label Apr 17, 2020
@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 23, 2020

@nnegrey Which PR? If it's blocked, maybe I can expedite the review process :)

@nnegrey
Copy link
Contributor

nnegrey commented Apr 23, 2020

I haven't yet seen the PR yet for Python. It is part of the DRIFT clean up process.
I've seen them go through: Java, Node.js, PHP, Golang so far.

@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 23, 2020

Isn't it #3249 ?

@nnegrey
Copy link
Contributor

nnegrey commented Apr 23, 2020

Ah! Yes it is, I must've missed it entirely. THANKS

@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 25, 2020

The test file was deleted in #3249. Closing

@tmatsuo tmatsuo closed this as completed Apr 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nnegrey nnegrey

Labels
api: dialogflow Issues related to the Dialogflow API. 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.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@tmatsuo @JustinBeckwith @dmahugh @nnegrey @yoshi-automation