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

Fix foreman service being enabled after disable #730

Merged
merged 1 commit into from
May 17, 2023

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented May 17, 2023

The conditional is no longer needed since having Puma as our app server for foreman. This code path introduced a bug where once foreman was disabled it would never be enabled again.

The conditional is no longer needed since having Puma as our app
server for foreman. This code path introduced a bug where once foreman
was disabled it would never be enabled again.
Copy link
Contributor

@Griffin-Sullivan Griffin-Sullivan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Followed steps in the BZ here with this build.

Result after reboot:

● foreman.service - Foreman
   Loaded: loaded (/usr/lib/systemd/system/foreman.service; enabled; vendor pre>
  Drop-In: /etc/systemd/system/foreman.service.d
           └─installer.conf
   Active: active (running) since Wed 2023-05-17 14:14:48 EDT; 2min 36s ago
     Docs: https://theforeman.org

@ehelms ehelms merged commit 0436eab into theforeman:master May 17, 2023
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants