Bug #13322

Impossible to select OS and media on discovered hosts

Added by Daniel Lobato Garcia about 2 years ago. Updated almost 2 years ago.

Status:Closed
Priority:Normal
Assigned To:Daniel Lobato Garcia
Category:Discovery plugin
Target version:-
Difficulty: Pull request:https://github.com/theforeman/foreman_discovery/pull/241
Bugzilla link:1313673
Story points-
Velocity based estimate-

Description

config/routes.rb matches hosts#architecture_selected and other routes as GET. These routes are POST in Foreman core, so when javascript contacts 'GET architecture_selected_discovered_hosts', Foreman 404s.
The solution is simple, just actually match the requests as POST. I am guessing this worked before Rails 4 because it used the method of the matched route.

I'd say this one is important as currently provisioning a discovered system isn't possible unless you select a hostgroup with the OS/media/etc.


Related issues

Related to Discovery - Bug #16750: *_selected methods expect params[:host] to exist Closed 09/30/2016

Associated revisions

Revision 4c9ed472
Added by Daniel Lobato Garcia almost 2 years ago

Fixes #13322 - Impossible to select OS and media on discovered hosts

config/routes.rb matches hosts#architecture_selected and other routes as
GET. These routes are POST in Foreman core, so when javascript contacts
'GET architecture_selected_discovered_hosts', Foreman 404s.
The solution is simple, just actually match the requests as POST. I am
guessing this worked before Rails 4 because it used the method of the
matched route.

I'd say this one is important as currently provisioning a discovered
system isn't possible unless you select a hostgroup with the
OS/media/etc.

History

#1 Updated by The Foreman Bot about 2 years ago

  • Status changed from New to Ready For Testing
  • Assigned To set to Daniel Lobato Garcia
  • Pull request https://github.com/theforeman/foreman_discovery/pull/241 added

#2 Updated by Daniel Lobato Garcia almost 2 years ago

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

#3 Updated by Lukas Zapletal almost 2 years ago

  • Bugzilla link set to 1313673

#4 Updated by Lukas Zapletal 3 months ago

  • Related to Bug #16750: *_selected methods expect params[:host] to exist added

Also available in: Atom PDF