Add distros to end-to-end automation #2769
Conversation
narrieta
requested review from
ZhidongPeng,
nagworld9 and
maddieford
as code owners
Codecov Report
@@ Coverage Diff @@
## develop #2769 +/- ##
===========================================
+ Coverage 71.97% 71.99% +0.01%
===========================================
Files 103 104 +1
Lines 15692 15831 +139
Branches 2486 2264 -222
===========================================
+ Hits 11295 11397 +102
- Misses 3881 3913 +32
- Partials 516 521 +5
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
| @@ -11,7 +11,9 @@ schedules: | |||
| - develop | |||
| always: true | |||
|
|
|||
| # no PR triggers | |||
| trigger: | |||
There was a problem hiding this comment.
The cleanup pipeline is being run on PRs. Adding the trigger to make it match the settings of the main pipeline.
tests_e2e/orchestrator/runbook.yml
Outdated
| @@ -123,7 +123,7 @@ combinator: | |||
| location: $(location) | |||
| vm_size: $(vm_size) | |||
|
|
|||
| concurrency: 10 | |||
| concurrency: 16 | |||
There was a problem hiding this comment.
What does concurrency here refer to?
There was a problem hiding this comment.
Nevermind, I see the Teams chat discussion
tests_e2e/test_suites/images.yml
Outdated
| # debian_8: "credativ Debian 8 latest" | ||
| # | ||
| alma_9: "almalinux:almalinux:9-gen2:latest" |
There was a problem hiding this comment.
can we keep same format across all distros? or is it supposed to be that way?
There was a problem hiding this comment.
yes, this was a copy-n-paste and forgot to replace the :
i want to use spaces all over because improves readibility. i'll update this one
There was a problem hiding this comment.
btw (LISA accepts both, CLI requires colon)
There was a problem hiding this comment.
I see few other places too; can you also update those? Thanks.
Including additional distros for the daily run, and increasing the concurrency level.
Fixes #2588