Bug #15769

Discovery still uses "short" name for Organizations and Locations

Added by Chris Edester almost 2 years ago. Updated almost 2 years ago.

Status:Closed
Priority:Normal
Assignee:Lukas Zapletal
Category:Discovery plugin
Target version:Discovery Plugin 6.0
Difficulty:trivial Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:https://github.com/theforeman/foreman_discovery/pull/287

Description

In Discovered Hosts => Select Action => Assign {Organization,Location}
Only the short name is listed instead of the full path "Server/SomeLocation"

Consequently when you set:
Administrator => Settings => discovery_{organization,location}
You are presented with the fill path, so they do not match when new hosts are discovered.
This effectively makes the discovery_{organization,location} settings not work unless you pick only a root item "Server".

Basically Discovery is still using the short name and not the full name, whereas the rest of foreman has moved on to the full name.
If you can point me to the code where this is determined (or I find time to do so myself) PRs will be forthcoming.

It's not that much of a thorn in my side yet... But I wanted to get it in.

Associated revisions

Revision 30a514d6
Added by Lukas Zapletal almost 2 years ago

Fixes #15769 - discovery nested taxonomy fixed

History

#1 Updated by Dominic Cleal almost 2 years ago

  • Project changed from Foreman to Discovery
  • Category changed from Plugins-Engines to Discovery plugin

#2 Updated by Lukas Zapletal almost 2 years ago

  • Assignee set to Lukas Zapletal
  • Target version set to Discovery Plugin 6.0

Thanks for report, what Dominic said.

#3 Updated by The Foreman Bot almost 2 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_discovery/pull/287 added

#4 Updated by Anonymous almost 2 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF