summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | Fix VPN ping packet length.Russell Bryant2012-01-311-1/+1
| | |_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug 906346. This patch addresses the typo pointed out in bug 906346. The ping being sent was 15 bytes long when it should have been 14. Removing a pad byte from the format string resolves this issue. I verified that the format described in the code comments (and now the code) was correct using an OpenVPN connection setup packet capture. Change-Id: Idbc5e48ede4a8d2836dd1b102a9a0e172540776c
* | | | | | | | | Merge "bug 921087: i18n-key and local-storage hard-coded in xenapi"Jenkins2012-02-013-40/+135
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | bug 921087: i18n-key and local-storage hard-coded in xenapiArmando Migliaccio2012-02-013-40/+135
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fix introduces a new flag 'sr_matching_filter', whose default value is 'other-config:i18n-key=local-storage'. This filter is used for finding the SR on which to install guest instances. The default value is the Local Storage in default XenServer/XCP installations, and it is what was hard-coded so far. To select an SR with a different matching criteria, this flag can be set to 'other-config:my_favorite_sr=true'. On the other hand, to fall back on the Default SR, as displayed by XenCenter and as returned by xenapi.pool.get_default_SR, this flag can be set to to 'default-sr:true'. This changeset also makes a small code simplification along the way. Change-Id: Ia5ee438389c59a5ef0b858e8548643d57ef16c77
* | | | | | | | | | Merge "Empty connection pool after test_kombu."Jenkins2012-02-011-0/+1
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Empty connection pool after test_kombu.Russell Bryant2012-01-311-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Empty the connection pool after the tests in test_kombu have run. This ensures that there is nothing leftover that will affect tests of other implementations that use nova.rpc.amqp (impl_qpid right now). Change-Id: Ib023722f97a21821869369d0024cf61272267ea3
* | | | | | | | | | | Merge "Use name filter in GlanceImageService show_by_name"Jenkins2012-02-012-11/+5
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / |/| | | | | | | | | |
| * | | | | | | | | | Use name filter in GlanceImageService show_by_nameBrian Waldon2012-01-282-11/+5
| | |_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 883289 Change-Id: Ie2e62aea55e6541dc4ad1a725130fbf0259362fb
* | | | | | | | | | Merge "Clear out RPC connection pool before exit."Jenkins2012-02-019-0/+43
|\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| | | | | | / / | |_|_|_|_|_|_|/ / |/| | | | | | | |
| * | | | | | | | Clear out RPC connection pool before exit.Russell Bryant2012-01-319-0/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 767984. This patch ensures that pooled connections to a messaging system get cleaned up before a process that has used the RPC API exits. Change-Id: I56eca54334075378534a7a5d3434c420319672b4
* | | | | | | | | Merge "lockfile.FileLock already appends .lock"Jenkins2012-02-011-1/+1
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | lockfile.FileLock already appends .lockJohannes Erdfelt2012-02-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise you end up with files like nova-iptables.lock.lock Change-Id: Ia7dd82c3721f89d438c1266e7ec3535a0d89334e
* | | | | | | | | | Merge "Ties quantum, melange, and nova network model"Jenkins2012-02-0129-768/+873
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
| * | | | | | | | | Ties quantum, melange, and nova network modelTrey Morris2012-02-0129-768/+873
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | get_instance_nw_info() now returns network model, and keeps the network info cache up to date. virt shim and translation in place for virts to get at the old stuff Change-Id: I070ea7d8564af6c644059d1c209542d250d19ddb
* | | | | | | | | | Merge "Instances to be created with a bookmark link"Jenkins2012-02-012-2/+45
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Instances to be created with a bookmark linkAlex Meade2012-01-272-2/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 913545 Change-Id: I6401a631211d76a8942d3435fb7dd463630cac7a
* | | | | | | | | | | Merge "optimize libvirt raw image handling. Bug 924970"Jenkins2012-02-014-25/+51
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | optimize libvirt raw image handling. Bug 924970Pádraig Brady2012-02-014-25/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tests were seen to time-out on libvirt when raw images were used, which was due to large disk images being copied around inefficiently. A system with standard disks was seen to take an extra 60s/10G which was a problem with large root and ephemeral disks. The changes below attempt to minimize the I/O in dealing with cached raw images. These changes should also help to minimize the disk space used for such images, by avoiding the naïve copy which undoes the sparseness. * nova/virt/libvirt/connection.py (_cache_image): Do the resize here, rather than in _fetch_image(), so that we can control when the resizing is done, to minimize the amount of data that needs to be copied. Also if we're generating rather than fetching the image, then just generate in the instance dir too, as this should be faster. * nova/tests/fake_libvirt_utils.py: Remove the resize functionality since it's no longer used. * nova/tests/test_libvirt.py: Likewise. * nova/virt/libvirt/utils.py (fetch_image): Likewise. (copy_image): Shell out to cp since it deals better with sparse files. Note the above changes avoid sparse copies, so this is just an ancillary improvement in the area. Change-Id: I678d125c61aab56c62c668559eb2220d56702952
* | | | | | | | | | | Merge "remove unsupported ec2 extensions"Jenkins2012-02-0110-1226/+10
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | remove unsupported ec2 extensionsJesse Andrews2012-01-3110-1226/+10
| | |_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | implements blueprint remove-ec2-extensions * remove deprecated unofficial ec2 admin api * remove admin api from wsgi/paste * remove unofficial ec2 call for ajaxterm / update calls * remove displayName/displayDescription from volumes/instances Change-Id: If5a5ae26ebb9456a3c7376ff4ff0aa9589be1f5b
* | | | | | | | | | | Merge "Make sure multiple calls to _get_session() aren't nested"Jenkins2012-02-011-1/+5
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Make sure multiple calls to _get_session() aren't nestedJohannes Erdfelt2012-02-011-1/+5
| | |_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 924918 async_call_plugin() acquires a xenapi session as does the nested call to get_xenapi_host(). This can cause a deadlock if multiple greenthreads all block waiting for the outer sessions to be freed to allocate the inner session. This change moves the call to get_xenapi_host() to outside the with statement to ensure calls to _get_session() aren't nested. Change-Id: I8f5490f40a9ccaf74a276187f66519a5d5f52b2e
* | | | | | | | | | | Merge "Correct checking existence of security group rule"Jenkins2012-02-013-25/+45
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Correct checking existence of security group ruleMotoKen2012-02-013-25/+45
| | |/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #900031 Change-Id: I4194610ce53d1c74bd99b6878339da6e0b6a3a73
* | | | | | | | | | | Merge "Add mkswap to rootwrap"Jenkins2012-02-011-0/+4
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Add mkswap to rootwrapMark Washenberger2012-01-311-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia70a3acf9f8ca6b8c552f208767a92f321b912a5
* | | | | | | | | | | Merge "Fix broken devref docs."Jenkins2012-02-011-1/+1
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Fix broken devref docs.Russell Bryant2012-01-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug 888102. This patch fixes the documentation generated here: http://nova.openstack.org/devref/index.html There was a typo in a script that resulted in the module names being invalid and no docs being generated. This one character patch fixes it up and makes the docs happy. Change-Id: I1986e8b468334a1a33264aaec740c4f535a7c30b
* | | | | | | | | | | | Merge "fixed bug 920856"Jenkins2012-02-013-4/+5
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | fixed bug 920856Thorsten Tarrach2012-02-013-4/+5
| | |_|_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changed the responses to match that of Amazon EC2 Changed test to expect that response Change-Id: Ia03a0ea663614944067c0a0193058ba91e149ad2
* | | | | | | | | | | | Merge "Expand policies for admin_actions extension"Jenkins2012-02-014-34/+38
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Expand policies for admin_actions extensionBrian Waldon2012-01-314-34/+38
| | |_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 924417 Change-Id: Ibf62e8e824753dff43e0e86cb9d320086c2c753b
* | | | | | | | | | | Merge "Use "display_name" in "nova-manage vm list"."Jenkins2012-02-011-1/+1
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Use "display_name" in "nova-manage vm list".Russell Bryant2012-01-311-1/+1
| | |_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 894807. This patch changes the output of "nova-manage vm list" to use the display_name for an instance instead of the hostname in the output. This makes the output consistent with what you get with "nova list". Change-Id: I3131c5aad8770481bac7731c06110a98b4cc2966
* | | | | | | | | | | Boto 2.2.x failes. Capping pip-requires at 2.1.1Brian Lamar2012-02-011-1/+1
| |/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 924933 Change-Id: I808daedf9a860a8ef153785693cbc49a0a25f49c
* | | | | | | | | | Merge "Allow for auditing of API calls."Jenkins2012-01-311-0/+12
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Allow for auditing of API calls.Kevin L. Mitchell2012-01-311-0/+12
| | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Splits Resource.__call__() into two parts to allow for easy auditing of the API stack--checking execution times, etc. No behavior change or execution time impact should be expected. Change-Id: If9629e8b85f490c66499c42d6f6977a09d641c22
* | | | | | | | | | Use single call in ExtendedStatus extensionBrian Waldon2012-01-312-28/+57
| |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 917400 Change-Id: I8bc78f5e5f03c1ea30e7ff23236774af48a2c059
* | | | | | | | | Merge "Use os.path.basename() instead of string splitting."Jenkins2012-01-311-1/+1
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Use os.path.basename() instead of string splitting.Russell Bryant2012-01-311-1/+1
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 886268. Change-Id: I16e05f4ead55c75f29217072d106421fdc7de201
* / | | | | | | | Remove utils.runthis().Russell Bryant2012-01-312-6/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 884825. The bug was pointing out some unused variables noticed by pylint. utils.runthis() was only used in one place and doesn't really provide much value over utils.execute(), so just remove it. Change-Id: I3dcc34bd840a128418b58a8a66acb62adf3fc9cc
* | | | | | | | Merge "Be more explicit about emptying connection pool."Jenkins2012-01-311-2/+2
|\ \ \ \ \ \ \ \
| * | | | | | | | Be more explicit about emptying connection pool.Russell Bryant2012-01-311-2/+2
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch makes some code in test_qpid be more explicit about emptying the connection pool. It now ensures that it removes exactly how many connections are in the pool. The previous code made a couple of assumptions. First, it assumed that only one connection was in the pool (which is true, but it's still nice not to make the assumption here in the cleanup code). Second, it assumed that free() returned the number of connections that have been placed in the pool. This is not correct. The result also includes the number of connections that could be created based on the max size of the pool. Use the free_items attribute instead, which gives the exact number of connections that have been put() in the pool. Change-Id: I97378919c2d3e68f224862f07a75529575647163
* | | | | | | | Merge "bug 923798: On XenServer the DomU firewall driver fails with ↵Jenkins2012-01-313-55/+50
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | NotImplementedError"
| * | | | | | | | bug 923798: On XenServer the DomU firewall driver fails with NotImplementedErrorArmando Migliaccio2012-01-313-55/+50
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move _provider_rules in the base class. This should be enough to deal with most distros on which a domU is based on. Change-Id: I738116c5f330c2493c62d8f1f3da39abb3c3ad11
* | | | | | | | Merge "fixes melange ipam lib"Jenkins2012-01-311-3/+3
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | |
| * | | | | | | fixes melange ipam libTrey Morris2012-01-311-3/+3
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Melange can have multiple ip_blocks assigned to a network. However the ipam lib assumes that each block is its own L2 network and assigns ips from each ip_block for every instance. From internal bug. Change-Id: I72699f178cfeb214b2911200a7e9ed9b255252f1
* / / / / / / Return instancesSet in TerminateInstances ec2 apiHengqing Hu2012-01-312-20/+140
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixe bug #862680 Return instancesSet to follow ec2 TerminateInstances api specification: 'http://docs.amazonwebservices.com/AmazonEC2/dg/2007-01-03/ApiReference-SOAP-TerminateInstances.html' Remove dead test_terminate_instances code in test_cloud.py Change-Id: Iabc0ccdbd80946b60af63bdc49bc7820c65ca006
* | | / / / Implementation of new Nova Volume driver for SolidFire ISCSI SANjohn-griffith2012-01-304-3/+456
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Adds new SolidFire driver that subclasses nova.volume.san.SanISCSIDriver * Adds unit tests for new driver * Adds new exception subclasses in nova.exception * Adds John Griffith to Authors Implements solidfire-san-iscsidriver Change-Id: I4dc7508ba08f5333cde74d4cfeaae3939c5d2b02
* | | | | Merge "Handle kepair delete when not found"Jenkins2012-01-302-10/+19
|\ \ \ \ \
| * | | | | Handle kepair delete when not foundBrian Waldon2012-01-302-10/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 900924 Change-Id: Icd0a1f69c0fb7c15110bbf0c416e38cdc1147f4d