Project

General

Profile

« Previous | Next » 

Revision 8c678130

Added by Tomáš Strachota about 9 years ago

Fixes #9480 - multiple NICs integration with compute profiles

From users point of view:
- interfaces setup in compute profiles is back
- interface related compute attrs get merged into host NICs upon compute
profile selection
- NIC overview table displays details in the column "Type"
- UI for direct creation of VMs fixed
- fixed removing interfaces and volumes from compute profiles

From technical point of view:
- NIC overview table is always built by js
- templates for compute resources got more structure (split to partials
"base", "interface", "volume")
- compute resources can provide custom implementation of
'providerSpecificNICInfo' function

(cherry picked from commit 6d05514ad02676067389f44c4ba1d3daf357884b)

View differences:

app/views/compute_resources_vms/form/vmware/_volume.html.erb
<div class="fields">
<%= selectable_f f, :datastore, vsphere_datastores(compute_resource), { }, :class => "span5 disable-unsupported", :label => _("Data store") %>
<%= text_f f, :name, :class => "col-md-2 disable-unsupported", :label => _("Name") %>
<%= text_f f, :size_gb,
:class => "col-md-2 disable-unsupported",
:label => _("Size (GB)") %>
<%= checkbox_f f, :thin, {
:label => _("Thin provision"),
:class => 'disable-unsupported'},
"true",
"false" %>
<%= checkbox_f f, :eager_zero, {
:label => _("Eager zero"),
:class => 'disable-unsupported',
:help_inline => remove_child_link("X", f, { :method => :'_delete', :title => _('remove volume'), :class => 'label label-danger disable-unsupported' })},
"true",
"false" %>
</div>
<%= selectable_f f, :datastore, vsphere_datastores(compute_resource), { }, :class => "span5", :label => _("Data store") %>
<%= text_f f, :name, :class => "col-md-2", :label => _("Name") %>
<%= text_f f, :size_gb,
:class => "col-md-2",
:label => _("Size (GB)") %>
<%= checkbox_f f, :thin, {
:label => _("Thin provision")},
"true",
"false" %>
<%= checkbox_f f, :eager_zero, {
:label => _("Eager zero")},
"true",
"false" %>

Also available in: Unified diff