tier4 / caret

CARET (Chain-Aware ROS Evaluation Tool) is one of performance analysis tools dedicated with ROS 2 applications
https://tier4.github.io/caret_doc/main/
Apache License 2.0
64 stars 16 forks source link

chore(ansible): add ulimit execution to setenv_caret.bash #62

Closed takam5f2 closed 1 year ago

takam5f2 commented 1 year ago

Signed-off-by: Takayuki AKAMINE takayuki.akamine@tier4.jp

Description

I want to add ulimit explanation to setenv_caret.bash.

After updating LTTng from 2.12 to 2.13, users have to mind the maximum number of file descriptor. I think that setenv_caret.bash must have some description for users to notice it.

Related links

https://tier4.github.io/CARET_doc/latest/recording/recording/#starting-lttng-session-manually

You may find that size of recorded data is strangely smaller than expected after updating LTTng to 2.13 if you apply CARET to a large application like Autoware which has hundreds of nodes. You have to suspect that maximum number of file descriptors is not enough in the case. You can check the number with ulimit -n command. The default maximum number is 1024, but it is not enough for the large application. You can avoid this problem by enlarging the maximum number with executing the command; ulimit -n 65536.

Notes for reviewers

Pre-review checklist for the PR author

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

After all checkboxes are checked, anyone who has write access can merge the PR.