Windows Handlers not handled by module

[DDSloan96]

Description of problem

• Attempted to add handler to windows sensu client
• Handler gets created
• Try to add a handler to windows

Platform and version information

• Windows
• Your version of this module: Latest

[ghoneycutt]

@DDSloan96 Thank you for letting us know about this issue! Could you please update this issue with example code regarding the handlers so that we can reproduce the problem.

[DDSloan96]

sensu::handlers:
    operations:                                                                   
      type: 'pipe'                                                                
      command: "/opt/sensu/embedded/bin/handler-pagerduty.rb"                     
      severities:                                                                 
        - 'critical' 

This is an example of what causes the issue. I have a feeling the issue stems from this code

sensu::handlers:                                                                
  operations:                                                                   
    type: 'pipe'                                                                
    command: "/opt/sensu/embedded/bin/handler-pagerduty.rb"                     
    severities:                                                                 
      - 'critical' 

As it doesn't have handling for windows

[ghoneycutt]

Do you get any error messages when using this code?

What data do you expect to be in which file?

[DDSloan96]

Error: Failed to apply catalog: Parameter path failed on File[/etc/sensu/conf.d/
handlers/operations.json]: File paths must be fully qualified, not '/etc/sensu/c
onf.d/handlers/operations.json' at /etc/puppetlabs/code/environments/ldr/modules
_forge/sensu/manifests/handler.pp:119

This is the error message we get, The expected output should be

{
  "handlers": {
    "operations": {
      "command": "/opt/sensu/embedded/bin/handler-pagerduty.rb",
      "type": "pipe",
      "filters": [

      ],
      "severities": [
        "critical"
      ],
      "handle_flapping": false,
      "handle_silenced": false
    }
  }
} 

[treydock]

@DDSloan96 Just to clarify, your attempting to apply the previously mentioned Hiera defined resource to a Windows puppet client that is acting as a Sensu server? If my understanding is correct, I would think command would need to be prefixed with C:. I'll also patch sensu::handler to better handle Windows.

[ghoneycutt]

BTW: We support windows as a client, not as a server. From the code, I believe that @DDSloan96 is using windows as a client.

[DDSloan96]

We are using it as a client

[treydock]

I've confirmed the issue inside a vagrant Windows VM. I am working on patches that will make sensu::handler compatible with Windows.

[ghoneycutt]

Released in v2.53.0