Enable Monitoring Library by default

[fgallegosalido]

Description

This pull request enables the Monitoring Library in the Connext RMW by default.

Is this user-facing behavior change?

The Connext RMW will now send monitoring data by default.

Did you use Generative AI?

No

Additional Information

We had to disable the monitoring library as we were very close to the Lyrical code freeze.

[fgallegosalido]

@fujitatomoya could you backport this to Lyrical?

[mergify]

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

• Queue this pull request

[ahcorde]

Pulls: #243
Gist: https://gist.githubusercontent.com/ahcorde/c083358f7409bd9e931b2e1c897939c1/raw/9b2d7cdd75cbb7c230c1a031306b0050b7290b55/ros2.repos
BUILD args: --packages-above-and-dependencies rmw_connextdds_common
TEST args: --packages-above rmw_connextdds_common
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/19681

• Linux
• Linux-aarch64
• Linux-rhel
• Windows

[fujitatomoya]

@fgallegosalido since this changes the default behavior, probably we do not backport this lyrical?

[fgallegosalido]

@fujitatomoya we would really like to get this backported to Lyrical. A couple of reasons to do so:

• This feature provides a way for ROS 2 users to monitor their applications through our Observability framework.
• Right now it is explicitly disabled, and it can not be enabled by any other means.
• This feature is transparent to users, and can easily be disabled through an environment variable if needed.
• The next ROS 2 LTS is in 2 years, which is a long wait for getting this feature in the RMW.

[fgallegosalido]

@fujitatomoya can you check my previous bullet points? We would like to backport this pull request to Lyrical. Let me know what you think.

[oliviadorenczrti]

New windows failures that seem related: #248

@fgallegosalido we should wait to backport until this is resolved

[fgallegosalido]

@fujitatomoya the Windows issues have now been fixed by #250. Let me know if you have any doubts about backporting this to Lyrical.

[fujitatomoya]

@fgallegosalido you mean backporting #243 and #250 to lyrical, right?

[fgallegosalido]

Indeed. #250 will be necessary too.

[fujitatomoya]

@Mergifyio backport lyrical

[mergify]

backport lyrical

✅ Backports have been created

Details

• #251 Enable Monitoring Library by default (backport #243) has been created for branch lyrical