summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | Merge "Use explicit args in run_instance scheduler code."Jenkins2012-08-067-40/+69
|\| | | | | | | | | |_|_|/ / / / / |/| | | | | | |
| * | | | | | | Use explicit args in run_instance scheduler code.Russell Bryant2012-08-067-40/+69
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch goes through code paths related to run_instance in the scheduler to use explicit arguments instead of **kwargs. I'm trying to work out exactly what data is getting passed through the system when an instance is created, so that I can make the no-db-messaging related changes. This patch helps clarify some things. Part of blueprint no-db-messaging. Change-Id: Id83fc1e399e1d5c75af6dd3c8972b8ba9d706af1
* | | | | | | | Merge "Update args to _set_vm_state_and_notify."Jenkins2012-08-062-10/+11
|\| | | | | | |
| * | | | | | | Update args to _set_vm_state_and_notify.Russell Bryant2012-08-062-10/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch removes *args and **kwargs from _set_vm_state_and_notify. The only additional argument used was request_spec, so just list it out explicitly. Change-Id: I64590a2d79c2d303e7c936a4c3b1545eecfd0f2b
* | | | | | | | Merge "Reduce db access in prep_resize in the compute manager."Jenkins2012-08-067-45/+96
|\| | | | | | |
| * | | | | | | Reduce db access in prep_resize in the compute manager.Russell Bryant2012-08-067-45/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch changes the arguments passed to prep_resize in the compute manager. Previously it took instance and instance_type IDs. It now receives a full dict for both of these. This cuts down on database access needed on the compute node. This method was not previously in the compute rpcapi module, so it has been added there. The scheduler had a bit of work done ot get it to use the rpcapi module for prep_resize, as well. Part of blueprint no-db-messaging. Change-Id: Idadbf4fc624d5d1b128f758a46c61b3c840b9898
* | | | | | | | Merge "Remove instance_id fallback from cast_to_compute_host()."Jenkins2012-08-062-7/+5
|\| | | | | | |
| * | | | | | | Remove instance_id fallback from cast_to_compute_host().Russell Bryant2012-08-062-7/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes some code that supported using either an instance id or uuid when doing an instance update. The db API no longer accepts an ID. A UUID must be used or it will raise an exception. I also updated the unit tests to reflect this. Change-Id: I5a9442f3b571f37950676e96e2f129ea45c55d78
* | | | | | | | Merge "Fix test_resize_xcp testcase - it never ran"Jenkins2012-08-061-12/+12
|\ \ \ \ \ \ \ \
| * | | | | | | | Fix test_resize_xcp testcase - it never ranMate Lakat2012-08-061-12/+12
| | |_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | Change-Id: I7a55c53769574251e046c9621363a00e716fc06d
* | | | | | | | Merge "tests: avoid traceback warning in test_live_migration"Jenkins2012-08-061-1/+9
|\ \ \ \ \ \ \ \
| * | | | | | | | tests: avoid traceback warning in test_live_migrationPádraig Brady2012-08-061-1/+9
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I959e780e88a31ad8105dc0d8bbb544c8c50f49e5
* | | | | | | | Merge "ensure_tree calls mkdir -p"Jenkins2012-08-061-1/+9
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | |
| * | | | | | | ensure_tree calls mkdir -pMichael Still2012-08-061-1/+9
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let's use os.makedirs() instead... Change-Id: Ie78b3c1107ac02263703cb5f4406c4ba4f83c7ad
* | | | | | | Merge "Remove old exception type"Jenkins2012-08-061-1/+2
|\ \ \ \ \ \ \
| * | | | | | | Remove old exception typeStanislaw Pitucha2012-08-041-1/+2
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Error doesn't exist anymore. Use a generic NovaException instead. Change-Id: I3b36fdf999f5e62a7b4137dfa874173de5fed9b8
* | | | | | | Merge "Remove unused InstanceInfo class."Jenkins2012-08-064-37/+0
|\ \ \ \ \ \ \
| * | | | | | | Remove unused InstanceInfo class.Dan Prince2012-08-064-37/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We no longer use InstanceInfo. Also removes some unused _map_to_instance_info functions in various virt drivers. Change-Id: I02ead10e43426b9ad051970e9e78e970de5b1ac1
* | | | | | | | Merge "Remove list_instances_detail from compute drivers."Jenkins2012-08-068-53/+3
|\| | | | | | |
| * | | | | | | Remove list_instances_detail from compute drivers.Dan Prince2012-08-068-53/+3
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removes the list_instances_detail function from various compute drivers which implement it. We no longer make use of this compute driver call when syncing power states. Change-Id: I4980bd8d4ec14c61e16d5be105659f0d5b6748ba
* | | | | | | Merge "Migrate a logging change from common:"Jenkins2012-08-061-6/+8
|\ \ \ \ \ \ \
| * | | | | | | Migrate a logging change from common:Andrew Bogott2012-08-031-6/+8
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | commit 58404a1a9461900367dbb7a8b3f2f98441e12959 Author: Mate Lakat <mate.lakat@citrix.com> Date: Tue Jul 31 17:53:57 2012 +0100 Install a qualified except hook. Fixes bug 1031351. As an Exception is raised, it was not visible in the log, because it wasn't qualified. The fix sets up the exception hook to be qualified as the product_name. Change-Id: I38c48d0dd2ac885fcb226310b6039be667af9e0d
* | | | | | | Merge "Adds per-user-quotas support for more detailed quotas management"Jenkins2012-08-0612-157/+1289
|\ \ \ \ \ \ \
| * | | | | | | Adds per-user-quotas support for more detailed quotas managementKylin CG2012-08-0612-157/+1289
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements blueprint per-user-quotas. Based on the original quotas structure. NOTE: quota_instances, quota_cores, quota_ram, quota_volumes, quota_gigabytes, quota_key_pairs and quota_security_groups are supported per user. Allow 'projectadmin' role to access the user quota setting methods. Add commands 'nova-manage quota project/user' for quotas management. Change-Id: I07a39499432571fedd819c53ae414240cefc3354
* | | | | | | Merge "Improve external lock implementation"Jenkins2012-08-067-354/+63
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Improve external lock implementationStanislaw Pitucha2012-08-047-354/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove a number of limitations from the external locks. - They can be nested now - They do not need external cleanup in case of failures - They do not rely on lockfile or greenlet internal implementation New implementation is based on fcntl locks and any crashing process will drop the lock. It does not have to rely on any cleanup code or handling exceptions. Because no cleanup is needed, a number of tests have been removed. This implementation is not portable outside of POSIX/BSD/SVR4 systems. Fcntl locks should work correctly with NFS mounts. Locks are cleaned up after the tests finish running via run_tests.sh, even though it's not strictly needed. This change requires eventlet >= 0.9.17. bp improve-external-locking Change-Id: Idf5424c04645f25097733848a007b150145b0b27
* | | | | | | Merge "Only log deprecated config warnings once."Jenkins2012-08-062-1/+20
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | |
| * | | | | | Only log deprecated config warnings once.Dan Prince2012-08-052-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates Nova's deprecated warn function so that we only log messages about deprecated config options once. The motivation for this change is to cut down the log file noise when deprecated config code gets called more than once. Change-Id: If8919817330dc9461a0472fea982d43c78a86f66
* | | | | | | Merge "Handle NetworkNotFound in _shutdown_instance."Jenkins2012-08-052-1/+28
|\ \ \ \ \ \ \
| * | | | | | | Handle NetworkNotFound in _shutdown_instance.Michael Still2012-08-052-1/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If no network is found, continue to perform the rest of the instance shutdown, instead of propogating the exception. Resolves bug 1008875. Change-Id: Ia78480edaed52932830da7b4e49c15fd2e1fa315
* | | | | | | | Merge "Remove **kwargs from prep_resize in compute manager."Jenkins2012-08-052-18/+9
|\ \ \ \ \ \ \ \
| * | | | | | | | Remove **kwargs from prep_resize in compute manager.Russell Bryant2012-08-032-18/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The only place where additional arguments were being passed in was via the unit tests, and they aren't used. Part of blueprint no-db-messaging. Change-Id: I13f4a209ac9acc732966cb6e01bdbbcc3e3bd3c8
* | | | | | | | | Merge "Updates to the prep_resize scheduler rpc call."Jenkins2012-08-0510-54/+96
|\| | | | | | | | | |_|/ / / / / / |/| | | | | | |
| * | | | | | | Updates to the prep_resize scheduler rpc call.Russell Bryant2012-08-0310-54/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I started looking at this because of the prep_resize rpc call in the compute manager. An API server calls prep_resize on the scheduler which then calls it on a compute node. I will need to make the same changes on the compute side, but this is the first step. The prep_resize call was taking two object IDs, an instance UUID and an instance_type ID. Both of these have been converted. It now takes an instance dict and an instance_type dict, instead. It can also handle receiving the old IDs for backwards compatibility. prep_resize also took a topic argument that was unused, so it has just been removed. There are a number of changes in the scheduler code tied up in this to make it more explicit about exactly what arguments are expected instead of just using *args, **kwargs. Part of blueprint no-db-messaging. Change-Id: I4af18e5575e2bb60a410fc8edabf3d607c72aabc
* | | | | | | | Merge "Drop AES functions and pycrypto dependency"Jenkins2012-08-053-58/+0
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | |
| * | | | | | | Drop AES functions and pycrypto dependencyStanislaw Pitucha2012-08-053-58/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At some point IV parameter has been removed making these functions dangerous to use unless keys are unique on every message. Since the functions were added the original consumer has disappeared too. With no more users it's better to get rid of the incomplete encryption system entirely. These were the only functions using pycrypto module directly, so it was also removed from the dependency list. Change-Id: I57b0a0a42dac455d5baae3d726aea1507121aa4d
* | | | | | | | Merge "Update reset_db to call setup if _DB is None."Jenkins2012-08-051-0/+2
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Update reset_db to call setup if _DB is None.Dan Prince2012-08-031-0/+2
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This resolves issues when running individual tests that require a database. With this fix I can once again run individual test modules that make use of the database. For example: ./run_tests.sh test_libvirt Previously this would fail with database errors. Fixes LP Bug #1032738. Change-Id: Icce7ac9414f0e19eece44819a217634947de7f73
* | | | | | | Merge "Simplify file hashing"Jenkins2012-08-051-1/+2
|\ \ \ \ \ \ \
| * | | | | | | Simplify file hashingStanislaw Pitucha2012-08-051-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since map() is not lazy, hash_file() created huge lists filled with `None` when hashing large files. Rewriting it to a simple loop prevents that. Change-Id: I65681f905bacc2afda52853bbe9e364b89f515a8
* | | | | | | | Merge "Fix test classes collision"Jenkins2012-08-051-2/+2
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Fix test classes collisionStanislaw Pitucha2012-08-041-2/+2
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test classes LibvirtConfigGuestSerialTest were colliding, hiding a failing test by accident. One has been renamed to *GuestConsoleTest. Change-Id: Idc67634b3e5a616021840fa66313a65b135e6f86
* | | | | | | Merge "Fix notification logic"Jenkins2012-08-052-2/+21
|\ \ \ \ \ \ \
| * | | | | | | Fix notification logicStanislaw Pitucha2012-08-042-2/+21
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Notifications were always treated as if they're about state changes due to a typo. Additionally exception handler did not work correctly. Regression tests included. Change-Id: I0b92a1baa17768d9cf4e709b3331480548dd041e
* | | | | | | Merge "Remove unused variables"Jenkins2012-08-041-5/+3
|\ \ \ \ \ \ \
| * | | | | | | Remove unused variablesStanislaw Pitucha2012-08-041-5/+3
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fetch() no longer returns metadata so fetch_to_raw does not need to pass it. Callers don't expect the return value either. Output of execute() is not used for anything Change-Id: I84be22e042696a51eb919f6c6f0ccf58aaf3ace3
* | | | | | | Merge "Update list_instances to catch libvirtError."Jenkins2012-08-042-3/+3
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | |
| * | | | | | Update list_instances to catch libvirtError.Dan Prince2012-08-032-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit be590cfa4b0025587a956642dc859902d577f3f7 caught (and tested for) the wrong exception. lookupByID throws a libvirtError. Fixes LP Bug #1032741. Change-Id: I6745660658df4d0f2bc4cd423e173200fc23cd8e
* | | | | | | Merge "Require eventlet >= 0.9.17"Jenkins2012-08-041-1/+1
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | / / | | |_|_|/ / | |/| | | |
| * | | | | Require eventlet >= 0.9.17Pádraig Brady2012-08-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0.9.17 was released 2012-08-03 and includes two particular fixes related to OpenStack 1. https://bitbucket.org/which_linden/eventlet/issue/123/ Fix an exception thrown by epoll in certain cases. This can cause jenkins to deadlock, triggered for example by: https://review.openstack.org/#/c/10767/ 2. https://bitbucket.org/which_linden/eventlet/issue/115/ https://bugs.launchpad.net/nova/+bug/903199 Fix a significant memory leak of _DummyThread objects. Change-Id: I449f30afde7d162ed2c0df149174f1250ef378ca
generated by cgit (git 2.34.1) at 2026-03-31 18:14:12 +0000