Re: [Freeze Break Request] Remove extraneous notify on /etc/sysconfig/puppet

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 14 Sep 2012 02:54:45 +0000
Jon Stanley <jonstanley@xxxxxxxxx> wrote:

> Since we're not using the puppet agent anymore, if
> the /etc/sysconfig/puppet file is changed, there is no need to notify
> the puppet service. At best, this would be a no-op, and at worst it
> would start the agent, which we don't want.
> ---
>  modules/puppet/manifests/init.pp |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/modules/puppet/manifests/init.pp
> b/modules/puppet/manifests/init.pp index c04f2e1..bd016aa 100644
> --- a/modules/puppet/manifests/init.pp
> +++ b/modules/puppet/manifests/init.pp
> @@ -31,7 +31,6 @@ class puppet::client {
>  
>      file { "/etc/sysconfig/puppet":
>          source  => "puppet:///puppet/puppet.sysconfig",
> -        notify  => Service["puppet"],
>          require => Package["puppet"],
>      }
>      file { "/usr/local/bin/run-puppet":

Sure, seems a simple and easy fix. +1

kevin

Attachment: signature.asc
Description: PGP signature

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux