Navigation Menu

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 #13287] Error message doesn't show list (id) if a key is not found #621

Closed
icinga-migration opened this issue Nov 23, 2016 · 3 comments
Labels
Milestone

Comments

@icinga-migration
Copy link

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

Created by tobiasvdk on 2016-11-23 09:54:45 +00:00

Assignee: (none)
Status: New
Target Version: (none)
Last Update: 2016-12-08 20:17:47 +00:00 (in Redmine)


I had the following error message because the key is missing in a list map:

This Import Source failed when last checked at 2016-11-23 09:50:56: "this-is-my-key" cannot be found in the "List with id " data list

The data list id is missing. Furthermore as a user I don't want to know the list id but the list name.

Attachments

@icinga-migration
Copy link
Author

Updated by tobiasvdk on 2016-12-08 20:17:47 +00:00

  • File added 0001-Return-data-list-name.patch

Patch shows the data list name instead of the id, e.g.

This Import Source failed when last checked at 2016-12-08 20:14:25: "down" cannot be found in the "disabled_map" data list

tobiasvdk pushed a commit to tobiasvdk/icingaweb2-module-director that referenced this issue Feb 17, 2017
Return the data list name instead of the id for the function
getDatalistName().

refs Icinga#621
tobiasvdk pushed a commit to tobiasvdk/icingaweb2-module-director that referenced this issue Feb 17, 2017
Return the data list name instead of the id for the function
getDatalistName().

refs Icinga#621
Thomas-Gelf pushed a commit that referenced this issue Feb 17, 2017
Return the data list name instead of the id for the function
getDatalistName().

refs #621
@tobiasvdk
Copy link

I think this can be closed @Thomas-Gelf

@Thomas-Gelf
Copy link
Contributor

@tobiasvdk: yes, has been merged with fd53224. Thank you!

@Thomas-Gelf Thomas-Gelf added this to the 1.3.2 milestone Jul 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants