Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move symlinks into clearpath-robot.service.wants instead of multi-user.target.wants #187

Merged
merged 6 commits into from
Apr 8, 2025

Conversation

civerachb-cpr
Copy link
Contributor

@civerachb-cpr civerachb-cpr commented Apr 3, 2025

Create a new systemd provider that doesn't explicitly add symlinks to multi-user.target.wants, use this provider to create the sub-jobs that are part of the clearpath-robot.service job.

Fix the symlink generated by RobotProvider so it properly creates the main job symlink in multi-user.target.wants.

Resolves RPSW-2228

@civerachb-cpr civerachb-cpr marked this pull request as ready for review April 4, 2025 17:42
@civerachb-cpr civerachb-cpr requested a review from a team as a code owner April 4, 2025 17:42
@civerachb-cpr civerachb-cpr requested review from roni-kreinin, luis-camero and hilary-luo and removed request for a team and luis-camero April 4, 2025 17:42
@tonybaltovski tonybaltovski merged commit 6d4ec34 into rc/jazzy/2.3 Apr 8, 2025
1 of 12 checks passed
@civerachb-cpr civerachb-cpr deleted the fix/symlinks branch April 8, 2025 15:04
luis-camero pushed a commit that referenced this pull request Apr 8, 2025
…user.target.wants` (#187)

* Add clearpath-robot.servce symlink to multi-user.wants, add a new provider to create symlinks in clearpath-robot.wants

* provider -> Provider

* Fix moving the symlink

* clearpath-robot.target -> clearpath-robot.service

* Update the post-install instructions to add systemctl-enable

* Fix the symlink generated by the RobotProvider
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.

3 participants