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

Let the icinga check command also output the error when hlast reload failed #7263

Open
dgoetz opened this issue Jun 28, 2019 · 1 comment · May be fixed by #8430
Open

Let the icinga check command also output the error when hlast reload failed #7263

dgoetz opened this issue Jun 28, 2019 · 1 comment · May be fixed by #8430
Assignees
Labels
area/distributed Distributed monitoring (master, satellites, clients) enhancement New feature or request ref/NC TBD To be defined - We aren't certain about this yet

Comments

@dgoetz
Copy link
Contributor

dgoetz commented Jun 28, 2019

Is your feature request related to a problem? Please describe.

With an externally hosted satellite like NWS provides it, it is not always possible to log into the system and run the config validation on your own. Solving such an issue involves then support calls and needs more time than it should require.

Describe the solution you'd like

The icinga check command already warns you when the last reload attempt fails. Having the config validation or at least the error message as long output would be great.

Describe alternatives you've considered

I do not see a reason to have this optional, but perhaps it makes sense to have this additional output only when switching a parameter verbose to true, because most people can verify it on CLI?

Additional context

ref/NC/619027

@dnsmichi
Copy link
Contributor

I played around with that while implementing the improved cluster sync - imho it is too verbose inside the icinga check, so I just left it available inside the REST API for debugging calls.

@dnsmichi dnsmichi added area/distributed Distributed monitoring (master, satellites, clients) ref/NC TBD To be defined - We aren't certain about this yet labels Jun 28, 2019
@Al2Klimov Al2Klimov added the enhancement New feature or request label Aug 13, 2020
@Al2Klimov Al2Klimov self-assigned this Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/distributed Distributed monitoring (master, satellites, clients) enhancement New feature or request ref/NC TBD To be defined - We aren't certain about this yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants