Project

General

Profile

Bug #9152

Provisioning new oVirt VM's fails

Added by Stephen Benjamin over 7 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Category:
Compute resources - oVirt
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Provisioning fails on oVirt VM's because of:

"(oVirt) instance isolated-client.stbenjam.satellite.lab.eng.rdu2.redhat.com: VM cluster.id|name required for add " 

The form changed:

<% javascript 'compute_resource', 'lookup_keys' %>
<%= text_f f, :name if show_vm_name? %>
<% new_host = new_host?(@host) %>
<% clusters = compute_resource.clusters %>
<%= select_f f, :cluster, clusters, :id, :name,

new_host?(@host) doesn't seem to work as the cluster field is always disabled...


Related issues

Related to Foreman - Refactor #8661: fix some problematic styling issues in the vm form partials.Closed2014-12-11
Related to Foreman - Bug #9320: Can't create host using VMWare compute resourceResolved2015-02-11
Blocked by Foreman - Bug #9480: Fix multiple NICs integration with compute profilesClosed2015-02-20

History

#1 Updated by Dominic Cleal over 7 years ago

  • Related to Refactor #8661: fix some problematic styling issues in the vm form partials. added

#2 Updated by Dominic Cleal over 7 years ago

  • Legacy Backlogs Release (now unused) set to 28

new_host?(nil) now returns false, when a nil host variable used to be considered as a new host.

https://github.com/theforeman/foreman/blob/1.7.1/app/views/compute_resources_vms/form/_ovirt.html.erb#L3

#3 Updated by Dominic Cleal over 7 years ago

  • Related to Bug #9320: Can't create host using VMWare compute resource added

#4 Updated by The Foreman Bot over 7 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2150 added
  • Pull request deleted ()

#5 Updated by Tom Caspy over 7 years ago

  • Assignee set to Tom Caspy

#6 Updated by Alexandre Barth over 7 years ago

Still have VMWare compute resource issue when applying this fix

#7 Updated by Dominic Cleal over 7 years ago

Yeah, I don't know why it's claimed that this PR fixes VMware, since it doesn't touch those code paths.

#8 Updated by Dominic Cleal over 7 years ago

  • Blocked by Bug #9480: Fix multiple NICs integration with compute profiles added

#9 Updated by Dominic Cleal over 7 years ago

  • Status changed from Ready For Testing to Resolved
  • Assignee changed from Tom Caspy to Tomáš Strachota

I believe this issue has been resolved via #9480 instead, which will be available in nightly builds soon. Please re-open if you see it in nightlies or 1.8.0-RC1, thanks!

Also available in: Atom PDF