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

Regenerate ood-portal.conf when Apache starts or is reloaded #371

Merged
merged 2 commits into from
Mar 3, 2020

Conversation

treydock
Copy link
Contributor

@treydock treydock commented Jan 8, 2020

This requires some extra testing. Failures from update_ood_portal will not break Apache because the command is prefixed with a hyphen.

@johrstrom johrstrom added this to the OOD1.7 milestone Mar 2, 2020
@johrstrom
Copy link
Contributor

I'm good with this. I can't find documentation on having multiple ExecReloads but I'm guessing you know it's ok?

If we're ready, I'm ready.

@treydock
Copy link
Contributor Author

treydock commented Mar 3, 2020

I'd like to test this further with Puppet, because we generate ood-portal.conf with Puppet using generator so this maybe conflict.

For ExecReload this is docs: https://www.freedesktop.org/software/systemd/man/systemd.service.html , follows pattern of ExecStart and supports multiple commands that are executed in serial order

treydock added a commit to OSC/puppet-module-openondemand that referenced this pull request Mar 3, 2020
@treydock
Copy link
Contributor Author

treydock commented Mar 3, 2020

I will just delete the file added with Puppet because the Puppet module expects to be the thing executing ood-portal-generator: OSC/puppet-module-openondemand#23

@treydock treydock merged commit f4dc474 into master Mar 3, 2020
@treydock treydock deleted the systemd-conf branch March 3, 2020 16:28
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.

2 participants