Skip to content

samples: increased wait time for undeployed model prediction - #3286

Merged
averikitsch merged 2 commits into
masterfrom
fix-predict-test
Jul 1, 2020
Merged

samples: increased wait time for undeployed model prediction#3286
averikitsch merged 2 commits into
masterfrom
fix-predict-test

Conversation

@munkhuushmgl

Copy link
Copy Markdown
Contributor

Fixes#2469

I was able to reproduce the error with the following steps:

  1. undeploy model
  2. run predict the test --> which will go into setup of the test (tries to deploy the model again) and since deploying model takes 5-30 min.

Then, the actual test runs while the deployment is not finished.

@munkhuushmgl
munkhuushmgl requested a review from a teamJune 30, 2020 23:24
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 30, 2020
@averikitschaverikitsch removed the cla: yes This human has signed the Contributor License Agreement. label Jul 1, 2020
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 1, 2020
@averikitsch
averikitsch merged commit 32173ce into masterJul 1, 2020
@averikitsch
averikitsch deleted the fix-predict-test branch July 1, 2020 15:26
Shabirmean pushed a commit that referenced this pull request Nov 17, 2022
…vision/automl (#261)
* samples: Automl (#1158)
* Test push
* Vision AutoML
* Vision AutoML updates + Translate AutoML
* Translate README fixes
* Fixing Kokoro failure issue
* Language AutoML
* Vision AutoML
* Translate AutoML files added
* Triggering tests
* Triggering tests
* samples: Automl (#1162)
* Test push
* Vision AutoML
* Vision AutoML updates + Translate AutoML
* Translate README fixes
* Fixing Kokoro failure issue
* Language AutoML
* Vision AutoML
* Translate AutoML files added
* Triggering tests
* Triggering tests
* Updates based on comments
* Updates after review comments
* Fixed build issue
* samples: Clean up the code and fix the sample tests (#1463)
* samples: Prevent Name collisions on tests (#1466)
* samples: fix misspelling reported by user (#1451)
* samples: Auto-update dependencies. (#1467)
* Auto-update dependencies.
* Rollback
* Auto-update dependencies.
* Fix merge errors
* Rollback
* Fix tests
* Fix test
* samples: Add new sample for deploying a model with a node count (#1601)
* samples: Add missing samples for classification and rename object detection sa… (#1604)
* Add missing samples for classification and rename object detection sample to add clarification
* Update test function names
* samples: automl: fix old beta snippet tests (#1994)
* samples: docs: update tests that are failing or not cleaning up resources
* samples: update shared config (#2443)
* update shared config
* Update to 1.0.13
* lint
* Fix linting
* lint
* fix imports
Co-authored-by: Les Vogel <lesv@users.noreply.github.com>
* samples: bug: fix importData sample in order to bump libraries-bom version (#3011)
Fixes#2943
* samples: automl: remove vision samples no longer on cgc (#2844)
- [ x] `pom.xml` parent set to latest `shared-configuration`
- [ in progress] Appropriate changes to README are included in PR
- [ ] API's need to be enabled to test (tell us)
- [ ] Environment Variables need to be set (ask us to set them)
- [see below] Tests pass (`mvn -P lint clean verify`)
- [x ] Please **merge** this PR for me once it is approved.
Need to take a look still at modelApi tests (it looks like the entire file is obviated, but could be refactored to test model creation?) Could use a second set of eyes here.
* samples: samples: increased wait time for undeployed model prediction (#3286)
* samples: increased wait time for undeployed model prediction
* fixed the lint issue
Co-authored-by: Nirupa Anantha Kumar <nirupa-kumar@users.noreply.github.com>
Co-authored-by: Noah Negrey <nnegrey@users.noreply.github.com>
Co-authored-by: Andrew Ferlitsch <aferlitsch@gmail.com>
Co-authored-by: DPEBot <dpebot@google.com>
Co-authored-by: nnegrey <nnegrey@google.com>
Co-authored-by: Averi Kitsch <akitsch@google.com>
Co-authored-by: Les Vogel <lesv@users.noreply.github.com>
Co-authored-by: Stephanie Wang <stephaniewang526@users.noreply.github.com>
Co-authored-by: Anthony <wens.ajw@gmail.com>
Co-authored-by: Mike <45373284+munkhuushmgl@users.noreply.github.com>
Shabirmean pushed a commit that referenced this pull request Nov 18, 2022
…vision/automl (#261)
* samples: Automl (#1158)
* Test push
* Vision AutoML
* Vision AutoML updates + Translate AutoML
* Translate README fixes
* Fixing Kokoro failure issue
* Language AutoML
* Vision AutoML
* Translate AutoML files added
* Triggering tests
* Triggering tests
* samples: Automl (#1162)
* Test push
* Vision AutoML
* Vision AutoML updates + Translate AutoML
* Translate README fixes
* Fixing Kokoro failure issue
* Language AutoML
* Vision AutoML
* Translate AutoML files added
* Triggering tests
* Triggering tests
* Updates based on comments
* Updates after review comments
* Fixed build issue
* samples: Clean up the code and fix the sample tests (#1463)
* samples: Prevent Name collisions on tests (#1466)
* samples: fix misspelling reported by user (#1451)
* samples: Auto-update dependencies. (#1467)
* Auto-update dependencies.
* Rollback
* Auto-update dependencies.
* Fix merge errors
* Rollback
* Fix tests
* Fix test
* samples: Add new sample for deploying a model with a node count (#1601)
* samples: Add missing samples for classification and rename object detection sa… (#1604)
* Add missing samples for classification and rename object detection sample to add clarification
* Update test function names
* samples: automl: fix old beta snippet tests (#1994)
* samples: docs: update tests that are failing or not cleaning up resources
* samples: update shared config (#2443)
* update shared config
* Update to 1.0.13
* lint
* Fix linting
* lint
* fix imports
Co-authored-by: Les Vogel <lesv@users.noreply.github.com>
* samples: bug: fix importData sample in order to bump libraries-bom version (#3011)
Fixes#2943
* samples: automl: remove vision samples no longer on cgc (#2844)
- [ x] `pom.xml` parent set to latest `shared-configuration`
- [ in progress] Appropriate changes to README are included in PR
- [ ] API's need to be enabled to test (tell us)
- [ ] Environment Variables need to be set (ask us to set them)
- [see below] Tests pass (`mvn -P lint clean verify`)
- [x ] Please **merge** this PR for me once it is approved.
Need to take a look still at modelApi tests (it looks like the entire file is obviated, but could be refactored to test model creation?) Could use a second set of eyes here.
* samples: samples: increased wait time for undeployed model prediction (#3286)
* samples: increased wait time for undeployed model prediction
* fixed the lint issue
Co-authored-by: Nirupa Anantha Kumar <nirupa-kumar@users.noreply.github.com>
Co-authored-by: Noah Negrey <nnegrey@users.noreply.github.com>
Co-authored-by: Andrew Ferlitsch <aferlitsch@gmail.com>
Co-authored-by: DPEBot <dpebot@google.com>
Co-authored-by: nnegrey <nnegrey@google.com>
Co-authored-by: Averi Kitsch <akitsch@google.com>
Co-authored-by: Les Vogel <lesv@users.noreply.github.com>
Co-authored-by: Stephanie Wang <stephaniewang526@users.noreply.github.com>
Co-authored-by: Anthony <wens.ajw@gmail.com>
Co-authored-by: Mike <45373284+munkhuushmgl@users.noreply.github.com>
anguillanneuf pushed a commit that referenced this pull request Dec 5, 2022
…vision/automl (#261)
* samples: Automl (#1158)
* Test push
* Vision AutoML
* Vision AutoML updates + Translate AutoML
* Translate README fixes
* Fixing Kokoro failure issue
* Language AutoML
* Vision AutoML
* Translate AutoML files added
* Triggering tests
* Triggering tests
* samples: Automl (#1162)
* Test push
* Vision AutoML
* Vision AutoML updates + Translate AutoML
* Translate README fixes
* Fixing Kokoro failure issue
* Language AutoML
* Vision AutoML
* Translate AutoML files added
* Triggering tests
* Triggering tests
* Updates based on comments
* Updates after review comments
* Fixed build issue
* samples: Clean up the code and fix the sample tests (#1463)
* samples: Prevent Name collisions on tests (#1466)
* samples: fix misspelling reported by user (#1451)
* samples: Auto-update dependencies. (#1467)
* Auto-update dependencies.
* Rollback
* Auto-update dependencies.
* Fix merge errors
* Rollback
* Fix tests
* Fix test
* samples: Add new sample for deploying a model with a node count (#1601)
* samples: Add missing samples for classification and rename object detection sa… (#1604)
* Add missing samples for classification and rename object detection sample to add clarification
* Update test function names
* samples: automl: fix old beta snippet tests (#1994)
* samples: docs: update tests that are failing or not cleaning up resources
* samples: update shared config (#2443)
* update shared config
* Update to 1.0.13
* lint
* Fix linting
* lint
* fix imports
Co-authored-by: Les Vogel <lesv@users.noreply.github.com>
* samples: bug: fix importData sample in order to bump libraries-bom version (#3011)
Fixes#2943
* samples: automl: remove vision samples no longer on cgc (#2844)
- [ x] `pom.xml` parent set to latest `shared-configuration`
- [ in progress] Appropriate changes to README are included in PR
- [ ] API's need to be enabled to test (tell us)
- [ ] Environment Variables need to be set (ask us to set them)
- [see below] Tests pass (`mvn -P lint clean verify`)
- [x ] Please **merge** this PR for me once it is approved.
Need to take a look still at modelApi tests (it looks like the entire file is obviated, but could be refactored to test model creation?) Could use a second set of eyes here.
* samples: samples: increased wait time for undeployed model prediction (#3286)
* samples: increased wait time for undeployed model prediction
* fixed the lint issue
Co-authored-by: Nirupa Anantha Kumar <nirupa-kumar@users.noreply.github.com>
Co-authored-by: Noah Negrey <nnegrey@users.noreply.github.com>
Co-authored-by: Andrew Ferlitsch <aferlitsch@gmail.com>
Co-authored-by: DPEBot <dpebot@google.com>
Co-authored-by: nnegrey <nnegrey@google.com>
Co-authored-by: Averi Kitsch <akitsch@google.com>
Co-authored-by: Les Vogel <lesv@users.noreply.github.com>
Co-authored-by: Stephanie Wang <stephaniewang526@users.noreply.github.com>
Co-authored-by: Anthony <wens.ajw@gmail.com>
Co-authored-by: Mike <45373284+munkhuushmgl@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PredictionApiIT(sponge_log).testPredict failed

3 participants

@munkhuushmgl@averikitsch@googlebot