sensu / puppet-module-sensuclassic

Puppet module to manage Sensu Classic (ruby version)
MIT License
0 stars 11 forks source link

Service started before config applied #32

Closed StuartMcManus closed 4 years ago

StuartMcManus commented 4 years ago

Description of problem

Command used and debugging output

Platform and version information

Anything else to add that you think will be helpful?

treydock commented 4 years ago

@StuartMcManus Can you provide logs of all changes made by Puppet on a failed Puppet run? You can limit them to just changes from sensuclassic module. The dsc_service['sensu-client'] resource is supposed to just install the service and the service['sensu-client'] resource is supposed to actually start the service. I am curious if in your case the dsc_service resource is actually starting the service.

StuartMcManus commented 4 years ago

Hello @treydock sorry I should have provided this straight off but it is definitely dsc that is starting the service. I have attached a refined log of just sensuclassic changes and below is the DSC JSON that gets created for starting the service along with a Windows eventlog screenshot for the service starting.

DSC module version: 1.9.3

{"time": "2020-07-13T09:56:44.707+01:00", "type": "verbose", "message": "[IP-X-X-X-X]: LCM: [ Start Set ] [[Service]DirectResourceAccess] "}, {"time": "2020-07-13T09:56:48.468+01:00", "type": "verbose", "message": "[IP-X-X-X-X]: [[Service]DirectResourceAccess] Service 'sensu-client' started."}, {"time": "2020-07-13T09:56:48.469+01:00", "type": "verbose", "message": "[IP-X-X-X-X]: LCM: [ End Set ] [[Service]DirectResourceAccess] in 3.7610 seconds."}, {"time": "2020-07-13T09:56:48.469+01:00", "type": "verbose", "message": "[IP-X-X-X-X]: LCM: [ End Set ] in 3.7920 seconds."}

Cheers, Stuart Puppet_Service_Run.txt sensu_service

treydock commented 4 years ago

@StuartMcManus Can you see if https://github.com/sensu/puppet-module-sensuclassic/pull/34 solves this issue?

StuartMcManus commented 4 years ago

Hello @treydock it looks like that does fix the issue so I have updated #34 with a comment and approved.