summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Implement security group quotas.Dan Prince2012-04-1910-10/+238
| |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes LP Bug #969545 Change-Id: I60d4300aa04597e2d8b0eea31ab0303b0b3c48f9
* | | | | | Merge "Updates launched_at in the finish and revert_migration calls"Jenkins2012-04-181-0/+2
|\ \ \ \ \ \
| * | | | | | Updates launched_at in the finish and revert_migration callsMatt Dietz2012-04-181-0/+2
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes lp984219 Change-Id: I06b113355acbc954052734ecb33a281649ec5876
* | | | | | Merge "Fix error message to report correct operation"Jenkins2012-04-181-1/+1
|\ \ \ \ \ \
| * | | | | | Fix error message to report correct operationJohannes Erdfelt2012-04-121-1/+1
| | |/ / / / | |/| | | | | | | | | | | | | | | | Change-Id: Ied2595b56da7e0a5b17d17282c2aa8dc066ef9f3
* | | | | | Merge "Reformat docstrings in n/c/a/o/servers as per HACKING."Jenkins2012-04-181-37/+27
|\ \ \ \ \ \
| * | | | | | Reformat docstrings in n/c/a/o/servers as per HACKING.Alex Meade2012-04-161-37/+27
| | |/ / / / | |/| | | | | | | | | | | | | | | | Change-Id: I80926766c1cc8d41a9fe35a625018022e732140a
* | | | | | Merge "Remove unused variable"Jenkins2012-04-181-1/+0
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Remove unused variableJohannes Erdfelt2012-04-131-1/+0
| | |/ / / | |/| | | | | | | | | | | | | Change-Id: I0a30c61db57d4704eb001f1d96127acc41878bfe
* | | | | Refactor checking instance count quota.Rick Harris2012-04-141-18/+22
| | | | | | | | | | | | | | | | | | | | Change-Id: I8ab738f3042ccf451d769c62b9fee2d20fa23953
* | | | | Small code cleanup for config_disk handling.Rick Harris2012-04-141-4/+6
| | | | | | | | | | | | | | | | | | | | Change-Id: I3456b4c98aa6527d10452e8f578a30e95a65db8e
* | | | | Refactors kernel and ramdisk handling into their own method.Rick Harris2012-04-131-17/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The size of `_create_instance` is making it increasingly difficult to work with. This is part of a series of patches aims to break it up into smaller, more digestible pieces. Change-Id: Ic668f0414937b255b1b1e1392c204307665822aa
* | | | | Merge _spawn and _configure_instance in xenapi driverJohannes Erdfelt2012-04-135-51/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They were separated only to make it easier to stubout, but we want more test coverage, not less. So merge them together and fix the tests while we're at it. Change-Id: I18333017868b91fb71b8d3dce831463a1cd5bf81
* | | | | Simplify _get_vm_opaque_ref in xenapi driverJohannes Erdfelt2012-04-132-51/+20
| |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Full instance objects are passed to all methods now, so it no longer needs to handle three different variants of argument. Also, make sure most code uses _get_vm_opaque_ref instead of calling VMHelper.lookup directly so it checks that a vm_ref was actually returned. Change-Id: Icc8b819e1d4b8bf342867d09e61fe05e6eab9e70
* | | | Test unrescue works as wellJohannes Erdfelt2012-04-133-33/+35
|/ / / | | | | | | | | | | | | | | | | | | | | | The only unrescue test that happens right now is that it fails if not already in rescue. This test fills out to test unrescue works as intended. Also, remove unnecessary stub from rescue test. Change-Id: I4451afe5a7d2ada8dfaefad804345b66ac895094
* | | Merge "Log more information when sending notifications"Jenkins2012-04-131-1/+2
|\ \ \
| * | | Log more information when sending notificationsJohannes Erdfelt2012-04-121-1/+2
| |/ / | | | | | | | | | | | | | | | | | | For debugging purposes, it's useful to see what event type is being sent and not just where it's being sent Change-Id: I809a15a76204c79d37ed68b649cd2aea80ad3db2
* | | Merge "Make exposed methods clearer in xenapi.vmops"Jenkins2012-04-133-24/+22
|\ \ \
| * | | Make exposed methods clearer in xenapi.vmopsJohannes Erdfelt2012-04-123-24/+22
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | VMOps is only used by connection.py, but it's not always obvious which methods are intended to be public and which are used internally for implementation reasons. Prefix any internal methods with _ to make that clearer. Change-Id: I50244c18233d6d7463057cf5df42d5a5a3a6fab9
* / / Renamed current_audit_period function to last_completed_audit_period to ↵Brian Elliott2012-04-125-16/+16
|/ / | | | | | | | | | | clarify its purpose. Change-Id: I7bcfbd1a0545f992b1deae54e4deb956ae33e15d
* / xenapi_conn -> xenapi.connectionJohannes Erdfelt2012-04-126-5/+5
|/ | | | | | | This keeps all of the XenAPI related code together in one directory, like libvirt. Change-Id: I1de51eb8bcead0695e9f8ede466416e0257d0148
* Merge "adjust logging levels for linux_net"Jenkins2012-04-112-64/+63
|\
| * adjust logging levels for linux_netPeng Yong2012-03-312-64/+63
| | | | | | | | Change-Id: Ifb199d644ce47c5a222f1f1bfc22a14a42f75686
* | Remove python-novaclient dependency from nova.Dan Prince2012-04-113-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | Nova should not depend on python-novaclient. Remove it. Note: In order to get unit tests to pass I had to add in a temporary dependency on httplib2. This is due to the fact that pip installing glance does not properly setup all the dependencies. See LP Bug #975483 for details. Fixes LP Bug #978992. Change-Id: I50c00139f9dc303223601230fd962abac977cd5e
* | Merge "add libvirt_inject_key flag fix bug #971640"Jenkins2012-04-101-1/+4
|\ \
| * | add libvirt_inject_key flagPeng Yong2012-04-021-1/+4
| | | | | | | | | | | | | | | | | | fix bug #971640 Change-Id: I48efc5babdd9b233342a33c87c461aabf5f5915b
* | | Merge "Extend instance UUID logging."Jenkins2012-04-101-34/+40
|\ \ \
| * | | Extend instance UUID logging.Michael Still2012-04-111-34/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add instance logging coverage to some methods in nova/compute/manager.py Change-Id: Ib1979d2bcc316cc5642c84b15af3b8ce89c3311e
* | | | Merge "Imported Translations from Launchpad"OpenStack Jenkins2012-04-1019-9039/+10407
|\ \ \ \
| * | | | Imported Translations from LaunchpadOpenStack Jenkins2012-04-0819-9039/+10407
| | | | | | | | | | | | | | | | | | | | Change-Id: I1eac99495dc2ed1b23ce12377aa93534ef4eb527
* | | | | Merge "Clean up read_deleted support in host aggregates code"Jenkins2012-04-105-61/+46
|\ \ \ \ \
| * | | | | Clean up read_deleted support in host aggregates codeMark McLoughlin2012-03-255-61/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tl;dr - this is a cleaner fix for bug #949038 It seems clear to me that all of the DB APIs should not explicitly pass read_deleted='no' or read_deleted='yes' to model_query without good reason. We want to allow callers to specify read_deleted via the context and that only works if we don't explicitly pass it to model_query(). If we don't explicitly specify it to model_query(), we use the value from the context which defaults to 'no'. Given all that, there is no need to support read_deleted to any of the DB API calls because they should support specifying the flag via the context. There should also be no need to pass read_deleted='no' because that is the default. Really, the only place there should be any mention of read_deleted is where we want read_deleted='yes' behaviour e.g. - In tests, where we want to check the operational_state of an aggregate after it has been deleted - Where we want to support undeleting an aggregate or aggreate host Change-Id: I916a8d189a33d7f30838cccb26531a024066ef96
* | | | | | Merge "Log kwargs on a failed String Format Operation"Jenkins2012-04-102-0/+12
|\ \ \ \ \ \
| * | | | | | Log kwargs on a failed String Format OperationAndrew Clay Shafer2012-04-102-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug 971552 If the string format operation in NovaException raises an exception be sure to log the kwargs at log level error Change-Id: Idfc54b8c05256bd7c849c86dc45f0b493727fe58
* | | | | | | Merge "remove unused flag: baremetal_injected_network_template baremetal_uri ↵Jenkins2012-04-101-9/+0
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | baremetal_allow_project_net_traffic"
| * | | | | | | remove unused flag:Peng Yong2012-04-021-9/+0
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | baremetal_injected_network_template baremetal_uri baremetal_allow_project_net_traffic Change-Id: I9766924cc7621e14fc52088a558b95cfd5d7ce8d
* | | | | | | Merge "Update doc to mention nova tool for type creation"Jenkins2012-04-101-1/+1
|\ \ \ \ \ \ \
| * | | | | | | Update doc to mention nova tool for type creationJoe Gordon2012-04-081-1/+1
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | Change-Id: I5ce1b4b7032f6f82dfaa7a8f60139e148a250db8
* | | | | | | Merge "Remove references to RemoteError in os-networks"Jenkins2012-04-102-18/+1
|\ \ \ \ \ \ \
| * | | | | | | Remove references to RemoteError in os-networksVishvananda Ishaya2012-04-102-18/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * this is a separate commit to make backporting easier Change-Id: I10ff04567809a2cdfbea8882f3f2c19571a60a40
* | | | | | | | Merge "Fix errors in os-networks extension"Jenkins2012-04-105-38/+65
|\| | | | | | |
| * | | | | | | Fix errors in os-networks extensionVishvananda Ishaya2012-04-105-38/+65
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Makes sure the uuid is returned as id if it exists * Simplifies db get for manager.get_networks * Removes direct db access from manager which was breaking test * Updates tests to verify the new logic * Makes sure Remote NotFounds are turned into 404s (The RemoteError blocks can be removed once https://review.openstack.org/5749 lands) * Fixes bug 977712 * Fixes bug 977723 Change-Id: I6aa815960782c7ae5165aeebd83bdaaa62c19b04
* | | | | | | Merge "fix TypeError with unstarted threads in nova-network"Jenkins2012-04-102-1/+2
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | fix TypeError with unstarted threads in nova-networkIonuț Arțăriși2012-04-022-1/+2
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (nova): TRACE: File "/usr/lib64/python2.6/site-packages/nova/utils.py", line 860, in __init__ (nova): TRACE: self.tname = "-%x-%x" % (ident & 0xffffffff, gident & 0xffffffff) (nova): TRACE: TypeError: unsupported operand type(s) for &: 'NoneType' and 'int' Fixes: bug 971629 Change-Id: I3d34816c36a11cd6fc2dd9c889c93c5872133a2c
* | | | | | Merge "Removes dead code around start_tcp in Server"Jenkins2012-04-101-21/+0
|\ \ \ \ \ \
| * | | | | | Removes dead code around start_tcp in ServerJay Pipes2012-04-101-21/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove dead code from nova/wsgi.py in Server around the irrelevant start_tcp method. Change-Id: I3222d90f6208bb3ff38d79f56365b22e551f2029
* | | | | | | Merge "Add the serialization of exceptions for RPC calls."Jenkins2012-04-1012-96/+330
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Add the serialization of exceptions for RPC calls.Alex Meade2012-04-0612-96/+330
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change uses json to serialize an exception so that it can be sent through RPC calls to be reconstructed on the other side. The traceback is added to the exception message. If recreating the exception fails for whatever reason then a RemoteError is created containing all of the exception information. Adds flag 'allowed_rpc_exception_modules' to prevent dangerous modules from being accessed and allowing arbitrary code to be run. Fixes bug 920705 Fixes bug 940500 Change-Id: Ife3b64b19fe8abbc730184d4ee7d9fcabfd29db3
* | | | | | | Merge "Improve grammar throughout nova."Jenkins2012-04-108-21/+24
|\ \ \ \ \ \ \
| * | | | | | | Improve grammar throughout nova.Paul McMillan2012-04-108-21/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes some "its" and "it's" mistakes, and unifies the tense of some documentation. Change-Id: I9b83aba535b554e80b2cf1e048bb3a4e5cac0e5e