search

saltstack-formulas / ntp-formula

http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
23 stars 158 forks source link

another try to fix the NTP service in SLE11 distros #27

Closed tampakrap closed 8 years ago

tampakrap commented 8 years ago

this time by using a map for osrelease

gravyboat commented 8 years ago

Looks good to me but I'll leave this one to @iggy since he's been working on it already.

patrickpierson commented 8 years ago

is this also tracking the broken ubuntu ntp load? or just SUSE? 

----------
          ID: ntp_running
    Function: service.running
        Name: ntp
      Result: False
     Comment: The following requisites were not found:
                                 watch:
                                     file: /etc/ntp.conf
     Started: 
    Duration: 
     Changes:   
----------
gravyboat commented 8 years ago

@patrickpierson That was already fixed: https://github.com/saltstack-formulas/ntp-formula/blob/master/ntp/init.sls#L27

Edit: Unless you're referring to something else regarding what the file is actually supposed to be.

aboe76 commented 8 years ago

@iggy are you able too look at this pr?