add CycloneDDS to Two Processes Test Results

[joespeed]

Please add CycloneDDS to the following tests:

• Performance Two Processes Test Results (Array1k)
• Performance Two Processes Test Results (multisize messages)

CycloneDDS is already included in the following tests:

• Performance One Process Test Results (Array1k)
• Performance One Process Test Results (multisize messages)

rmw_cyclonedds is already in all tests.
@cottsay @dirk-thomas @eboasson @k0ekk0ek
http://build.ros2.org/job/Fci__nightly-performance_ubuntu_focal_amd64/plot/

[cottsay]

Here is the spot that needs to change (be removed) for this to happen:

buildfarm_perf_tests/test/CMakeLists.txt

Lines 97 to 103 in bc4e002

	# TODO (ahcorde): perf_test is not working with CycloneDDS when
	# processes are splitted in two
	# what(): Round trip mode is not implemented for Cyclone DDS!
	set(PERF_TEST_SKIP_two_process_CycloneDDS_async ON
	CACHE BOOL "Skip two-process CycloneDDS async tests")
	set(PERF_TEST_SKIP_two_process_CycloneDDS_sync ON
	CACHE BOOL "Skip two-process CycloneDDS sync tests")

Has the referenced issue been resolved? Is that fix released for Foxy or Eloquent as well?

[joespeed]

@cottsay thank you for the insight, we will check that
@eboasson @k0ekk0ek

[cottsay]

It appears that this is still not implemented in performance_test, so we can't do anything about it here. When the situation changes, we can remove the "skip" flags here to enable the tests you want.

https://gitlab.com/ApexAI/performance_test/-/blob/master/performance_test/src/communication_abstractions/cyclonedds_communicator.hpp#L186-188

I think the best thing to do would be to track this on the Apex.AI repository, since that is where the feature needs to be implemented.

[y-okumura-isp]

Has anyone noticed my PR?