sensu / sensu-transport

The Sensu transport abstraction library.
MIT License
14 stars 19 forks source link

Initial support for a NATS transport. #48

Open jmkeyes opened 7 years ago

jmkeyes commented 7 years ago

This PR implements a rudimentary NATS transport for Sensu. (#44)

I set up a test environment consisting of two VMs:

In this basic test environment I was able to send/receive messages from Sensu through NATS.

This library is only a first pass, so I'd like to get some feedback and see what I can do to improve it.

Cheers!

portertech commented 7 years ago

This excites me :+1: I'll have to give it a whirl tomorrow afternoon.

portertech commented 7 years ago

@cwjohnston how did this work for you?

portertech commented 7 years ago

This pr can potentially close https://github.com/sensu/sensu-transport/issues/44

cwjohnston commented 7 years ago

Worked for me in a simple test with gnatsd running locally, no TLS. I installed the system profile extension in the client and was able to observe events being processed by the server.

cwjohnston commented 6 years ago

@jmkeyes sorry that the work you have offered here has languished for so long. As a company Sensu has been working towards releasing a next-generation product using a completely different transport model, so we aren't in any position to actively support or promote alternative transports.

Have you considered packaging this independently as an Ruby gem? One example of this approach would be SimpleFinance's SNS SQS transport.