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 #12314] Add command definition for check_mysql_query #4437

Closed
icinga-migration opened this issue Aug 4, 2016 · 7 comments

Comments

Projects
None yet
1 participant
@icinga-migration
Copy link
Member

commented Aug 4, 2016

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

Created by birkc on 2016-08-04 13:56:25 +00:00

Assignee: birkc
Status: Resolved (closed on 2016-08-08 13:20:05 +00:00)
Target Version: 2.5.0
Last Update: 2016-08-22 11:59:23 +00:00 (in Redmine)

Backport?: Not yet backported
Include in Changelog: 1

Add" check_mysql_query" to the itl CheckCommand library.
This command is present in the nagios plugins and the monitoring plugins.

I'm preparing a patch to add this check command.

Attachments

Changesets

2016-08-08 13:16:38 +00:00 by birkc 98a9702

Add "check_mysql_query" CheckCommand definition

fixes #12314

Signed-off-by: Gunnar Beutner <gunnar.beutner@netways.de>
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 4, 2016

Updated by birkc on 2016-08-04 14:02:34 +00:00

  • File added 0001-PATCH-Add-check_mysql_query-CheckCommand-definition.patch

Patch file added. Please review it.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 5, 2016

Updated by birkc on 2016-08-05 07:12:09 +00:00

  • File added 0001-add-check_mysql_query-CheckCommand-definition.patch

New version of patch added.
Please review this one instead of the first one.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 8, 2016

Updated by gbeutner on 2016-08-08 05:46:15 +00:00

  • Status changed from New to Assigned
  • Assigned to set to birkc

By convention custom variables for check commands should be prefixed with the command's name.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 8, 2016

Updated by birkc on 2016-08-08 08:12:23 +00:00

  • File added 0001-Add-check_mysql_query-CheckCommand-definition.patch

Add new version of patch after feedback.

Please review the new patch.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 8, 2016

Updated by gbeutner on 2016-08-08 13:17:14 +00:00

  • Target Version set to 2.5.0

Fixed incorrect custom var name and merged into the master branch. Thanks.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 8, 2016

Updated by birkc on 2016-08-08 13:20:06 +00:00

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

Applied in changeset 98a9702.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 22, 2016

Updated by gbeutner on 2016-08-22 11:59:23 +00:00

  • Subject changed from Add "check_mysql_query" CheckCommand definition to Add command definition for check_mysql_query
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.