Skip to content

fix: vision automl - delete unused files that have moved #2835

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
Merged
merged 8 commits into from
Mar 5, 2020
Merged

fix: vision automl - delete unused files that have moved #2835

merged 8 commits into from
Mar 5, 2020

Conversation

nnegrey
Copy link
Contributor

@nnegrey nnegrey commented Feb 10, 2020 •
edited
Loading

Fix: #2823
Fix: #2824

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 10, 2020
@nnegrey
Copy link
Contributor Author

nnegrey commented Feb 10, 2020

@kurtisvg, do we have docker installed on kokoro?
I guess some other tests are trying to use it and the sample is in docs.
https://github.com/GoogleCloudPlatform/python-docs-samples/blob/master/vision/automl/edge_container_predict/automl_vision_edge_container_predict_test.py#L61

@nnegrey
Copy link
Contributor Author

nnegrey commented Feb 18, 2020

Was blocked by: googleapis/testing-infra-docker#51

@nnegrey nnegrey marked this pull request as ready for review February 27, 2020 20:28
@nnegrey nnegrey requested a review from a team as a code owner February 27, 2020 20:28
@nnegrey nnegrey added the automerge Merge the pull request once unit tests and other checks pass. label Mar 4, 2020
@nnegrey nnegrey merged commit 3a76887 into master Mar 5, 2020
@nnegrey nnegrey deleted the fix-automl-vision branch March 5, 2020 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leahecole leahecole leahecole approved these changes

@busunkim96 busunkim96 busunkim96 approved these changes

Assignees

@kurtisvg kurtisvg

Labels
automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Milestone
No milestone
5 participants
@nnegrey @leahecole @busunkim96 @googlebot @kurtisvg