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

Static systemd unit files #88

Open
ghost opened this issue Mar 9, 2018 · 5 comments
Open

Static systemd unit files #88

ghost opened this issue Mar 9, 2018 · 5 comments

Comments

@ghost
Copy link

ghost commented Mar 9, 2018

The systemd unit files as shipped with RobinHood version 3.1 are static, ie. lack an Install section. Therefore, RobinHood instances can only be started explicitly, or as a dependency from another, custom unit file. Has this been done intentionally, or should RobinHood be able to automatically start on boot by default like in non-systemd environments?

@tl-cea
Copy link
Member

tl-cea commented Mar 9, 2018

Nice to ear from you :-)
Most of the time admins don't want robinhood to start on boot to make sure to mount the filesystem before.

@ghost
Copy link
Author

ghost commented Mar 9, 2018

Hi Thomas, good to hear you, too! Thanks for the quick response.
In this case, wouldn't it be more straightforward to install the service, and default to disabled (which at least on EL7 seems to be the default, anyway). This way, one can systemctl enable/disable robinhood.service as desired whereas in the current setup, one has to add a drop-in or a dummy service.

@tl-cea
Copy link
Member

tl-cea commented Mar 9, 2018

This sounds good. It is true we saw this static service is disappointing for puppet that considers "static" as enabled.
Could you provide us with an example of such Install section? I'm not a systemd expert, so if you can give me the right config, it will be faster. I'll be glad to integrate your proposal.

@ghost
Copy link
Author

ghost commented Mar 9, 2018

Sure, I'll send you a PR when I'm done. (And, yes, I started looking into this because our configuration management tools are unhappy with static services as well.)

@ghost
Copy link
Author

ghost commented Jul 9, 2018

Finally managed to upload the versions we've been running for a while, cf. https://review.gerrithub.io/c/cea-hpc/robinhood/+/418699 for a simple version, and https://review.gerrithub.io/c/cea-hpc/robinhood/+/418704 with more systemd magic, but also additional naming restrictions.

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

No branches or pull requests

1 participant