summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | Fix XML response for return_reservation_id.Matthew Treinish2012-10-041-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The XML serializer for create uses the server template for it's response. However, when return_reservation_id is set, only a reservation_id is going to be returned. To enable the XML response to be serialized the reservation id is now returned in a server tag/object. This commit fixes bug 1061124 Change-Id: Ic74071b1d527c3f3752e296d3fd46a28a99beb76 Signed-off-by: Matthew Treinish <treinish@linux.vnet.ibm.com>
* | | | | | | | | Merge "libvirt: Fix _cleanup_resize"Jenkins2012-10-042-1/+15
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | libvirt: Fix _cleanup_resizeJian Wen2012-09-252-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Undefine domain and unplug vifs in _cleanup_resize if the instance was migrated to another host. Fixes bug 1015731 Change-Id: If6eb1ada029d462777bbd6ea113c98c3b801cf5d
* | | | | | | | | | Merge "Add api samples to Certificates extension"Jenkins2012-10-0413-0/+58
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Add api samples to Certificates extensionMauro S. M. Rodrigues2012-10-0213-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ifb61c9cd90f03225c27b5683e5f54f30c636994c
* | | | | | | | | | | Merge "Add nova-cert service to integrated_helpers"Jenkins2012-10-042-0/+114
|\| | | | | | | | | |
| * | | | | | | | | | Add nova-cert service to integrated_helpersVishvananda Ishaya2012-10-022-0/+114
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows us to do integration tests that need to use nova-cert. Note that creating a certs for every test seems a bit excessive, so this adds a fake_crypto module to return fake values. Change-Id: Ia92c1dde8b122aef096004e9aad0a14ee587bbf0
* | | | | | | | | | | Merge "powervm: remove broken instance filtering"Jenkins2012-10-042-8/+1
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / |/| | | | | | | | | |
| * | | | | | | | | | powervm: remove broken instance filteringMatt Odden2012-10-032-8/+1
| | |_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Filtering isn't necessary (nor desired in some cases) Irrelevant comment cleanup fixes bug #1061291 Change-Id: I0798ff20aea7551b1d1100e8a439c39be99291e1
* | | | | | | | | | Merge "powervm: exception handling improvements"Jenkins2012-10-042-2/+2
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | powervm: exception handling improvementsMatt Odden2012-10-032-2/+2
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PowerVMLPARInstanceNotFound should extend InstanceNotFound so nova can handle it as an InstanceNotFound when LPARs are missing Small bug fix so cleanup failing due to LPARs not existing does not cause the delete to fail out, since we had nothing to cleanup fixes bug #1061173 Change-Id: I3b26a477e55b4627f401dbb25c66d5e84ab1fa3f
* | | | | | | | | | Merge "libvirt: ignore deleted domain while get block dev"Jenkins2012-10-041-1/+1
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / |/| | | | | | | | |
| * | | | | | | | | libvirt: ignore deleted domain while get block devMotoKen2012-10-041-1/+1
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The get_all_block_devices could raise libvirtError if domain was deleted while listing instances. This will cause LibvirtISCSIVolumeDriver to fail disconnect_volume. This patch ignores the domain that could not be found to avoid this issue. Fixes bug 1060836 Change-Id: I9dc994d8ca7fc084830adb4f6f9d3ca10e0fe577
* | | | | | | | | Merge "Updated code to update attach_time of a volume while detaching"Jenkins2012-10-044-1/+6
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Updated code to update attach_time of a volume while detachingArathi2012-09-264-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While detaching volume from an instance, attach_time of the volume is not updated. This task updates volume['attach_time'] to NONE at detach_volume. Also modified the default value of XML attribute volume['attach_time'] '-' to ''(a 0 length string) to maintain consistency with existing code. Fixes bug: 1056122 Change-Id: If56ceda6caab80fb803fcf194ed1dfb18808ec3a
* | | | | | | | | | Merge "Stop network.api import on network import"Jenkins2012-10-049-22/+27
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / |/| | | | | | | | |
| * | | | | | | | | Stop network.api import on network importVishvananda Ishaya2012-10-049-22/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The nova.network.__init__.py was importing the network.api automatically at import time. This was leading to some silly workarounds (for example in nova/virt/firewall.py) involving late importing nova.network to stop circular imports. It is also causing issues with https://review.openstack.org/#/c/13007/ because quantum api needs to import a flag from network manager which is causing a circular import. This fixes the issue by moving to use the code from volume_api which only imports the api class when the method is called. It cleans up the tests that were stubbing out network.API and also removes the unneeded workarounds. Change-Id: I4c9b372d078ab4398e797335a8de5e9e1077e31f
* | | | | | | | | | Merge "Zmq register opts fix in receiver"Jenkins2012-10-041-1/+1
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Zmq register opts fix in receiverEric Windisch2012-09-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Following dprince's change to use FLAGS, rather than cfg.CONF, zmq_opts needs to be registered explicitly. Change-Id: I66bc71563435088ee9224fdd603602e35f3e3bff
* | | | | | | | | | | Merge "Add ability to download images via BitTorrent."Jenkins2012-10-046-11/+519
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Add ability to download images via BitTorrent.Rick Harris2012-10-036-11/+519
| | |_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds a new dom0 plugin which supports downloading images via BitTorrent. Torrent metadata files are assumed to be served from a webserver which is specified by the `torrent_base_url` config. Under the hood, the dom0 plugins calls out to rasterbar's libtorrent via Python bindings in order perform the initial download as well as the seeding thereafter. Implements BP xenserver-bittorrent-images Change-Id: I824720a6e3a37317080a22cd7405d2a88172c3ef
* | | | | | | | | | Merge "Remove deprecated Folsom code: config convert"Jenkins2012-10-044-442/+0
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Remove deprecated Folsom code: config convertJoe Gordon2012-10-024-442/+0
| | |_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removes config file converter: nova-manage config convert Change-Id: I146bff9ac00a4885e7054f999ae52159da3f86ef
* | | | | | | | | | Merge "ip_protocol for ec2 security groups"Jenkins2012-10-044-7/+12
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | |
| * | | | | | | | | ip_protocol for ec2 security groupsJohn Herndon2012-10-024-7/+12
| | |_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix for bug 1057196. -Enforce restrictions on IP protocol in AuthorizeSecurityGroupIngress call. This value may only be: 'tcp', 'udp', 'icmp', '6', '17' or '1' -Ensure that ip protocol returned from DescribeSecurityGroups is in lower case, in case security group was set improperly in previous versions -Update EC2APIError in nova/exception.py to remove the HTTP error code from the message. The HTTP error code is inserted in nova.api.ec2.faults.Fault. Inserting the code in the EC2APIError causes the HTTP code to appear twice in the message -Update unit tests to expect new error message when invalid protocol is entered. -Fixing pep8 issues Change-Id: Iffe0f38abde327288e6a3960a72ad6a273f78aca
* | | | | | | | | Merge "Compare lists in api samples against all matches"Jenkins2012-10-033-30/+18
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Compare lists in api samples against all matchesVishvananda Ishaya2012-10-023-30/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lists of dictionaries don't always sort the same way, so rather than attempting to sort them, just compare each item in the list against all other copies looking for a match. This change uncovered a bug in one of the api samples test where an extra item in the template was being ignored. Change-Id: I8750560a1760de7f1991563788680b2dac445aaa
* | | | | | | | | | Create util for root device path retrievalBoris Filippov2012-10-033-4/+22
| |_|_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blueprint snapshots-for-everyone Create libvirt.utils.find_disk(virt_dom) function. This function will retrieve disk path from instance configuration. Will return path both for file and device backed instance root devices. Will throw error in case, when device can't be found. Change-Id: I612a19221c6ff78079ab53a8d77295c555514c77
* | | | | | | | | Merge "Remove dependency on python-ldap for tests"Jenkins2012-10-022-16/+7
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Remove dependency on python-ldap for testsVishvananda Ishaya2012-10-022-16/+7
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have a fake ldap implementation for testing, but the tests still fail if ldap isn't installed. This modifies the usage of ldap in the LdapDNS tests to stub the module before attempting to import it. This allows us to test without having python-ldap installed. It removes the unneccessary FakeLdapDNS in favor of using the real LdapDNS with the module stubbed out. It also removes the dependency on python-ldap from test-requires. Change-Id: Ifbe09e16436afa6999dcb7385763492b0026556c
* | | | | | | | | Merge "Remove deprecated Folsom code: bandwith_poll_interval"Jenkins2012-10-021-1/+0
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
| * | | | | | | | Remove deprecated Folsom code: bandwith_poll_intervalJoe Gordon2012-10-021-1/+0
| | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove bandwidth_poll_interval's deprecated name: bandwith_poll_interval Change-Id: I7015b306994096610aaaffc03ae5246ae160cbc3
* / | | | | | | Make resource tracker uses faster DB queryJohannes Erdfelt2012-10-023-15/+19
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1060363 db.instance_get_all_by_filters() uses a regex when filtering on host, which will do a full table scan. A regex isn't needed and instead, db.instance_get_all_by_host() can be used which does not impact the performance so much. Change-Id: I812e1d5320763e5c65232351e1e21285e04e6a4b
* | | | | | | Add TestCase.stub_module to make stubbing modules easierJohannes Erdfelt2012-10-023-15/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously various test cases would modify sys.modules directly, duplicating code unnecessarily. Change-Id: I09d333d6daf527e34db0262b28edcb0323cb2855
* | | | | | | Merge "Fix booting a raw image on XenServer"Jenkins2012-10-022-12/+33
|\ \ \ \ \ \ \
| * | | | | | | Fix booting a raw image on XenServerMate Lakat2012-09-252-12/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1055413. ImageType.from_string removed, as it was only used by its test. Introduced ImageType.get_role, that returns the image's role within the VM. In case of DISK_* image types, the role is always 'root'. Role should be the key when accessing the vdis dictionary. Change-Id: Ieb952a9246d06a8280510113a98b888237abcc65
* | | | | | | | Merge "Add aggregates extension to API samples test."Jenkins2012-10-0255-6/+644
|\ \ \ \ \ \ \ \
| * | | | | | | | Add aggregates extension to API samples test.Matthew Treinish2012-10-0155-6/+644
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I8f0f5b022795dc1d766e585cab8fe5f8445a3722 Signed-off-by: Matthew Treinish <treinish@linux.vnet.ibm.com>
* | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2012-10-0221-2379/+2445
|\ \ \ \ \ \ \ \
| * | | | | | | | Imported Translations from TransifexOpenStack Jenkins2012-10-0221-2379/+2445
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | Change-Id: I7d4e3134917f274fa7385115b7dc4ba4f92f36b8
* | | | | | | | Merge "remove deprecated connection_type flag"Jenkins2012-10-0212-143/+9
|\ \ \ \ \ \ \ \
| * | | | | | | | remove deprecated connection_type flagSean Dague2012-10-0112-143/+9
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1025712 connection_type was deprecated in Folsom and should now be removed in Grizzly as early as possible to shake out any fallout. This removes all references to it, changes the config sample and tests appropriately. Remove old default fixtures for flavors that specify ephemeral disks if the connection_type flag was set to 'libvirt' Change-Id: I8af831600a1931ae92c6d06c5105bd1bd81debe3
* | | | | | | | Remove outdated moduleauthor tagsJoe Gordon2012-10-015-27/+0
| |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The moduelauthor tags are used by the docs to see who wrote what. But many of them are outdated and wrong. fixes bug 1044578 Change-Id: I9cc27430c906d1418faf28e1b53f6306c62e8f98
* | | | | | | Merge "xenapi: Removing legacy swap-in-image."Jenkins2012-10-015-92/+6
|\ \ \ \ \ \ \
| * | | | | | | xenapi: Removing legacy swap-in-image.Rick Harris2012-09-265-92/+6
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch removes our legacy handling of swap in the image. Now that we're generating swap on-the-fly, this stop-gap solution can go away. Change-Id: Ied3198f77af8dabb6cfbf2ab9cfb3a4eb18e32ea
* | | | | | | Merge "Catch NotFound exception in FloatingIP add/remove"Jenkins2012-10-012-2/+43
|\ \ \ \ \ \ \
| * | | | | | | Catch NotFound exception in FloatingIP add/removeSulochan Acharya2012-09-202-2/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Catch the FloatingIPNotFoundForAddress exception for add/remove operations in floating ip extension for non existing floating ips. Fixes bug 977536 Change-Id: Ied450aa56ef7a52a015c6d78a099d445ab6258b6
* | | | | | | | Merge "Remove TestCase.assertNotRaises"Jenkins2012-10-015-70/+24
|\ \ \ \ \ \ \ \
| * | | | | | | | Remove TestCase.assertNotRaisesJohannes Erdfelt2012-10-015-70/+24
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | assertNotRaises appears to have been added to provide an easier way to assert that an exception should not be raised by a particular call in the test. However, the implementation has a couple of problems. The first is that it catches every other exception than the one specified, and silently ignores them. This had lead to one bug where a legitimate bug in the test (test_exception_with_rescheduling_enabled) was silently ignored. The call intended to be tested was called with too few arguments and the resulting TypeError was silently ignored. The second is that assertNotRaises could be called with None, which meant that no exceptions should be raised. However, this didn't do anything other than reraise the original exception. Calling assertNotRaises just adds an extra call with no value. From a design stand point, it's not clear what value this assertion provides since the test runner will already fail a test if an exception is raised. Change-Id: I64502dfa0e873f2bbd9b733a5fb63023e1b4e296
* / | | | | | | Update RPM SPEC to include new bandwidth plugin.Brian Lamar2012-10-011-0/+1
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RPM builds will fail until this line is added to include the new plugin. Fixes bug 1059772 Change-Id: I06d78cf199d4cb580003e4c220af3efc8dbe437c
generated by cgit (git 2.34.1) at 2026-02-13 03:22:54 +0000