CI: Re-combine Build and Test Steps for Coverage Generation
Description
Issue #694 (closed) broke up the build
and test
portions of the coverage
CI jobs. Unfortunately, this had unintended consequences. For all other build/test CI jobs, the build is done with colcon build --cmake-args=-DCMAKE_BUILD_TYPE=Release
and no additional flags. This currently produces a build/
folder just under 600MB (the limit for artifacts which can be transferred between build jobs is 1GB). However, when the profilinng flags are added and the tests are built with RelWithDebInfo
as is done for the coverage
jobs, the build/
folder grows to over 1.6GB, mostly due to ament_acado
, which will be moved to an external repository as part of #702 (closed). However, even if it weren't for ament_acado
, the testing job for coverage
needs significantly more context from the build/
and install/
folders than other build/test jobs.
Expected behavior
While splitting the build
and test
jobs makes sense and is feasible for most other stages of the CI build, doing so for the coverage
stage is not reasonable. It should be re-combined into a single step.
Definition of Done
-
Re-combine build
andtest
jobs of thecoverage
build step into a single job.