Feature #15519

Speed up host editing when using vmware compute resource

Added by Alex Fisher about 1 year ago. Updated 8 months ago.

Status:Closed
Priority:Normal
Assigned To:Timo Goebel
Category:Compute resources - VMware
Target version:-
Difficulty: Bugzilla link:
Found in release:1.11.3 Pull request:https://github.com/theforeman/foreman/pull/3964
Story points-
Velocity based estimate-
Release1.14.0Release relationshipAuto

Description

Hi

Since the vmware properties are read-only after host creation, would it be possible to not automatically load them when editing a host?
Perhaps only load them after clicking on the 'virtual machine' tab? The vmware API is so slow and editing a host can be really painful.


Related issues

Related to Foreman - Tracker #18255: Speed up vmware New 01/26/2017

Associated revisions

Revision 0d851b1d
Added by Timo Goebel 8 months ago

fixes #15519 - speed up host edit page rendering on vmware

This commit speeds up the rendering of the host edit page for hosts on
VMWare. The API calls to list networks and datastores are expensive.
When editing a host, these information is never showed to a user and
furthermore CR editing on VMWare is not supported.
So instead of displaying a select box, this commit shows a textbox.

History

#1 Updated by Dominic Cleal about 1 year ago

  • Category set to Compute resources - VMware

Some option to keep showing them is probably required, people do rely on it (we get bug reports when it's broken).

#2 Updated by Timo Goebel 8 months ago

  • Status changed from New to Assigned
  • Assigned To set to Timo Goebel

The reason for this is that in the host edit views select boxes are rendered with all available volumes or networks. This is a very expensive call with VMWare. In addition, this is completely unnecessary as they are disabled the whole time.

Affected views:

app/views/compute_resources_vms/form/vmware/_network.html.erb
app/views/compute_resources_vms/form/vmware/_volume.html.erb

What they need:
<% if new_host >
<
end %>

#3 Updated by The Foreman Bot 8 months ago

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

#4 Updated by Timo Goebel 8 months ago

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

#5 Updated by Dominic Cleal 8 months ago

  • Release set to 1.14.0

#6 Updated by Timo Goebel 5 months ago

Also available in: Atom PDF