Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make icinga2-diagnostics part of Icinga2? #57

Closed
jenslink opened this issue Dec 3, 2018 · 3 comments
Closed

Make icinga2-diagnostics part of Icinga2? #57

jenslink opened this issue Dec 3, 2018 · 3 comments

Comments

@jenslink
Copy link

jenslink commented Dec 3, 2018

Hi,

I would like to see this script as part of the Icinga2 package so I don't need to manualy copy this to a system if I need to run it. Some few line in the official documentation would also be great.

Jens

@widhalmt
Copy link
Member

Hi,

We'll discuss this and see what we can do.

Would it be ok if there was an icinga2-diagnostics package in the Icinga 2 repository so you could install it alongside Icinga 2?

Cheers,
Thomas

@dnsmichi
Copy link

dnsmichi commented Jan 30, 2019

I like the way it can be fetched independently. Once it is bundled in another package (icinga2 or icingaweb2), this needs a point release to update the diagnostics script. The past tells me that this isn't the best idea.

Depending on the involved scripts I would create a package, or not. If it is just this shell script, a simple curl/wget is sufficient imho (and one can automate this with Puppet for instance).

Cheers,
Michael

@widhalmt
Copy link
Member

We talked about how to distribute Diagnostics and devided we'd go for a seperate package. I'll close this issue in favor of #80 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants