summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | | | | | | | Merge "Add virt driver capabilities definition"Jenkins2012-10-243-0/+10
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|/ / / / / / / / / | |/| | | | | | | | | / / / / / / | |_|_|_|_|_|_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Add virt driver capabilities definitionDan Smith2012-10-233-0/+10
| | |_|_|_|_|_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Going forward with the early stages of no-db-compute, we will consistently hit cases where we need to eject some database usage from one virt driver up a few layers. Since those are not commonly used by all, some way of determining (and the higher layer) the properties of the virt driver being used will be useful. We have discussed using something like a mixin interface class and multiple inheritance. We have discussed tricks like detecting if the manage_image_cache() method has been overridden. We have discussed trying it once and catching NotImplementedError and then never trying again. Aside from the first, none of these are unsneaky enough to bite us later. This approach will provide us a way to declare such properties succinctly in the compute driver to help the higher layers know what we want them to do on our behalf. Change-Id: I74dea9322a5b4688319ebf5d9afe416e93401c58
* | | | | | | | | | | | | | | Merge "Fix bad Log statement in nova-manage"Jenkins2012-10-231-1/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Fix bad Log statement in nova-manageDavanum Srinivas2012-10-231-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove the log statement per discussion in the bug report https://bugs.launchpad.net/nova/+bug/1018431/comments/3 Fixes bug 1018431 Change-Id: Ia6fe511ae23155de8b78689639b53a0d9ed1d051
* | | | | | | | | | | | | | | | Merge "Move mkfs from libvirt.utils to utils"Jenkins2012-10-238-42/+39
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Move mkfs from libvirt.utils to utilsMate Lakat2012-10-238-42/+39
| | |_|_|_|_|_|_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to blueprint xenapi-config-drive Move mkfs function to nova.utils, so configdrive module does not depend on the libvirt module. This way, the configdrive module could be used by other virt modules. Change-Id: Ibda2aacfa0b66bc087e397df1474b5dfbe86d923
* | | | | | | | | | | | | | | | Merge "Fix nova-volume-usage-audit"Jenkins2012-10-231-1/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Fix nova-volume-usage-auditChuck Short2012-10-211-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | utils.default_cfgfile have been removed from nova/utils, so remove it here as well. Fixes LP: #1065069 Change-Id: I71f6c7850123ef0c1ef4fedf608de13b31dafb32 Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | | | | | | | | | | | | | | | Merge "Remove duplicate methods from network/rpcapi.py"Jenkins2012-10-231-8/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Remove duplicate methods from network/rpcapi.pyVishvananda Ishaya2012-10-231-8/+0
| | |_|_|/ / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ief1792f258d23f19bd4de115f7903b0b687db0e3
* | | | | | | | | | | | | | | | | Merge "Fix bug in a test for the scheduler DiskFilter."Jenkins2012-10-231-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Fix bug in a test for the scheduler DiskFilter.Andrea Rosa2012-10-231-2/+2
| | |/ / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is an error in the logic for a unit test in the Scheduler DiskFilter. We need to test the failure of the host_passes check with the "test_disk_filter_fails" test. Fixes bug 1070264. Change-Id: Ib07f47b74a6634c573dca8fb4efa2d95b2e1ed87
* | | | | | | | | | | | | | | | Merge "add port support when QuantumV2 subclass is used"Jenkins2012-10-232-2/+58
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | add port support when QuantumV2 subclass is usedMark McClain2012-10-222-2/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug 1070045 Fix the current implementation by checking if FLAGS.network_api_class is a subclass of nova.network.quantumv2.api.API. Previously, the API would not accept port references unless the network_api_class string was anything other than the QuantumV2 class. This limitation prevented subclassing. This changeset includes additional API unit tests focused on QuantumV2 and requested networks. Change-Id: I6b7b7389e127ff8b084ac379cb9aebccc102620c
* | | | | | | | | | | | | | | | | Merge "Enable custom eventlet.wsgi.server log_format"Jenkins2012-10-231-2/+14
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / / / / |/| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Enable custom eventlet.wsgi.server log_formatJoe Gordon2012-10-221-2/+14
| |/ / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * The default log format has duplicate info. (datetime etc) * Reference: http://eventlet.net/doc/modules/wsgi.html * contains information not shown elsewhere (wall_seconds, body_length) Change-Id: I57917b62c1f2cc4bba06011191dcf0b5342d7d43
* | | | | | | | | | | | | | | | Merge "Sync with latest version of openstack.common.timeutils"Jenkins2012-10-231-5/+16
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Sync with latest version of openstack.common.timeutilsYunhong, Jiang2012-10-231-5/+16
| | |/ / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changes since last sync: - Added is_newer_than function - Normalize_time() always return naive object - Use pep8 v1.3.3 Change-Id: I47c74671907a2f64a3fdd2caf857999c33033bdd
* | | | | | | | | | | | | | | | Merge "Fixes bug Snapshotting LXC instance fails"Jenkins2012-10-233-6/+204
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Fixes bug Snapshotting LXC instance failsDavid Kang2012-10-233-6/+204
| |/ / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1058273. The path of the disk for snapshotting LXC instance is debugged. Also, virt_dom.managedSave(0) and self._create_domain(domain=virt_dom) are not called for LXC instance, because they are supported by libvirt. Change-Id: Idf8275a0222b2eb289e16dd245f7721e17fce21d
* | | | | | | | | | | | | | | Merge "Update policies"Jenkins2012-10-237-530/+1063
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Update policiesKevin L. Mitchell2012-10-197-530/+1063
| | |_|_|_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merge in update openstack-common policy code. Updates Nova-specific policy glue code to eliminate deprecated openstack-common policy interfaces. Also cleans up policy code to allow for returning fine-grained policy values. As a side effect, fixes bug 1039132. Change-Id: I2951a0de3751bd2ec868e7a661070fed624e4af2
* | | | | | | | | | | | | | Merge "Fix nova-volume-usage-audit"Jenkins2012-10-231-4/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Fix nova-volume-usage-auditDavanum Srinivas2012-10-221-4/+4
| | |_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ensure visible message in nova-volume-usage-audit go through translation. Fixes LP: #1068946 Change-Id: Iaffe415b709dc0f515d32cad46c3fa851321fd31
* | | | | | | | | | | | | | Merge "Check parameter 'marker' before make request to glance"Jenkins2012-10-231-2/+7
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|/ / / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | Check parameter 'marker' before make request to glanceTianTian Gao2012-10-161-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix bug: 1065053 Since glance need parameter 'marker' to be a image-id, can not be None. check 'marker' before make request to glance. Change-Id: I7fc063ea74ae3be71e2effdc0b62432ebdd52c27
* | | | | | | | | | | | | | Merge "Add trove classifiers for PyPI"Jenkins2012-10-221-0/+10
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Add trove classifiers for PyPIDoug Hellmann2012-10-221-0/+10
| | |_|_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add classifiers so we can eventually register the project on PyPI to reserve the name, even though we won't release packages there. Change-Id: I35f4fd72de7781d3e01f1d122d5d3267544ffbd6 Signed-off-by: Doug Hellmann <doug.hellmann@dreamhost.com>
* | | | | | | | | | | | | | Merge "Add meaningful server diagnostic information to fake hypervisor"Jenkins2012-10-222-6/+34
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / |/| | | | | | | / / / / / / | | |_|_|_|_|_|/ / / / / / | |/| | | | | | | | | | |
| * | | | | | | | | | | | Add meaningful server diagnostic information to fake hypervisorMauro S. M. Rodrigues2012-10-222-6/+34
| | |/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the fake hypervisor didn't returned a fake information similar to the real output so this patch add this sample fake information. Also removed a duplicated check on the tests Change-Id: Iab72a867e6e2606beb15db17701375f3c8f61a79
* | | | | | | | | | | | Merge "Create Flavors without Optional Arguments"Jenkins2012-10-223-2/+28
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Create Flavors without Optional ArgumentsSathish Nagappan2012-10-193-2/+28
| | |_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes Bug #1056910 Allows the user to create a flavor without optional arguments. The variable, ephemeral, is not currently set to a default value. This none type value causes exceptions in subsequent code, which occurs when specifying only mandatory parameters. Change-Id: Ic440136b000dc296bef3bb6f8b0856ff5e7e4e79
* | | | | | | | | | | Merge "Deallocate network if instance is deleted in spawn"Jenkins2012-10-222-1/+26
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Deallocate network if instance is deleted in spawnJason Kölker2012-10-192-1/+26
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the instance is deleted while the virt driver is spawning, an InstanceNotFound exception is raised and caught in compute to prevent it from being rescheduled. This also deallocates the network to prevent leaking interfaces/mac/ips. * Fixes LP1068716 Change-Id: I991f5956facf7b105faea137d4a5b48897e68ae6
* | | | | | | | | | Merge "Allow local rbd user and secret_uuid configuration"Jenkins2012-10-222-3/+78
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | |
| * | | | | | | | | Allow local rbd user and secret_uuid configurationJames Page2012-10-172-3/+78
| | |_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By default, the rbd_user and rbd_secret_uuid are specified in the nova-volume/cinder configuration and passed to nova-compute when volumes are attached to instances. This change allows these values to be specified locally in nova-compute which means access control to RADOS devices in ceph can be managed independently from nova-volume/cinder with no requirement for consistent uuid's for libvirt secrets. Fixes bug 1065883. Change-Id: I9f07d040ae267bfbe8f794a5d22d327106314cc6
* | | | | | | | | Merge "handles empty dhcp_domain with hostname in metadata"Jenkins2012-10-211-3/+8
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | handles empty dhcp_domain with hostname in metadataMelanie Witt2012-10-131-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1064713. Currently hostname in metadata is created by concatenating the instance hostname with the dhcp_domain. When dhcp_domain is empty, it results in a hostname with a trailing dot. Change-Id: I52a50ff7f62c0a0dc900ddf707cc3dddce62bcb3
* | | | | | | | | | Merge "Do not call directly vmops.attach_volume"Jenkins2012-10-211-2/+2
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Do not call directly vmops.attach_volumeMate Lakat2012-10-181-2/+2
| | |_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Relates to blueprint xenapi-volume-drivers Driver already have an attach_volume function call which calls volumeops.attach_volume. This change replaces the direct volumeops.attach_volume calls with driver.attach_volume, so that driver.attach_volume will be the entry point for attaching the volume. Change-Id: I86e7c7df3d37076d7b7945f3504c7f305dce6820
* | | | | | | | | | Merge "Avoid leaking BDMs for deleted instances."Jenkins2012-10-212-0/+8
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Avoid leaking BDMs for deleted instances.Eoghan Glynn2012-10-202-0/+8
| | |_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1069099 Previously when an instance was deleted, the corresponding entries in the block device mappings table were not also marked as deleted. Change-Id: I0184eb2f02a9ed363019c585efffdfdb95f0bb43
* | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2012-10-2121-4473/+4562
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ic9f29d3a3ba9b046b868bf5986f8bf08fb9d7fe6
* | | | | | | | | | update kwargs with args in wrap_instance_faultTianTian Gao2012-10-192-0/+26
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix bug: 1067254 If argument 'instance' is in args rather than kwargs, we will get a KeyError exception which will cover up the real exception. So, we update kwargs with the values from args first. then, we can get 'instance' from kwargs easily. Change-Id: Ife89f9e9629e4b581d96289434e9ececced17915
* / / / / / / / / Remove ComputeDriver.update_host_status()Arata Notsu2012-10-187-35/+0
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Nobody calls the method. Change-Id: Ib585f769788753c64e19fe49a1f186880e1c6f49
* | | | | | | | Merge "Add version to network rpc API."Jenkins2012-10-187-195/+622
|\ \ \ \ \ \ \ \
| * | | | | | | | Add version to network rpc API.Russell Bryant2012-10-097-195/+622
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds versioning support for the nova-network rpc API. During Folsom development I added versioning to all rpc APIs with the exception of network and volume. I was holding off on those to see what the fate of that code in nova was going to be for sure. Adding it to the volume API in nova doesn't make much sense at this point (but doing it in Cinder does). Since nova-network is sticking around for a while, it seems worthwhile to add a version number to the API for any future changes while the code is still in the tree. There are plenty of things in this rpc API that could be cleaned up or improved, but I held off on doing any of those things. I would recommend the same for anyone else (unless it's a bug that needs to be fixed). I think any further improvements should be focused on Quantum integration with Nova or Quantum itself. Fixes bug 1060197. Change-Id: I63ce57657388166544202af9c44c2298ae551aea
* | | | | | | | | Merge "Use consoleauth rpcapi in nova-novncproxy."Jenkins2012-10-171-3/+3
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Use consoleauth rpcapi in nova-novncproxy.Russell Bryant2012-10-171-3/+3
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The nova-novncproxy service was using rpc.call directly when it should have been going through the rpcapi client proxy class. This patch fixes that. Fix bug 1067858. Change-Id: I626fb01ff61ff507520d528122358ef687d297b7
* / | | | | | | | Fixes syntax error in nova.tools.esx.guest_tools.pySulochan Acharya2012-10-171-1/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes syntax error in nova.tools.esx.guest_tools.py. Fixes bug1067840. Change-Id: Ib08e03a2be05f02c30e9578fa8773f1171a8b330