Closed kevindweb closed 5 years ago
CI won't run because it's not up right now, not a bug
@onvm quick test here
@onvm quick test here
Your results will arrive shortly
qqq
Your results will arrive shortly
qqq
Your results will arrive shortly
@onvm are we testing today?
@onvm for real this time
@onvm for real this time
Your results will arrive shortly
@onvm quick test
@onvm quick test
Another CI run in progress, adding request to the end of the list
@onvm quick
@onvm quick
Another CI run in progress, adding request to the end of the list
@onvm test 2 dont run
@onvm test 2 dont run
Duplicate request already waiting, ignoring message
@onvm can we do it
@onvm can we do it
Another CI run in progress, adding request to the end of the list
@onvm test 4 dont run manager
@onvm test 4 dont run manager
Duplicate request already waiting, ignoring message
@onvm can we do it
Your results will arrive shortly
@onvm unauthorized run linter
@onvm unauthorized run linter
Your results will arrive shortly
@onvm did we mess up
@onvm did we mess up
Another CI run in progress, adding request to the end of the list
@onvm did we mess up
Your results will arrive shortly
@onvm this will run first
@onvm this will run first
Your results will arrive shortly
@onvm start
@onvm same test
@onvm last test
@onvm one last time please
qqq
Your results will arrive shortly
qqq
Your results will arrive shortly
Fixes to performance degradation of onvm's CI
Summary:
There are a few slow-downs that occur in the startup process. The main one is with dependency installation, right now it has to install a lot of modules and packages every CI run. There is no reason that after starting up the web server, the packages would suddenly disappear. Only after very long running CI runs would this ever happen.
Usage: run a PR and see the difference in speed, mostly a difference over the course of many PRs consectively
Merging notes:
TODO before merging :
Test Plan:
make sure CI still does it's job, just tested with #126 , not incredibly fast, but much better and eliminates install redundancy
Review:
@koolzz pls make sure I didn't break again, there's going to be another update probably tonight, too