-
Notifications
You must be signed in to change notification settings - Fork 138
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
[Backport 2.x] Fix flaky test of visualization tool (#2416) #2435
[Backport 2.x] Fix flaky test of visualization tool (#2416) #2435
Conversation
test case failures are related to PredictionITTests, looks like they are flaky also. I have create an issue to track #2436 and will fix it with another PR. |
@ylwu-amzn @dhrubo-os can you help to review #2437 first, this should fix the flaky test of PredictionITTest and get it backport to 2.x, after that i will rebase this backport PR with 2.x |
This PR has conflicts, @Hailong-am Can you fix ? |
* add wait for model to be undeployed Signed-off-by: Hailong Cui <ihailong@amazon.com> * spotless Signed-off-by: Hailong Cui <ihailong@amazon.com> * update model undeploy status Signed-off-by: Hailong Cui <ihailong@amazon.com> * reduce total wait time Signed-off-by: Hailong Cui <ihailong@amazon.com> --------- Signed-off-by: Hailong Cui <ihailong@amazon.com> (cherry picked from commit aa09014)
d8818d4
to
d7b62e7
Compare
Signed-off-by: Hailong Cui <ihailong@amazon.com>
this may not needed anymore as #2510 fix the undeploy issue. i will close this one. |
backport aa09014 from #2416