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 #8979] Missing DEL_DOWNTIME_BY_HOST_NAME command required by Classic UI 1.x #2867

Closed
icinga-migration opened this issue Apr 2, 2015 · 8 comments
Labels
bug
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Apr 2, 2015

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

Created by vaisov on 2015-04-02 10:52:10 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2015-07-31 12:55:03 +00:00)
Target Version: 2.3.9
Last Update: 2015-07-31 13:06:12 +00:00 (in Redmine)

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

While trying to remove downtimes for all services on selected host I get the error in log:

warning/ExternalCommandListener: External command failed.Error: The external command 'DEL_DOWNTIME_BY_HOST_NAME' does not exist.

Changesets

2015-07-31 12:50:17 +00:00 by (unknown) 766e2d7

Implement external command DEL_DOWNTIME_BY_HOST_NAME

Required by Classic UI 1.x, this will hopefully reduce
the noise on non-working commands with the old legacy stuff.

fixes #8979

2015-07-31 12:59:34 +00:00 by (unknown) 1e490dc

Implement external command DEL_DOWNTIME_BY_HOST_NAME

Required by Classic UI 1.x, this will hopefully reduce
the noise on non-working commands with the old legacy stuff.

fixes #8979

Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 12, 2015

Updated by mfriedrich on 2015-04-12 13:27:22 +00:00

  • Relates set to 3165
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 12, 2015

Updated by mfriedrich on 2015-04-12 13:28:12 +00:00

Requires optional arguments detection which is not implemented in the external command processor at this time.

DEL_DOWNTIME_BY_HOST_NAME;[;[;[;]]]
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 12, 2015

Updated by mfriedrich on 2015-04-12 13:28:43 +00:00

  • Relates set to 7074
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 17, 2015

Updated by mfriedrich on 2015-04-17 15:46:28 +00:00

  • Category set to libicinga
  • Status changed from New to Rejected

I am sorry, but we won't implement that check command for Icinga 2. 2.4 shall provide a general overhauled api and long term, Icinga Web 2 will replace Classic UI as well.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jul 31, 2015

Updated by mfriedrich on 2015-07-31 12:49:34 +00:00

  • Subject changed from ClassicUI doesn't allow to remove downtime for services anymore to Missing DEL_DOWNTIME_BY_HOST_NAME command required by Classic UI 1.x
  • Status changed from Rejected to Assigned
  • Assigned to set to mfriedrich
  • Target Version set to 2.4.0
  • Estimated Hours set to 2

Re-opening this by customer request, will push a fix soon.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jul 31, 2015

Updated by Anonymous on 2015-07-31 12:55:03 +00:00

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

Applied in changeset 766e2d7.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jul 31, 2015

Updated by mfriedrich on 2015-07-31 13:06:12 +00:00

  • Target Version changed from 2.4.0 to 2.3.9
  • Backport? changed from TBD to Yes
@icinga-migration

This comment has been minimized.

Copy link
Member Author

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

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

  • Relates set to 11240
@icinga-migration icinga-migration added this to the 2.3.9 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.