search

liquidat / nagios-icinga-openvpn

Nagios/Icinga check for OpenVPN availability monitoring
MIT License
51 stars 27 forks source link

Add distutils support #20

Closed orlitzky closed 6 years ago

orlitzky commented 6 years ago

Hi, a long time ago, I wrote a very basic nagios plugin called _checkopenvpn-simple. I'm happy to say, yours is way better. In particular, it supports TLS-auth and I never got around to that. So, tl;dr, I'd like to switch.

To keep things neat and clean on our servers, I'd like to have this plugin packaged for Gentoo. No problem there; I'm a gentoo developer, I'll do it. But packaging would be a lot easier with two things:

This PR adds the basic distutils support. I moved some things around, but am happy to amend the PR with whatever you fancy. Once setup.py is there and can be used for the installation, it will build on Gentoo almost automatically. That leaves only the release tarball, which I hope I can convince you to tag and release afterwards =)

With the new distutils support, creating a release is as simple as updating the version in setup.py and running python setup.py sdist.

liquidat commented 6 years ago

Hi @orlitzky , thanks for the PR, looks good! I'll try to create the release tarball.

orlitzky commented 6 years ago

Added to Gentoo,

https://packages.gentoo.org/packages/net-analyzer/nagios-icinga-openvpn

and everything looks good. Thanks again!