Bug #19436

dropdowns on discovered host do not work

Added by Tomer Brisker 12 months ago. Updated 12 months ago.

Status:Closed
Priority:Normal
Assigned To:Tomer Brisker
Category:Host creation
Target version:-
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/foreman/pull/4498
Story points-
Velocity based estimate-
Release1.15.0Release relationshipAuto

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

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

#2 Updated by Tomer Brisker 12 months ago

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

#3 Updated by The Foreman Bot 12 months ago

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

#4 Updated by Timo Goebel 12 months ago

  • Release set to 1.15.0

#5 Updated by Anonymous 12 months ago

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

Also available in: Atom PDF