Bug #19436

dropdowns on discovered host do not work

Added by Tomer Brisker about 1 year ago. Updated 7 days ago.

Status:Closed
Priority:Normal
Assignee:Tomer Brisker
Category:Host creation
Target version:1.15.0
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:https://github.com/theforeman/foreman/pull/4498

Description

fix for issue #18064 changed the url for the dropdowns from method_path (which mapped them to the discovered host controller) to host_or_hostgroup_path (which maps them to the hosts controller) leading to them not functioning correctly.


Related issues

Related to Foreman - Feature #18064: Ability to import and provision existing VMs Closed 01/13/2017
Related to Discovery - Bug #19414: Discovery host form errors out with Please try to update ... Closed 04/27/2017

Associated revisions

Revision e59bc189
Added by Tomer Brisker about 1 year ago

Fixes #19436 - Better controller selection on host dropdown (#4498)

Always setting to `host` unless the controller is `hostgroup` broke
discovery's usage of the form as it uses the `discoverd_host`
controller.
This reverts the change from `method_path` to `host_or_hostgroup_path`
from commit 637da2f28accc1378f0326d97adcbb2efc5d9808 and moves the
controller correction into `method_path`.

Revision 05d62230
Added by Tomer Brisker about 1 year ago

Fixes #19436 - Better controller selection on host dropdown (#4498)

Always setting to `host` unless the controller is `hostgroup` broke
discovery's usage of the form as it uses the `discoverd_host`
controller.
This reverts the change from `method_path` to `host_or_hostgroup_path`
from commit 637da2f28accc1378f0326d97adcbb2efc5d9808 and moves the
controller correction into `method_path`.
(cherry picked from commit e59bc18973d2ff9bd7b06b6c08ef600dc641886f)

History

#1 Updated by Tomer Brisker about 1 year ago

  • Related to Feature #18064: Ability to import and provision existing VMs added

#2 Updated by Tomer Brisker about 1 year ago

  • Related to Bug #19414: Discovery host form errors out with Please try to update your request added

#3 Updated by The Foreman Bot about 1 year ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4498 added

#4 Updated by Timo Goebel about 1 year ago

  • Legacy Backlogs Release (now unused) set to 209

#5 Updated by Anonymous about 1 year ago

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

Also available in: Atom PDF