Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixes #7716: Restart foreman-tasks when katello.yml changes. #61

Merged
merged 1 commit into from
Feb 26, 2015

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Feb 19, 2015

No description provided.

@jlsherrill
Copy link
Contributor

Should this also notify the httpd service?

@ehelms
Copy link
Member Author

ehelms commented Feb 20, 2015

Good call. I was just blindly following the bug.
On Feb 19, 2015 6:19 PM, "Justin Sherrill" notifications@github.com wrote:

Should this also notify the httpd service?


Reply to this email directly or view it on GitHub
#61 (comment).

@ehelms
Copy link
Member Author

ehelms commented Feb 23, 2015

@jlsherrill Added httpd

@@ -15,6 +15,7 @@
group => $katello::group,
mode => '0644',
before => [Class['foreman::database'], Exec['foreman-rake-db:migrate']],
notify => [Service['foreman-tasks'], Service['httpd']],
Copy link
Member

Choose a reason for hiding this comment

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

I thought this was the same as #62, but also here you may want to notify Class['foreman::service'].

@ehelms
Copy link
Member Author

ehelms commented Feb 24, 2015

Green here to!

@iNecas
Copy link
Member

iNecas commented Feb 26, 2015

ACK

ehelms added a commit that referenced this pull request Feb 26, 2015
Fixes #7716: Restart foreman-tasks when katello.yml changes.
@ehelms ehelms merged commit f9406fa into theforeman:master Feb 26, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants