Bug #15769

Discovery still uses "short" name for Organizations and Locations

Added by Chris Edester 10 months ago. Updated 10 months ago.

Status:Closed
Priority:Normal
Assigned To:Lukas Zapletal
Category:Discovery plugin
Target version:Plugin 6.0
Difficulty:trivial Pull request:https://github.com/theforeman/foreman_discovery/pull/287
Bugzilla link:
Story points-
Velocity based estimate-

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 10 months ago

Fixes #15769 - discovery nested taxonomy fixed

History

#1 Updated by Dominic Cleal 10 months ago

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

#2 Updated by Lukas Zapletal 10 months ago

  • Assigned To set to Lukas Zapletal
  • Target version set to Plugin 6.0

Thanks for report, what Dominic said.

#3 Updated by The Foreman Bot 10 months ago

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

#4 Updated by Anonymous 10 months ago

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

Also available in: Atom PDF