summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | | bug 929428: pep8 validation on all xapi pluginsArmando Migliaccio2012-02-091-2/+4
| | |_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | check every python file under <nova_root>/plugins/xenserver/. Ignore patch files. Change-Id: Ib1170ddabe03de746aae570d30b133aaffb09c88
* | | | | | | | | | | Merge "Pass instance to log messages."Jenkins2012-02-141-24/+23
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Pass instance to log messages.Michael Still2012-02-102-25/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch converts more log messages to passing their instance. Having the instance in a consistant format in the logs makes it easier for operations staff to debug instance problems. Change-Id: I842f26309e17c0923e71fbfb00bdcdb033f92297
* | | | | | | | | | | | Merge "fix FlatNetworkTestCase.test_get_instance_nw_info"Jenkins2012-02-131-9/+12
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | fix FlatNetworkTestCase.test_get_instance_nw_infoPádraig Brady2012-02-101-9/+12
| | |_|_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most of this function was bypassed and the bypassed portion contained syntax errors. So reneable the skipped branch and adjust to pass. Change-Id: I364ccfbe5be5bc050ba0004133f353beed97fae7
* | | | | | | | | | | | Merge "remove a private duplicate function"Jenkins2012-02-131-19/+10
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | remove a private duplicate functionPádraig Brady2012-02-131-19/+10
| | |_|_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * nova/network/ldapdns.py (utf-8): This function is already available in nova.utils. Note this duplicate was catching impossible to trigger exceptions, and was referencing an undefined variable 'val' Change-Id: I0e47f8b976a3326c99927990f91be473c0a85f78
* | | | | | | | | | | | Merge "remove unused and buggy function from S3ImageService"Jenkins2012-02-131-4/+0
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | remove unused and buggy function from S3ImageServicePádraig Brady2012-02-111-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * nova/image/s3.py (S3ImageService.get): Remove Change-Id: Ibe4526c821b9d05025956d49a6cb840d11ce013a
* | | | | | | | | | | | | Merge "Use stubs in libvirt/utils get_fs_info test"Jenkins2012-02-131-11/+28
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Use stubs in libvirt/utils get_fs_info testMark Washenberger2012-02-131-11/+28
| | |_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This test frequently breaks for me because of a race condition where some small change will happen to the file system between when df is called and when os.statvfs is called. By refactoring the test to stubs, we can eliminate the race condition. Note that I do not modify the code under test at all in this change, so the risk of introducing a defect is zero. Change-Id: I83e4b38b25fef7e5e27fcf81b5e5a566406b05dd
* | | | | | | | | | | | Merge "Adding (-x | --stop) option back to runner.py"Jenkins2012-02-131-0/+1
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Adding (-x | --stop) option back to runner.pyAlex Meade2012-02-131-0/+1
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug 931608 Change-Id: I355a3bba38367c820a0eddc006ef480e5134b317
* | | | | | | | | | | | Merge "dont show blank endpoint headers"Jenkins2012-02-133-16/+35
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | dont show blank endpoint headersNikhil Komawar2012-02-073-16/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug: 741972 Change-Id: I4a661a13c5baaa79ee647a6c1cedcef29a2642f6
* | | | | | | | | | | | Merge "Remove duplicate variable"Jenkins2012-02-131-2/+0
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Remove duplicate variableJohannes Erdfelt2012-02-131-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ie92a534c566dd411c2bd3e73031336974b1e5505
* | | | | | | | | | | | | Fixing a unicode related metadata bug.Naveed Massjouni2012-02-133-7/+3
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a server had metadata with unicode characters in the values, doing a GET on servers details would return a 500 error. This fixes that bug. bug: 929281 Change-Id: I6162532c9a5a615802eb23e7bf9a80d3faf6e7a0
* | | | | | | | | | | | Merge "Make melange_port an integer"Jenkins2012-02-131-3/+3
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | |
| * | | | | | | | | | | Make melange_port an integerJohannes Erdfelt2012-02-131-3/+3
| | |_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's nonsensical as anything but an integer and it makes it clearer what type it should be. Change-Id: Icd62023b1835d33fb977302208744657e4557d2a
* | | | | | | | | | | Merge "Log instance id consistently inside the firewall code."Jenkins2012-02-132-9/+14
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Log instance id consistently inside the firewall code.Michael Still2012-02-102-9/+14
| | |_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This brings this code inline with the way connection.py does instance logging now. Change-Id: I3a9b256cb87bc273352f4b9338ae32dbc1faa6f6
* | | | | | | | | | | Merge "Fix minor typo in runner.py"Jenkins2012-02-131-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Fix minor typo in runner.pyHengqing Hu2012-02-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #930519 Change-Id: I53f5f85d1ef8117f6a40a90ee1e9eb12a27dcd53
* | | | | | | | | | | Merge "Converting db tables to utf8."Jenkins2012-02-111-0/+61
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Converting db tables to utf8.Naveed Massjouni2012-02-101-0/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug: 929232 Change-Id: I6bea01b243e0c9dbd7e3b3559adbb73e42484a4c
* | | | | | | | | | | Merge "remove all instance_type db lookups from network"Jenkins2012-02-117-50/+29
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | remove all instance_type db lookups from networkTrey Morris2012-02-107-50/+29
| | |_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | increases efficiency without requiring extra parameters Change-Id: I858067afb94007c8e71748373873f7cc8bd54662
* | | | | | | | | | Merge "Remedies LP Bug #928910 - Use libvirt lookupByName() to check existence"Jenkins2012-02-113-1/+24
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Remedies LP Bug #928910 - Use libvirt lookupByName() to check existenceJay Pipes2012-02-103-1/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make determining if an instance exists on a host more efficient by adding an instance_exists() method to the base virt driver that can be overridden by drivers that have a more efficient mechanism of looking up an instance by its ID / name. Modifies the _check_instance_already_created method of the compute manager to use this new instance_exists() method. Someone from Citrix should look into how to make the instance_exists() method in the Xen and VMWare virt drivers more efficient than the base "loop over all domains and see if the instance ID exists" method now in the base driver class. Change-Id: Ibf219788f9c104698057367da89300a060945778
* | | | | | | | | | | Remove relative imports from scheduler/filtersJoe Gordon2012-02-101-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I576740e512c92fb633f0029f4ab738d74600d8f9
* | | | | | | | | | | Merge "Handle network api failures more gracefully"Jenkins2012-02-112-2/+20
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Handle network api failures more gracefullyMark Washenberger2012-02-102-2/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | addresses bug 930329 Change-Id: Idb0934b9dd6ebc0e5e81495fb4454e4e50bc1f57
* | | | | | | | | | | | Merge "remove unused and buggy function from baremetal proxy"Jenkins2012-02-101-5/+0
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | remove unused and buggy function from baremetal proxyPádraig Brady2012-02-101-5/+0
| | |_|_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * nova/virt/baremetal/proxy.py (_fetch_image): This function reference 'images' which is undefined. So just remove it as it's unused. Change-Id: I7016f6e8a68fefab4342adadd032e93f963a074e
* | | | | | | | | | | | Merge "Retry on network failure for melange GET requests"Jenkins2012-02-102-13/+28
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Retry on network failure for melange GET requestsJohannes Erdfelt2012-02-102-13/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 929041 The melange service may need to periodically be restarted for various operations reasons, so retry GET requests if we receive a socket error. Change-Id: I010650dac7793d0041d41be067eb4b32e2171c8a
* | | | | | | | | | | | Merge "Automatic confirmation of resizes on libvirt"Jenkins2012-02-102-2/+28
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Automatic confirmation of resizes on libvirtPhilip Knouff2012-02-102-2/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug #911217 Change-Id: I135002a3010416e8c63b3dce303f4e4d47c7c6d3
* | | | | | | | | | | | | Merge "linux_net: Also ignore shell error 2 from ip addr"Jenkins2012-02-101-7/+7
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | linux_net: Also ignore shell error 2 from ip addrAdam Gandelman2012-02-081-7/+7
| | |_|_|_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ignores error code 2 from 'ip addr {add, del}' which, in recent versions of iproute2, means the address has already been removed or added. Fixes bug 929127 Change-Id: I09274454847ff834ac3da0e38022b30a7a7f1676
* | | | | | | | | | | | | Merge "Fix exception by passing timeout as None"Jenkins2012-02-101-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Fix exception by passing timeout as NoneJohannes Erdfelt2012-02-101-1/+1
| | |/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The call method takes a new timeout argument, but not all uses of it were updated to pass the new argument. Change-Id: Ie4377419aafa5606ade803f8e4cd715cb53b1547
* | | | | | | | | | | | | Merge "Force imageRef to be a string"Jenkins2012-02-102-1/+22
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Force imageRef to be a stringJason Kölker2012-02-102-1/+22
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes lp930409 Change-Id: Iff07a9b70210a43eb9442222e4d6cac60dd8f375
* | | | | | | | | | | | Merge "Removes constraints from instance and volume types"Jenkins2012-02-1016-249/+278
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Removes constraints from instance and volume typesVishvananda Ishaya2012-02-0816-249/+278
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Gets rid of annoying purge semantics * removes unique constraints from the db * deletes extra specs when a volume is deleted * adds exceptions for when the type already exists * fixes bug 854930 * fixes bug 925823 Change-Id: I4618759e31501b2e85325f4e9b9895f04dc151d0
* | | | | | | | | | | | | Merge "Ensures that hostId's are unique"Jenkins2012-02-102-1/+25
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Ensures that hostId's are uniquePhilip Knouff2012-02-092-1/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug #928015 Change-Id: I26e1e036ee56e0ee4344f9254df4c7024e0ceddf
* | | | | | | | | | | | | | Merge "Remove unused compute_service from images controller"Jenkins2012-02-101-3/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \