Project

General

Profile

Actions
Copy link

Bug #9152

closed

Provisioning new oVirt VM's fails

Added by Stephen Benjamin almost 10 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Tomáš Strachota
Category:
Compute resources - oVirt
Target version:
1.8.0
Difficulty:
Triaged:
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/2150
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

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 3 (0 open — 3 closed)

Related to Foreman - Refactor #8661: fix some problematic styling issues in the vm form partials.ClosedTom Caspy12/11/2014Actions
Related to Foreman - Bug #9320: Can't create host using VMWare compute resourceResolvedTomáš Strachota02/11/2015Actions
Blocked by Foreman - Bug #9480: Fix multiple NICs integration with compute profilesClosedTomáš Strachota02/20/2015Actions
Actions
Copy link
 #1

Updated by Dominic Cleal almost 10 years ago

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

Updated by Dominic Cleal almost 10 years ago

  • Translation missing: en.field_release 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

Actions
Copy link
 #3

Updated by Dominic Cleal almost 10 years ago

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

Updated by The Foreman Bot almost 10 years ago

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

Updated by Tom Caspy almost 10 years ago

  • Assignee set to Tom Caspy
Actions
Copy link
 #6

Updated by Alexandre Barth almost 10 years ago

Still have VMWare compute resource issue when applying this fix

Actions
Copy link
 #7

Updated by Dominic Cleal almost 10 years ago

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

Actions
Copy link
 #8

Updated by Dominic Cleal almost 10 years ago

  • Blocked by Bug #9480: Fix multiple NICs integration with compute profiles added
Actions
Copy link
 #9

Updated by Dominic Cleal almost 10 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!

Actions
Copy link

Also available in: Atom PDF