Skip to content

Comments

fix: update ansible roles and setup script for jazzy#228

Open
xygyo77 wants to merge 6 commits intomainfrom
fix_for_jazzy
Open

fix: update ansible roles and setup script for jazzy#228
xygyo77 wants to merge 6 commits intomainfrom
fix_for_jazzy

Conversation

@xygyo77
Copy link
Contributor

@xygyo77 xygyo77 commented Feb 15, 2026

Description

In jazzy, add "--break-system-packages" option to pip installations.

Related links

https://tier4.atlassian.net/browse/SYSPERF-104

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.

  • The PR has been properly tested.
  • The PR has been reviewed.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • (Optional) The PR has been properly tested with CARET_report verification.
  • There are no open discussions or they are tracked via tickets.
  • The PR is ready for merge.

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

@miyakoshi-dev
Copy link
Contributor

In setup_caret.sh, ansible-playbook returned the error "command not found."
I set the path myself, but
Wouldn't it be better if it could run without the user having to set the PATH, or if the message explained how to set it up?
I also received the error "command not found" for jupyter-lab.

Signed-off-by: ISP akm <akm@isp.co.jp>
@xygyo77
Copy link
Contributor Author

xygyo77 commented Feb 25, 2026

I have updated the script to require the environment variable ALLOW_BREAK_SYSTEM_PACKAGES=true when running on ROS 2 Jazzy.

xygyo77 and others added 4 commits February 25, 2026 08:22
Signed-off-by: ISP akm <akm@isp.co.jp>
Signed-off-by: ISP akm <akm@isp.co.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants