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 #10669] Add IPv4/IPv6 support to the rest of the monitoring-plugins #3683

Closed
icinga-migration opened this issue Nov 19, 2015 · 9 comments

Comments

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

commented Nov 19, 2015

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

Created by chrostek on 2015-11-19 07:26:01 +00:00

Assignee: chrostek
Status: Resolved (closed on 2016-02-23 10:39:02 +00:00)
Target Version: 2.5.0
Last Update: 2016-08-22 11:47:43 +00:00 (in Redmine)

Backport?: No
Include in Changelog: 1

Refs #10513

Attachments

Changesets

2015-11-30 07:36:36 +00:00 by chrostek ab9218e

Add ipv4 / ipv6 support for all monitoring-plugins
Add some options to check_smtp/check_ssmtp/check_imap/check_simap/check_pop/check_spop

Refs #10669

Signed-off-by: Gunnar Beutner <gunnar@beutner.name>

2016-02-23 10:41:45 +00:00 by (unknown) ccaf6e3

Fix spelling mistakes in the documentation

refs #10669

Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Nov 19, 2015

Updated by chrostek on 2015-11-19 07:27:14 +00:00

  • File added 0001-Added-ipv4-ipv6-support-for-all-monitoring-plugins.patch

Please merge this patch

Added the ipv4/ipv6 support to the rest of the monitoring-plugins
Added some missing options: check_smtp / check_ssmtp / check_imap / check_simap / check_pop / check_spop

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Nov 20, 2015

Updated by gbeutner on 2015-11-20 10:29:37 +00:00

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

Wow, that's a lot of changes. :)

  • Your e-mail address ('Author') seems to be incorrect: Author: Sebastian Chrostek
  • The documentation for the _ipvX attributes should probably mention that the default value is false (i.e. to suggest that their type is 'bool'). Right now users have to guess that those attributes are boolean flags.
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Nov 20, 2015

Updated by gbeutner on 2015-11-20 10:31:30 +00:00

  • Relates set to 6225
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Nov 21, 2015

Updated by chrostek on 2015-11-21 11:09:09 +00:00

  • File added 0001-Added-ipv4-ipv6-support-for-all-monitoring-plugins-A.patch

Sorry for the wrong Author ...
Added my correct e-mail address and added "Defaults to false." to the documentation.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Nov 30, 2015

Updated by gbeutner on 2015-11-30 07:37:52 +00:00

  • Target Version set to 2.5.0

Merged with some minor modifications (replaced spaces with tabs). Thanks. :)

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Feb 23, 2016

Updated by gbeutner on 2016-02-23 10:27:22 +00:00

  • Backport? changed from Not yet backported to No
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Feb 23, 2016

Updated by gbeutner on 2016-02-23 10:39:02 +00:00

  • Status changed from Assigned to Resolved

This patch introduces a spelling mistake:

-mem_warning  | **Required.** Specify the warning threshold as number interpreted as percent.
-mem_critical | **Required.** Specify the critical threshold as number interpreted as percent.
+mem_warning  | **Required.** Specifiy the warning threshold as number interpreted as percent.
+mem_critical | **Required.** Specifiy the critical threshold as number interpreted as percent.
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Mar 17, 2016

Updated by mfriedrich on 2016-03-17 13:44:38 +00:00

  • Relates set to 11403
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Aug 22, 2016

Updated by gbeutner on 2016-08-22 11:47:43 +00:00

  • Subject changed from Add ipv4/ipv6 support to the rest of the monitoring-plugins to Add IPv4/IPv6 support to the rest of the monitoring-plugins
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.