fog / fog-vsphere

Fog for vSphere
MIT License
36 stars 63 forks source link

speed up folder loading #158

Closed timogoebel closed 6 years ago

timogoebel commented 6 years ago

This uses the same technique we introduced for networks (a property collector) to load folders more efficiently. In Foreman, this reduced load times of the host create form a lot.

@chris1984: Can you review, test and merge?

timogoebel commented 6 years ago

@chris1984: Thanks. I enhanced the path detection algorithm. Do you mind to test and merge this?

chris1984 commented 6 years ago

Thanks! I will start working on testing this

chris1984 commented 6 years ago

Tests pass with Foreman and template renders correctly in compute profile/host form:

PXE:

Normal VM creation with single HDD on Local Datastore - Passed
Normal VM creation with added HDD on Local Datastore - Passed
Normal VM creation with single HDD on Storage Pod - Passed
Normal VM creation with added HDD on Storage Pod - Passed

Bootdisk:

Normal VM creation with single HDD on Local Datastore - Passed
Normal VM creation with added HDD on Local Datastore - Passed
Normal VM creation with single HDD on Storage Pod - Passed
Normal VM creation with added HDD on Storage Pod - Passed

Image:

Normal VM creation with single HDD on Local Datastore - Passed
Normal VM creation with added HDD on Local Datastore - Passed
Normal VM creation with single HDD on Storage Pod - Passed

Network:

Normal VM creation with NIC in portgroup on Standard Switch - Passed
Normal VM creation with NIC in portgroup on Distributed Switch - Passed
Normal VM creation with NIC in portgroup on Distributed Switch with VLAN - Passed