summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | | Merge "Scope coverage report generation to nova module."Jenkins2012-04-253-1/+3
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Scope coverage report generation to nova module.Renier Morales2012-04-243-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Coverage report generation was not using --include filters. This resulted in generating a report that included all external dependencies. Some of these may include problematic characters for utf8 decoding. In the odd case this was found, coverage would fail with a UnicodeDecodeError. Fixes bug #987077. Change-Id: I276698e65ac1037972209a406058168433cd826b
* | | | | | | | | | | | | Merge "Fixed metadata validation err. Fixes bug 965102."Jenkins2012-04-254-6/+79
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Fixed metadata validation err. Fixes bug 965102.Unmesh Gurjar2012-04-124-6/+79
| | |_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Fixed the error message on specifying invalid server metadata key/value. 2. Added check to disallow specifying blank metadata key. 3. Added unit test coverage. Addressed review comments. Change-Id: I1f0002971dc236045d6e622641f46a4a30249d54
* | | | | | | | | | | | Set a more reasonable default RPC thread pool sizeJohannes Erdfelt2012-04-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | No service should be doing over 1000 different tasks at once. It can only possibly end up overloading other dependent services. 64 should be a more reasonable starting point. Change-Id: I7690ad4a9cfc5ee11e1990d5cecdf23522f11f72
* | | | | | | | | | | | Merge "Removes unnecessary code in _run_instance."Jenkins2012-04-251-35/+1
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Removes unnecessary code in _run_instance.MotoKen2012-04-241-35/+1
| | |_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove _is_instance_terminated and _shutdown_instance_even_if_deleted since run/terminate instance is synchronized. * Avoid potential deadlock. * Fixes bug 981316. Change-Id: I4fe7e2ea00b85d9e0802fe93655e67db3909e86b
* | | | | | | | | | | Merge "Use utils.save_and_reraise_exception"Jenkins2012-04-241-8/+6
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Use utils.save_and_reraise_exceptionJohannes Erdfelt2012-04-201-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's less code and also logs when exceptions gets dropping instead of silently dropping them Change-Id: I407049861b1a402549f58939b654d1760cdb2d26
* | | | | | | | | | | | Merge "Typo fix in bin/instance-usage-audit."Jenkins2012-04-241-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Typo fix in bin/instance-usage-audit.Rick Harris2012-04-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The notify method now requires a context, which in this case should be `admin_context` not `context`. Change-Id: I53275d0e1c9a898a7c0d9685382bb52ffb2efa2c
* | | | | | | | | | | | | Merge "Improved tools/hacking.py"Jenkins2012-04-244-46/+146
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Improved tools/hacking.pyJoe Gordon2012-04-244-46/+146
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * cleaner output * fix bug 980009 * Fix N201 * N306: alphabetical order imports * N401: docstring start * N402: one line docstring start * N403: multi line docstring end * Until fixed, N40* will be disabled by default Change-Id: I9addafdaa7a1f8fb950e14a5409f661dec6c7b87
* | | | | | | | | | | | | | Merge "nova.virt.xenapi_conn -> nova.virt.xenapi.connection"Jenkins2012-04-241-3/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / |/| | | | | | | | | | | | |
| * | | | | | | | | | | | | nova.virt.xenapi_conn -> nova.virt.xenapi.connectionJohannes Erdfelt2012-04-191-3/+3
| | |_|_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a couple of references that were missed in the recent move Change-Id: I653b8b85294838a39f9d805bef094938f692e7be
* | | | | | | | | | | | | Merge "Include name being searched for in exception message"Jenkins2012-04-241-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Include name being searched for in exception messageJohannes Erdfelt2012-04-191-2/+2
| |/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise, it's not always clear what was happening while looking at logs. Also, remove unused instance kwarg, this isn't a logging call. Change-Id: Ib1be2d7ed5b3c658dc2a3ac4094b812ba95a5d3c
* | | | | | | | | | | | | Merge "Removing XenAPI class variable, use session instead."Jenkins2012-04-245-27/+21
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Removing XenAPI class variable, use session instead.Rick Harris2012-04-205-27/+21
| | |/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is part of a series of refactoring which aim move towards using functions instead of classes + classmethods within vm_utils and volume_utils. Change-Id: Id0d7d21a240dd3cbf2237d480dc23dfd4d8c0dc1
* | | | | | | | | | | | Merge "Remove unused time keyword arg"Jenkins2012-04-242-13/+3
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Remove unused time keyword argJohannes Erdfelt2012-04-192-13/+3
| | |/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's not used anywhere, it aliases the imported time module and is broken at best if given since we want to use the eventlet sleep anyway (which is monkey patched into time.sleep) Change-Id: I7322ab5b5526afd3c02f48eb85066570b556bb75
* | | | | | | | | | | Merge "Logging updates in IptablesFirewallDriver."Jenkins2012-04-241-4/+4
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Logging updates in IptablesFirewallDriver.Dan Prince2012-04-191-4/+4
| | |_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update IptablesFirewallDriver so that it uses the debug log channel when logging specifics about each firewall rule. Fixes LP Bug #969545. Change-Id: Ie50a3607fff17f032d577af8a87d5f4582bcc919
* | | | | | | | | | Adding context to usage notifications.Rick Harris2012-04-234-54/+69
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch passes context objects all the way down to the usage generating code. This allows that code (in future) patches to make context-dependent calls in order to augment the usage_info dict. Change-Id: I6592d7aecd99750cab6a91bbadc2c761155d2d4d
* | | | | | | | | | Making `usage_from_instance` private.Rick Harris2012-04-234-29/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reduces the surface area of usage notifications down to just `compute_utils.notify_about_instance_usage`, making future refactoring easier. Change-Id: If70aff191c0135c1883c39c9783f5c8433953a4e
* | | | | | | | | | Merge "Remove unused variable"Jenkins2012-04-231-1/+0
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Remove unused variableJohannes Erdfelt2012-04-191-1/+0
| | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I80710b266d8bd089828996060a0746cd0a0fef01
* | | | | | | | | | Merge "Exclude xenapi plugins from pep8/hacking checks."Jenkins2012-04-231-2/+2
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Exclude xenapi plugins from pep8/hacking checks.Dan Prince2012-04-221-2/+2
| | |_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes LP Bug #987055. Change-Id: I7132b88c5e0a90b16d0f8e35df45b82e7991280f
* | | | | | | | | | Remove __init__.py from locale dirBrian Waldon2012-04-231-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, the __init__.py indicated that we have a custom locale package, preventing imports of the locale package that ships with python itself. * Fixes bug 977021 Change-Id: I5fcfb8936137b9db8a556439ca53ffeec6bbe67c
* | | | | | | | | | Merge "Improve instance logging in compute/manager."Jenkins2012-04-231-65/+62
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / |/| | | | | | | | |
| * | | | | | | | | Improve instance logging in compute/manager.Michael Still2012-04-141-65/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds instance logging coverage to all of nova/compute/mananger.py. Resolves bug 977976. Change-Id: I9392cb6202ac7997cdfd994c67374e738e8c8b10
* | | | | | | | | | Merge "Log instance."Jenkins2012-04-231-2/+2
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Log instance.Michael Still2012-04-201-2/+2
| | |_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I603eaa43b784e38532e88e9d2c844183730d44e9
* | | | | | | | | | Merge "Log instance consistently."Jenkins2012-04-231-6/+6
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Log instance consistently.Michael Still2012-04-211-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I5f85dcc1001e94886a14127d7517acdf4c2298be
* | | | | | | | | | | Merge "Log instances consistently."Jenkins2012-04-231-8/+6
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / |/| | | | | | | | | |
| * | | | | | | | | | Log instances consistently.Michael Still2012-04-211-8/+6
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I9f4ab63405e7214b082e726cc9d59626ca503c66
* | | | / / / / / / Remove unnecessary power state translation messagesJohannes Erdfelt2012-04-201-4/+0
| |_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The messages don't provide any extra value in debugging since compile_info is only called in a few cases. Also, they are anonymous and don't include information about what instance the power state is for. Change-Id: Ib4dae30a2e9c14d8b3bb9c47fe54742897dd3492
* | | | | | | | | Add instance logging.Michael Still2012-04-211-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Icff3d19fe6ee707388185d56446c980ec98fde25
* | | | | | | | | Merge "Keep nova-manage commands sorted"Jenkins2012-04-201-1/+2
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | |
| * | | | | | | | Keep nova-manage commands sortedJohannes Erdfelt2012-04-201-1/+2
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Id9934ff278a87c3ccb629cb6c9b27fd57276992c
* / / / / / / / Moves `usage_from_instance` into nova.compute.utils.Rick Harris2012-04-205-38/+40
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | Change-Id: If6ae23847fb6d150f97a538dc37d47279c183243
* | | | | | | Refactor instance image property inheritance out to a method.Rick Harris2012-04-191-24/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I14c3ff3981b610893296b59ea6a09a9f21b19133
* | | | | | | Refactor availability zone handling out to a method.Rick Harris2012-04-191-15/+20
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also moves filter_properties closer to where it's used. Change-Id: Ic2a95c2f15d138632e807249da0c26082dd1a7bf
* | | | | | Merge "Add deleted_at to instance usage notification."Jenkins2012-04-191-4/+6
|\ \ \ \ \ \
| * | | | | | Add deleted_at to instance usage notification.Rick Harris2012-04-131-4/+6
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes it easier to determine the uptime of a particular instance. Change-Id: If94d811b07120662ec3c8ce0a275f4c6178be56d
* | | | | | Merge "Make run_tests.sh just a little bit less verbose"Jenkins2012-04-191-1/+0
|\ \ \ \ \ \
| * | | | | | Make run_tests.sh just a little bit less verboseJohannes Erdfelt2012-04-121-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I9d4dc11083cb51040677eeda2d28d4b23539a304
* | | | | | | Be more tolerant of deleting failed buildsJohannes Erdfelt2012-04-191-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 985705 If an instance fails to build, the xen server may not have an instance. This can cause _get_opaque_vm_ref to unnecessarily raise an exception and add an instance fault that is misleading. Change-Id: I068d0af284a5671f429b2d8b13a25b629ede59bd