Added a new cmake flag to enable isolation#501
Added a new cmake flag to enable isolation#501adityapande-1995 wants to merge 2 commits intoament:rollingfrom
Conversation
Signed-off-by: Aditya Pande <adityapande@google.com>
cottsay
left a comment
There was a problem hiding this comment.
We should also trigger a build where the flag is inverted and unconditionally enabled so we can "see what breaks" :)
Signed-off-by: Aditya Pande <adityapande@google.com>
|
Build round 2, with isolation enabled and a parallel executor : Build args : Without isolation but with a parallel executor : Build args : |
|
TODO :
|
This adds a new cmake argument called "ISOLATE_TEST" to use https://github.com/adityapande-1995/linux_isolate_process for isolating a test. This would mean more tests can be run in parallel, and can speed up CI. Due to the underlying implementation using the
unsharesystem call, this can only be used in linux.The proposal is to add
linux_isolate_processas a dependency ofament_cmake_test, so that downstream packages get it for free, and hopefully addlinux_isolate_processas a core package in ROS 2.Custom repos file : https://gist.github.com/adityapande-1995/0f05b9acf72be5fbb34e80e4342cfe26
The plan further is to add a similar pytest plugin.