Fixes #560 - Add systemctl wants to foreman.target for httpd

[qcjames53]

Why are you introducing these changes? (Problem description, related links)

When running systemctl start foreman.target, Apache (httpd) does not start on it's own. I attempted to include this as a part of the fix in #536 but biffed the implementation.

What are the changes introduced in this pull request?

• Corrects foreman.target's wants to properly include httpd. The old add-wants block was skipped by systemctl due to httpd being in the multi-user.target (higher prio). This was fine unless the user manually wanted to run systemctl stop foreman.target and systemctl start foreman.target, which might sometimes be the case.

How to test this pull request

Steps to reproduce:

1. Pull changes and rebuild the VMs.
2. Run vagrant ssh quadlet after VM boot.
3. Wait for services to spin up, then run systemctl list-dependencies foreman.target to verify httpd made the list.
4. Run systemctl stop foreman.target and systemctl status httpd to ensure it stopped.
5. Run systemctl start foreman.target and systemctl status httpd to ensure it is running.
6. Validate unit test changes.

Checklist

• Tests added/updated (if applicable)
• Documentation updated (if applicable) N/A

[ehelms]

This is how we do it for postgres -- https://github.com/theforeman/foremanctl/blob/master/src/roles/postgresql/tasks/main.yml#L37-L41 I think we are missing the default part.

[qcjames53]

Funnily enough that is where I stole the original approach from. I'm not a systemd expert by any means but Claude is indicating there's a difference between postgresql and httpd because the former is a quadlet and the latter is an rpm installed drop-in service; allegedly systemd does not merge "[Install]" blocks for drop-in services.

Is httpd intended to also be a quadlet? If so that's a bigger fish to fry.

[ehelms]

systemd is systemd. And yes, it won't merge the sections, drop-ins overwrite the previous instance of it.

Have you tried this as the drop-in?

        [Install]
        WantedBy=default.target foreman.target
        [Unit]
        PartOf=foreman.target

[qcjames53]

I'll report back on adding default.

[qcjames53]

I rebuilt a box from scratch and lo and behold default.target fixed everything (/etc/systemd/system/foreman.target.wants/httpd.service was properly created on deploy). I've adjusted the role to match this. Thanks Eric!

[ehelms]

Why did this need to move?