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

bug: Resolve difference between service unit and config #894

Closed

Conversation

brianredbeard
Copy link

In configuring system the foreman-proxy service refuses to start.
Upon investigation this is due to a conflict in the default
settings.yml file and the systemd service unit. In the
settings.yml file the default behavior is to run in the foreground
while in the systemd service unit the process is specified as
forking, utilizing a pid file. Either the settings file for
systemd based systems needs to be changed to reflect the intended
behavior of the service unit or the service unit needs to be
changed to reflect the desired behavior of settings.yml.

In configuring system the foreman-proxy service refuses to start.
Upon investigation this is due to a conflict in the default
settings.yml file and the systemd service unit.  In the
settings.yml file the default behavior is to run in the foreground
while in the systemd service unit the process is specified as
forking, utilizing a pid file. Either the settings file for
systemd based systems needs to be changed to reflect the intended
behavior of the service unit or the service unit needs to be
changed to reflect the desired behavior of settings.yml.
@domcleal
Copy link
Contributor

Thanks for the PR and bug report. We just addressed in this in the Debian packages actually, but we took the approach of changing settings.yml to have a forking configuration.

The reason for this is that our installer (https://github.com/theforeman/puppet-foreman_proxy) defaults to this mode for all OSes, including non-systemd ones.

Would you mind doing the same here? Probably applying a patch in the spec against the default settings.yml.example will work.

The equivalent Debian commit is here: 38a2776. We additionally set it to listen on localhost by default as that's needed to ensure the dpkg postinstall doesn't fail (as the service won't start). It's up to you if you want to do that here or not.

@mmoll
Copy link

mmoll commented Dec 2, 2015

@brianredbeard what's the status here? :)

@ehelms
Copy link
Member

ehelms commented Mar 9, 2017

This file no longer exists - I think we can close this out?

@mmoll
Copy link

mmoll commented Mar 9, 2017

while this PR stalled, I think the original issue was not fixed.

@mmoll mmoll closed this Mar 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants