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
Cleanup presubmit #246
Cleanup presubmit #246
Conversation
/gcbrun |
Pull Request Test Coverage Report for Build 3333365663
💛 - Coveralls |
/gcbrun |
this will likely timeout due to loading of very large e8000 image |
/gcbrun |
/gcbrun |
/gcbrun |
/gcbrun |
1 similar comment
/gcbrun |
/gcbrun |
/gcbrun |
1 similar comment
/gcbrun |
/gcbrun |
Presubmit has issue bringing up ceos node (same as b/254952420) |
/gcbrun |
/gcbrun |
1 similar comment
/gcbrun |
/gcbrun |
/gcbrun |
The included ondatra test will not be used in the presubmit until the config reset through ondatra knebind works as expected for all vendors (currently there are timeout issues) |
/gcbrun |
|
||
import ( | ||
// "context" | ||
// "io" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the commented code.
* Add ondatra test to presubmit * ignore non unit tests in check * use internal deployment config * lint * linter * linter * Linter * Use latest image for prototyping * Use presubmit-ready deployment * Cleanup test * Fix imports * Cleanup config in multivendor nokia * Fix multivendor topo * Fix flags for ondatra test * fix paths * Comment out unused code * lint * linter * Increase timeout of presubmit * specify a custom topology for presubmit * fix linter * reduce timeout * roll forward ondatra knebind * Use approved base image * increase machine size * persist inotify changes after reboot * Disable ondatra test until config reset is more stable * Fix watches to instances for inotify
No description provided.