ovn-org / ovn-heater

Mega script to deploy/configure/run OVN scale tests.
Apache License 2.0
12 stars 12 forks source link

ci: Fix logs checking in Cirrus-CI jobs. #164

Closed dceara closed 1 year ago

dceara commented 1 year ago

The check stage only cares about the last instruction's exit code. Run all checks inside a script that fails on the first error.

dceara commented 1 year ago

CC: @igsilya

igsilya commented 1 year ago

OK. The main issue here is the negation. It's treated as a subshell, so errexit is not effective. We need to generate the exit manually it seems.

dceara commented 1 year ago

OK. The main issue here is the negation. It's treated as a subshell, so errexit is not effective. We need to generate the exit manually it seems.

You're right. I added explicit exits now.