summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | | | xenapi: refactor: Agent classMate Lakat2012-10-262-118/+130
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to blueprint xenapi-config-drive As config drive and agent are doing similar operations, part of the config drive introduction, we would like to make the agent optional. This refactor is introducing an agent class holding together the agent operations, as a first step. The steps were: Introduce XenAPIBasedAgent Added constructor parameters to XenAPIBasedAgent get rid of parameters on get_agent_version get rid of parameters on agent_update get rid of parameters on set_admin_password get rid of parameters on inject_file get rid of parameters on resetnetwork Correct newlines add factory method for getting an agent instance For the small changes, see: https://github.com/citrix-openstack/nova/commits/agent2class Change-Id: I3928c0776ff98c830c19487eb8ff7820e41ca025
* | | | | | | | | | | | | Merge "Add call to trigger_instance[add/remove]_security_group_refresh quantum"Jenkins2012-10-301-6/+20
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Add call to trigger_instance[add/remove]_security_group_refresh quantumAaron Rosen2012-10-291-6/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The calls to trigger_instance[add/remove]_security_group_refresh were not being called in quantumv2/api. This patch adds those calls and also removes the code previously required to handle the circular dependency which is no longer there. Fixes bug 1057069 Change-Id: Ia76d37ebf2546a26f893dcbc8d5a05f02324ad6b
* | | | | | | | | | | | | Fix typo in xenapi/host.pyDan Smith2012-10-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The failure path for host evacuation has a typo in the instance_update() call to restore the 'host' attribute. Change-Id: If45ea907b411ffc8c751c608e487828972337c5c
* | | | | | | | | | | | | Merge "Remove extra print line in hacking.py"Jenkins2012-10-291-1/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Remove extra print line in hacking.pyJoe Gordon2012-10-291-1/+0
| |/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: If3d5da52a55c1834f4dfa49d817010d314d1403a
* | | | | | | | | | | | | Merge "Validates Timestamp or Expiry time in EC2 requests"Jenkins2012-10-293-0/+127
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Validates Timestamp or Expiry time in EC2 requestsSirisha Devineni2012-10-293-0/+127
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Validating the format of Timestamp/Expires in the EC2 requests and checking for the expiry of the request. 'ec2_timestamp_expiry' flag is the time in seconds before ec2 timestamp expires. Fixes bug 1036343 Change-Id: I2b63d85dc1d658a58ceda67c0dfd0a8eac807577
* | | | | | | | | | | | | Merge "Add API samples to Admin Actions"Jenkins2012-10-2963-0/+396
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Add API samples to Admin ActionsMauro S. M. Rodrigues2012-10-2963-0/+396
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: If072bf9e9a55c79c71fd3911776045ee46483a00
* | | | | | | | | | | | | | Merge "Add live migration helper methods to fake hypervisor driver"Jenkins2012-10-291-0/+13
|\| | | | | | | | | | | | |
| * | | | | | | | | | | | | Add live migration helper methods to fake hypervisor driverMauro S. M. Rodrigues2012-10-291-0/+13
| | |_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These methods were necessary to api samples tests. Change-Id: I49315a7376b4165d0a51f4110711ccd16bc83aa0
* | | | | | | | | | | | | Merge "Use testtools as the base testcase class."Jenkins2012-10-2912-149/+35
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Use testtools as the base testcase class.Monty Taylor2012-10-2912-149/+35
| | |/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As a step towards parallelizing the test runs, use testtools as the base unittest test case class. In this patch, it has caused us to catch places where setUp and tearDown were not being chained properly. Additionally, we get to remove our custom skip decorators and a few backported assert calls. Part of blueprint grizzly-testtools Change-Id: Ia2025e1b7191b65130b85b30ce6239d5c431c8ba
* | | | | | | | | | | | | Merge "Addtional 2 packages for dev environment on ubuntu"Jenkins2012-10-291-0/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Addtional 2 packages for dev environment on ubuntuJolyon Brown2012-10-251-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In file doc/source/devref/development.environment.rst added packages build-dep and python-mysqldb which I have had to add on ubuntu 12.04 to enable the dev environment to be setup. Without these run_tests.sh fails with error 'mysql_config not found' when setting up the virtual env. Change-Id: I4af775386469deecf122a9e165457f241339b20b
* | | | | | | | | | | | | | Make nova-rootwrap optionalBrian Elliott2012-10-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the use of nova-rootwrap optional if the current process's effective user id is already root. Change-Id: I2cf5c743c6f4d6f235a54ca70c11d8191deeed6a
* | | | | | | | | | | | | | Merge "Let scheduler know services' capabilities at startup"Jenkins2012-10-295-4/+19
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Let scheduler know services' capabilities at startupArata Notsu2012-10-285-4/+19
| | |_|_|_|_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1071254. Changes: * Add new rpc-api(fanout) of compute "publish_service_capabilities" This rpc-api urges services to send its capabilites to the scheduler. * Scheduler calls publish_service_capabilities right after the start By them, the scheduler get to know the capabilities earlier. Now we can expect that the scheduler always holds the capabilities. So it is reasonable to change HostManager to ignore hosts whose capabilities are "None" since it becomes a rare case; this will make scheduling more reliable. This will achieved by Another patch. Change-Id: If6582765011fd5e1b794bfdc068e17630ba381cb
* | | | | | | | | | | | | | Merge "Remove duplicated definition of is_loaded()"Jenkins2012-10-291-3/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | Remove duplicated definition of is_loaded()Yun Mao2012-10-281-3/+0
| | |_|_|_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I34c247bc0bc349bf0f7be49a35bb03596c070134
* | | | | | | | | | | | | Merge "Fix nova-network MAC collision logic"Jenkins2012-10-292-1/+34
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Fix nova-network MAC collision logicDan Smith2012-10-262-1/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The exception being caught and transformed in sqlalchemy/api.py was incorrect, causing nova-network to miss it in the MAC collision retry loop. This patch corrects that and adds a test of the collision logic. Fixes bug 1059366 Change-Id: I6bf16c2f59f06c7e84441604b057afd8dfe6d709
* | | | | | | | | | | | | | Merge "removes the nova-volume code from nova"Jenkins2012-10-29103-17830/+110
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | removes the nova-volume code from novaVishvananda Ishaya2012-10-28103-17830/+110
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes the majority of the nova-volume code from the codebase. It updates relevent config options to default to cinder. It updates a number of existing tests that were depending on code that was removed. A few things still need to be removed: * volume/driver.py & volume/iscsi.py These files are used by the libvirt volume driver tests. These tests should be updated to mock the relevant calls. * scheduler/simple.py & scheduler/multi.py These files should no longer be necessary so they can be removed in a subsequent patch * exception.py cleanup Once the above files are removed there are a number of unused exceptions which can be removed * database calls and database tables The database calls have not been removed and the tables have not been dropped. This can be done in a separate migration * additional config options and nova.conf.sample There may be a few extra config options that can be removed and the conf sample can be regenerated Implements bp delete-nova-volume Change-Id: I0b540e54dbabd26901a7530035a38583bb521fda
* | | | | | | | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2012-10-2921-2319/+2268
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2012-10-2921-2319/+2268
| | |_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I5994735bd5cc6ba558361ea9397937d0cf58f95c
* | | | | | | | | | | | | Merge "Don't elevate context when calling run_instance"Jenkins2012-10-292-12/+6
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Don't elevate context when calling run_instanceVishvananda Ishaya2012-10-282-12/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The filter scheduler should be passing a regular context to compute when calling run_instance instead of an elevated context. Fixes bug 1072205 Change-Id: I4d4f96e17eef8050c25b63f4b26d7d376a91cb80
* | | | | | | | | | | | | Merge "Make instance_get_all() not require admin context"Jenkins2012-10-272-20/+26
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Make instance_get_all() not require admin contextChris Behrens2012-10-272-20/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Helps to clean up tests which use this method. Change-Id: Ic707b1d1b679b983b4d2b7dcad5a5a33ed771d3c
* | | | | | | | | | | | | | Merge "Rehydrate NetworkInfo in reboot_instance()"Jenkins2012-10-272-2/+44
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | Rehydrate NetworkInfo in reboot_instance()Dan Smith2012-10-252-2/+44
| | |_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the process of implementing no-db-compute, the compute manager's reboot_instance() was stripped of its database accesses for getting an instance's block and network device info. That information is now passed in from the API layer over RPC. In the offending changeset (), the network_info was getting coerced to a list and thus calls to NetworkInfo.legacy() were failing in the manager's _legacy_nw_info() method. This patch rehydrates the NetworkInfo using the existing method for doing so, which fixes bug 1071017. A test is also added to prevent future regressions. Change-Id: I8fd1c2dde573bb7da3bfe1ee3193f9ba1167aaa7
* | | | | | | | | | | | | Fix compute tests abusing admin contextChris Behrens2012-10-271-33/+47
| |/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some compute tests are using admin context when not necessary, which could cover up potential issues in the future. Change-Id: Ia755c397454c4c8a8504bb76c0126ce61e7c014b
* | | | | | | | | | | | Merge "Change hacking.py N306 to use logical_lines"Jenkins2012-10-271-3/+3
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Change hacking.py N306 to use logical_linesJoe Gordon2012-10-261-3/+3
| | |_|_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only logical lines (does not include comments) should be checked for alphabetical import order. This patch changes N306 (imports not in alphabetical order) to run on logical lines not physical lines. Fixes bug 1071849 Change-Id: Ib2cb92f9eb82aeed1ff5879d3e38c8961b4f06cc
* | | | | | | | | | | | Merge "Fix use of elevated context for resize methods"Jenkins2012-10-273-119/+105
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Fix use of elevated context for resize methodsChris Behrens2012-10-273-119/+105
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | elevated contexts were being passed over RPC to confirm_resize, revert_resize, and finish_revert_resize methods. If those methods need to elevate context, they should do so themselves. There was 1 case (when deleting an instance in RESIZED) when we'd send a non-elevated context to confirm_resize(). This would fail because the migration_get() call there requires admin context and confirm_resize() on the manager side did not elevate the context itself. This fixes bug 1071600 and removes sending elevated contexts for these methods over RPC. (Ultimately we need some work to make the migration* DB API calls verify project_id vs requiring admin context..) Change-Id: I907889502f19fba8059253e52dca084a123016e2
* | | | | | | | | | | | | Merge "Correct _extract_query_params in image.glance"Jenkins2012-10-271-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Correct _extract_query_params in image.glanceMotoKen2012-10-261-2/+2
| | |_|_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates _params rather than params to ensure filters. Fixes bug 1070138. Change-Id: I4e71d19f120ea91d8dfca32785ccb0aa7b1c770b
* | | | | | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2012-10-2721-10260/+9768
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2012-10-2721-10260/+9768
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I8930843feb15ad76de53a133af1ee60e5dd7124a
* | | | | | | | | | | | Fix check for memory_mbDavanum Srinivas2012-10-272-2/+4
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | assert_compute_node_has_enough_memory should check memory_mb is None or not _assert_dest_node_has_enough_disk should check available_mb as well Fixes bug 884842 Change-Id: Ieaa33efd50bb84f1b475c437b26af1302fdfd19f
* | | | | | | | | | | Merge "Send full migration data to finish_resize."Jenkins2012-10-264-29/+33
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Send full migration data to finish_resize.Russell Bryant2012-10-244-29/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch updates the finish_resize method in the compute rpc api to receive all of the migration data instead of just the migration id. This avoids an unnecessary database read in finish_resize. Part of blueprint no-db-compute. Change-Id: I4791551a13a8b73e7a5c22ab9f49cdab7db8de1f
* | | | | | | | | | | | Merge "Fix rpcapi version for new methods."Jenkins2012-10-261-2/+4
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Fix rpcapi version for new methods.Russell Bryant2012-10-261-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Two methods were recently added to the network rpcapi in version 1.1. Properly require that the manager side implement version 1.1 to accept rpc messages for these methods. Change-Id: I7257301a150889bc3d643197bf1e5e9a656b1d9c
* | | | | | | | | | | | | Merge "Extend IPv6 subnets to /64 if network_size is set smaller than /64"Jenkins2012-10-261-0/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Extend IPv6 subnets to /64 if network_size is set smaller than /64Takashi Sogabe2012-10-251-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If cidr_v6 is set with network_size smaller than /64, this fix extends the subnet to /64. Fixes bug #1011122. Radvd (which is used by nova-network) also requires as /64 for autoconf at the client(instance) side. Change-Id: I0a6ac363e46dd9b8c9cfb750ce37720fdbc56a1f
* | | | | | | | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2012-10-2621-17439/+18470
|\ \ \ \ \ \ \ \ \ \ \ \ \ \