summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | Merge "handle IPv6 race condition due to hairpin mode"Jenkins2012-10-101-9/+26
|\ \ \ \ \ \
| * | | | | | handle IPv6 race condition due to hairpin modeTakashi Sogabe2012-10-091-9/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug 1011134 When using IPv6 an instance sees its own neighbour advertisement, because of the reflective property of the hairpin mode. Because of this the trigger-happy duplicate address detection in the instance's kernel deconfigures the IPv6 address on the interface, resulting in no IPv6 connectivity. Approach of this commit is to to add an nwfilter to libvirt which identifies this particular scenario and filters it. Change-Id: I28f9b49cee4b2ab6ff591fae4feee623955f845f
* | | | | | | Merge "Update kernel_id and ramdisk_id while rebuilding instance"Jenkins2012-10-102-1/+78
|\ \ \ \ \ \ \
| * | | | | | | Update kernel_id and ramdisk_id while rebuilding instanceSirisha Devineni2012-10-052-1/+78
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While rebuilding an instances if kernel_id and ramdisk_id associated with new image_ref then update instances table for instance_id with the new kernel and ramdisk. Fixes bug 1060925 Change-Id: I07dfdbed8580411adea14fd6c5be1a9569a7a021
* | | | | | | Merge "Fix reserve_block_device_name while attach volume"Jenkins2012-10-104-7/+11
|\ \ \ \ \ \ \
| * | | | | | | Fix reserve_block_device_name while attach volumeMotoKen2012-10-054-7/+11
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The block device name will not be reserved in current implementation due to missing volume_id in block_device_mapping. This will cause block_device.instance_block_mapping does not return the ebs device which should be reserved. This patch adds the volume_id to block_device_mapping while reserving block device name and changes the rpcapi to send the volume_id. Fixes bug 1062033 Change-Id: I8f08b568619fefa99850de0bee7d605749d53b0a
* | | | | | | Merge "Tell SQLite to enforce foreign keys."Jenkins2012-10-101-0/+10
|\ \ \ \ \ \ \
| * | | | | | | Tell SQLite to enforce foreign keys.David Ripton2012-10-091-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1021023. This requires SQLite version 3.6.19 or greater. Older versions of SQLite have no support for enforcing foreign keys. However, SQLite ignores unknown PRAGMA commands rather than treating them as errors, so this change should be harmless with old versions of SQLite. SQLAlchemy events require SQLAlchemy 0.7 or greater. But other code in this module already uses events, so this change should impose no new SQLAlchemy version requirements. Change-Id: Ic3eeeddbe012af65fe18c884674699cf16725d8d
* | | | | | | | Merge "Remove unneeded lines from aggregates extension API sample tests."Jenkins2012-10-101-2/+0
|\ \ \ \ \ \ \ \
| * | | | | | | | Remove unneeded lines from aggregates extension API sample tests.Matthew Treinish2012-10-021-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In change I8f0f5b022795dc1d766e585cab8fe5f8445a3722 there was an unneeded entry in the subs dictionary for 2 different tests. Change-Id: I2dd5286acbde57499a1594d2c09e38d9ba6254f7 Signed-off-by: Matthew Treinish <treinish@linux.vnet.ibm.com>
* | | | | | | | | Merge "Update tools hacking for pep8 1.2 and beyond"Jenkins2012-10-103-8/+10
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Update tools hacking for pep8 1.2 and beyondJoe Gordon2012-10-013-8/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * https://github.com/jcrocholl/pep8/commit/b9f72b16011aac981ce9e3a47fd0ffb1d3d2e085 broke tools/hacking.py * Upgrade pep8 to 1.2 and disable the following new tests: E12,E711,E712,E721,E502 Change-Id: I32a8808c6c9cccfedcc4d2a26649333aca1cd713
* | | | | | | | | | Merge "Return proper error messages while associating floating IP"Jenkins2012-10-102-1/+23
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Return proper error messages while associating floating IPSirisha Devineni2012-10-032-1/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Handled the FloatingIpNotFoundForAddress, NotAuthorized exceptions raised while associating floating IPs which are not allocated or allocated to another project. Fixes bug 1057906 Change-Id: I2d6eba9159b4e855fe615ed4dfa23c388d28c6d0
* | | | | | | | | | | Merge "Clarify trusted_filter conf options"Jenkins2012-10-101-10/+16
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Clarify trusted_filter conf optionsJoe Gordon2012-10-081-10/+16
| | |_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The trusted_filter conf options are vague with names like 'port' and 'server'. Clarified that options are for the attestation server * Prepend conf options with 'attestation' * deprecate old names for Grizzly. Change-Id: I6ad6c6731343e87e058a9f4e6bb7aa8decf1847a
* | | | | | | | | | | Merge "Add util for image conversion"Jenkins2012-10-101-2/+7
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Add util for image conversionBoris Filippov2012-10-101-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blueprint snapshots-for-everyone Add nova.virt.images.convert_image(source, dest, format) that converts image to other format using qemu-img convert Change-Id: I2c92cb31114c42717d0ddd011645dc7c661c82c6
* | | | | | | | | | | | Merge "Add util for disk type retrieval"Jenkins2012-10-104-12/+18
|\| | | | | | | | | | |
| * | | | | | | | | | | Add util for disk type retrievalBoris Filippov2012-10-104-12/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blueprint snapshots-for-everyone Add libvirt.utils.get_disk_type that retrieves format for given file. Format can be raw, qcow2, or lvm. Change-Id: I33898983cfa15db6b2654648ce32f72b6dd920cb
* | | | | | | | | | | | Fixes test_libvirtr spawn_with_network_info testNikola Dipanov2012-10-103-19/+42
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes the LibvirtConnTestCase.test_spawn_with_network_info test case that was reported as bug 1053572. The new implementation of the test makes no assumptions about the raised exceptions, and exercises the LibvirtDriver.spawn method completely. The test has the network_info faked and the libvirt connection mocked out. In addition. it also fixes the fake_network.FakeVIFDriver class used in this test to return a LibvirtConfigGuestInterface instance with fake values instead of just a dict. Also fixes some docstring inaccuracies in the nova.libvirt.vif module that may have lead to the FakeVIFDrivers wrong implementation before. Fixes bug 1053572 Change-Id: Ie5b69a2269ab099267231329eae21bd308fef835
* | | | | | | | | | | Merge "Remove cast_to_network from scheduler."Jenkins2012-10-102-42/+1
|\ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|_|/ | |/| | | | | | | | |
| * | | | | | | | | | Remove cast_to_network from scheduler.Russell Bryant2012-10-092-42/+1
| | |_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch removes some dead code from the scheduler. rpc casting to network hosts has been unused in the scheduler since commit d328ddcadb24d1b1961bd05a7676bc8f54b6776f. Change-Id: I0ba49d0e9f7e9476f84b5d389c06b7fc43cf0438
* | | | | | | | | | Merge "Remove unneeded temp variable"Jenkins2012-10-102-12/+12
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Remove unneeded temp variableJoe Gordon2012-10-092-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed in network.manager and tests.network.test_manager Change-Id: I118661457ab2be665f0aacb119d7f65752591791
* | | | | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2012-10-0921-15549/+16266
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2012-10-0921-15549/+16266
| | |_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Icc57f190f53df71160d549903a93eef4d0908add
* | | | | | | | | | Merge "powervm: add polling timeout for LPAR stop command"Jenkins2012-10-092-6/+28
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | |
| * | | | | | | | | powervm: add polling timeout for LPAR stop commandMatt Odden2012-10-082-6/+28
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need to poll the LPAR state after a stop command is sent to verify that the LPAR actually stops. Optionally, a timeout value in seconds can be specified. Also, fixes power_on and power_off calling the wrong start/stop functions bp powervm-compute-enhancements Change-Id: I9ffd0125c1968e9a656d9456ea3eb85f5db8f1ab
* | | | | | | | | Merge "Drop claim timeouts from resource tracker"Jenkins2012-10-092-84/+18
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | |
| * | | | | | | | Drop claim timeouts from resource trackerBrian Elliott2012-10-082-84/+18
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed the claim timeouts. The timeouts are basically irrelevant since the re-work done here: https://review.openstack.org/#/c/13182/ Each claim is a marker of an operation consuming resources, valid until the next invocation of the audit process update_available_resource(), which does a full reconciliation of all resource usage values from the DB. Change-Id: I5cb0dd8903d358272ec223cf73f6eb4c06371a96
* | | | | | | | Merge "Set instance host field after resource claim"Jenkins2012-10-098-31/+56
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ |/| | | | | | |
| * | | | | | | Set instance host field after resource claimBrian Elliott2012-10-088-31/+56
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Set the 'host' field on the instance after the resource tracker on the compute node has accepted the build. The field is set after resources are confirmed to be available while the COMPUTE_RESOURCES_SEMAPHORE is held. The semaphore ensures the resources usage values will be consistent even if the update_available_resource periodic task audit runs. bug 1060255 Change-Id: I92105ec14924960ac8ef7ca8c810783085314e10
* | / / / / / XenAPI should only snapshot root diskChris Behrens2012-10-094-66/+32
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change ID I3e2973 broke XenAPI migrations (resizes) as it caused snapshots of all disks (including swap and ephemeral disks, if enabled) to be sent over to the destination host. The destination only expects vhds for the root disk, named by sequence numbers. swap and ephemeral disks ended up with sequence numbers, also, messing up the re-building of the chain on the destination. Fixes bug 1064083 Change ID I3e2973 also didn't change the stub of VM.snapshot to VDI.snapshot, but tests still passed due to the new code ignoring all exceptions while snapshotting. Potential issues in snapshot_attached_here() would also be ignored due to excessive stubbing in the xenapi migration testing. This is also addressed here. Change-Id: I145030f92a75615d056a30cabcb6ca97e34a8b6c
* | | | | | Fix pep8 error in bin/nova-manageChris Behrens2012-10-081-4/+0
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Also remove unused imports. Change-Id: I6a5379ef9493fe39ba9442e809d0aeeaa97fb7b1
* | | | | Merge "Num instances scheduler filter"Jenkins2012-10-082-0/+56
|\ \ \ \ \
| * | | | | Num instances scheduler filterBrian Elliott2012-09-282-0/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow for scheduler filtering based on number of instances already assigned to that host. FLAGS.max_instances_per_host controls the maximum number of instances allowed per host. Change-Id: Id3e8973311ea5ac8db2d9e4e594a5288a9e3f94d
* | | | | | Merge "Set read_deleted='yes' for instance_id_mappings."Jenkins2012-10-071-1/+4
|\ \ \ \ \ \
| * | | | | | Set read_deleted='yes' for instance_id_mappings.Adam Gandelman2012-10-041-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since the migration that creates the instance_id_mappings does not populated the 'deleted' column, queries to this table should not limit results to 'deleted=0'. Limiting to non-deleted rows results in duplicate mappings being created for existing instance mappings after an upgrade, and throws off the entire EC2 instance ID to UUID mapping. Fixes LP: #1061166 Change-Id: I8893954fcae94a71dcc284c1b3b23b53901437eb
* | | | | | | Merge "Fix typo in policy docstring"Jenkins2012-10-051-1/+1
|\ \ \ \ \ \ \
| * | | | | | | Fix typo in policy docstringJulien Danjou2012-10-051-1/+1
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I778d7dd74f05f0db1a1e61fc64619fba7fca4c28 Signed-off-by: Julien Danjou <julien@danjou.info>
* | | | | | | Merge "Centralize sent_meta definition"Jenkins2012-10-051-4/+5
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Centralize sent_meta definitionKevin L. Mitchell2012-10-041-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In _create_image(), we create a properties dictionary to contain the properties of the image we're creating, then create a sent_meta dictionary to contain all the other image creation parameters. This simplifies the creation of sent_meta to make the logic flow a little easier to understand. Fixes bug 1061922. Change-Id: I9b25a16815f266b0d3f51aee097a0aadc51aad91
* | | | | | | Merge "Always use bdm in instance_block_mapping on Xen"Jenkins2012-10-052-13/+36
|\ \ \ \ \ \ \
| * | | | | | | Always use bdm in instance_block_mapping on XenClay Gerrard2012-10-042-13/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It doesn't seem that Xen will set the 'root_device_name' property on instances. This is causing instance_block_mapping to return early without evaluating the bdm before returning the list of device_names in use. I'm not sure why instance_block_mapping does this (optimization?) but on Xen at least it seems that it should not. Added a check for Xen compute_driver flag in instance_block_mapping to "guess" the root_device_name (similarlly to what is done in compute.utils for swap and ephemeral). fixes bug #1061944 Change-Id: If5b1a2b7377232c78f0629a3624552ecf6ceb0ee
* | | | | | | | Merge "xenapi: Refactor snapshots during resize"Jenkins2012-10-056-30/+118
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | xenapi: Refactor snapshots during resizeRenuka Apte2012-10-036-30/+118
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, we use VM.snapshot for resize, which fails if we have a volume attached to the VM, which does not support snapshots. This change uses VDI.snapshot instead, for all VDIs that are not attached by nova. Also needed for xenapi: detaching and reattaching volumes during migrations and reverting of migrations. Fixes Bug #1028092 Change-Id: I3e2973747135a9c33de194e38537620c397bb87e
* | | | | | | | Merge "Move snapshot image property inheritance"Jenkins2012-10-045-24/+43
|\ \ \ \ \ \ \ \ | | |/ / / / / / | |/| | | | | |
| * | | | | | | Move snapshot image property inheritanceKevin L. Mitchell2012-10-045-24/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The xenapi snapshotting code had support for inheriting properties from the base image of the snapshot (by way of the system metadata set on the snapshotted instance). The problem, however, came in the fact that some image properties were set in nova.compute.api, but those properties were not excluded from this inheritance logic except through a configuration option called "non_inheritable_image_properties". I had previously updated the default setting for this option to work around the bugs introduced by setting image properties in two different locations, but now it is time for the real fix. This change moves the inheritance logic into nova.compute.api:API._create_image. Note that two properties are still set through the xenapi snapshotting logic: the "os_type" and the "auto_disk_config" properties, which are presumed to be xenapi specific. The change also alters the inheritance logic to ensure that the work-around listing of image properties in non_inheritable_image_properties is no longer necessary; the default for this configuration option is updated accordingly. (Note: It will not harm anything to have these image properties still listed in non_inheritable_image_properties, so configurations that override this option do not need to be altered.) Change-Id: I3514da432cc10c75418e1de9752f60640d579136
* | | | | | | | Merge "Fix XML response for return_reservation_id."Jenkins2012-10-041-1/+8
|\ \ \ \ \ \ \ \
generated by cgit (git 2.34.1) at 2026-02-11 04:08:18 +0000