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 #11240] DEL_DOWNTIME_BY_HOST_NAME does not accept optional arguments #3981

Closed
icinga-migration opened this issue Feb 26, 2016 · 6 comments
Labels
bug
Milestone

Comments

@icinga-migration
Copy link
Member

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

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

Created by julianbrost on 2016-02-26 13:40:18 +00:00

Assignee: julianbrost
Status: Resolved (closed on 2016-03-05 17:45:03 +00:00)
Target Version: 2.4.4
Last Update: 2016-03-15 13:37:31 +00:00 (in Redmine)

Icinga Version: 2.4.3
Backport?: Already backported
Include in Changelog: 1

The external command DEL_DOWNTIME_BY_HOST_NAME is supposed to support the optional parameters service name, start time and comment. Currently this does not work and when submitting a command that has those, it results in this error: warning/ExternalCommandListener: External command failed.Error: Cannot schedule host services downtime for non-existent host 'somehost;;;test'

I've attached a patch that fixes this problem.

Attachments

Changesets

2016-03-05 17:43:02 +00:00 by (unknown) 7439633

Fix max number of args for DEL_DOWNTIME_BY_HOST_NAME

The RegisterCommand call for DEL_DOWNTIME_BY_HOST_NAME was missing the
max args parameter. This prevented the optional service name, start time
and comment arguments to be passed.

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

fixes #11240

2016-03-07 08:09:01 +00:00 by (unknown) 554531a

Update AUTHORS

refs #11240

2016-03-11 14:57:35 +00:00 by (unknown) ca86971

Fix max number of args for DEL_DOWNTIME_BY_HOST_NAME

The RegisterCommand call for DEL_DOWNTIME_BY_HOST_NAME was missing the
max args parameter. This prevented the optional service name, start time
and comment arguments to be passed.

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

fixes #11240

2016-03-11 14:57:43 +00:00 by (unknown) cc562e2

Update AUTHORS

refs #11240

Relations:

@icinga-migration
Copy link
Member Author

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

Updated by mfriedrich on 2016-02-26 13:46:18 +00:00

  • Relates set to 8979
@icinga-migration
Copy link
Member Author

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

Updated by mfriedrich on 2016-03-05 17:43:13 +00:00

  • Status changed from New to Assigned
  • Assigned to set to julianbrost
  • Target Version set to 2.5.0
@icinga-migration
Copy link
Member Author

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

Updated by Anonymous on 2016-03-05 17:45:03 +00:00

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

Applied in changeset 7439633.

@icinga-migration
Copy link
Member Author

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

Updated by mfriedrich on 2016-03-11 13:26:59 +00:00

  • Target Version changed from 2.5.0 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:57:50 +00:00

  • Backport? changed from Not yet backported to Already backported
@icinga-migration
Copy link
Member Author

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

Updated by gbeutner on 2016-03-15 13:37:31 +00:00

  • Subject changed from DEL_DOWNTIME_BY_HOST_NAME not accepting optional arguments to DEL_DOWNTIME_BY_HOST_NAME does not accept optional arguments
@icinga-migration icinga-migration added this to the 2.4.4 milestone Jan 17, 2017
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.