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

[dev.icinga.com #11205] Add "query" option to check_postgres command. #3958

Closed
icinga-migration opened this issue Feb 23, 2016 · 5 comments
Closed
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Feb 23, 2016

This issue has been migrated from Redmine: https://dev.icinga.com/issues/11205

Created by midn on 2016-02-23 07:52:12 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2016-02-23 21:20:03 +00:00)
Target Version: 2.4.4
Last Update: 2016-03-11 14:49:00 +00:00 (in Redmine)

Backport?: Already backported
Include in Changelog: 1

Hello,

Check_postgres Command was missing "query", "valtype" and "reverse" options for check_postgres Command/Plugin to use with "custom_query" action.
This allows to specify custom query for usage with built-in Command definition instead of having the need to build your own.

Regards,
Andres Ivanov

Changesets

2016-02-23 21:16:00 +00:00 by andres@andres.wtf 7e4a087

ITL: Add query option to check_postgres

There were "query", "valtype" and "reverse" options missing
for "custom_query" action type in check_postgres plugin.

fixes #11205

Signed-off-by: Michael Friedrich <michael.friedrich@netways.de>

2016-02-23 21:19:03 +00:00 by mfriedrich 4c59ffa

Update AUTHORS

refs #11205

2016-03-11 14:24:57 +00:00 by andres@andres.wtf 3df7336

ITL: Add query option to check_postgres

There were "query", "valtype" and "reverse" options missing
for "custom_query" action type in check_postgres plugin.

fixes #11205

Signed-off-by: Michael Friedrich <michael.friedrich@netways.de>

2016-03-11 14:25:08 +00:00 by mfriedrich e415d15

Update AUTHORS

refs #11205
@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Feb 23, 2016

Updated by midn on 2016-02-23 07:55:14 +00:00

Pull Request - #69

@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Feb 23, 2016

Updated by mfriedrich on 2016-02-23 21:18:01 +00:00

  • Status changed from New to Assigned
  • Assigned to set to mfriedrich
  • Target Version set to 2.4.3

Thanks. I've modified the author to your github's full name and email address. Please keep that in mind for future patches :)

@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Feb 23, 2016

Updated by andres@andres.wtf on 2016-02-23 21:20:03 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset 7e4a087.

@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Feb 24, 2016

Updated by mfriedrich on 2016-02-24 09:53:30 +00:00

  • Target Version changed from 2.4.3 to 2.4.4
@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Mar 11, 2016

Updated by mfriedrich on 2016-03-11 14:49:00 +00:00

  • Backport? changed from Not yet backported to Already backported
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.