lp:~xtoddx/nova/lp720157

Branch merges

Propose for merging

No branches dependent on this one.

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #720157: Nova returns HTTP 400 for SignatureVersion=1 requests

Medium

Fix Released

Link a bug report

Related blueprints

682. By Todd Willey on 2011-02-16

Fix parameter parsing in ec2 for older (version 1 signature) clients.

681. By Soren Hansen on 2011-02-16

Add back --logdir=DIR option. If set, a logfile named after the binary (e.g. nova-api.log) will be kept in DIR.

680. By Vasiliy Shlykov on 2011-02-16

This branch should resolve nova bug #718675 (https://bugs.launchpad.net/nova/+bug/718675).

Also added dropping of tables in case of failures so `nova-manage sync db` will can restart next time.

679. By Trey Morris on 2011-02-15

this bug bit me hard today. pv can be None, which does not translate to %d and this error gets clobbered by causing errors in the business in charge of capturing output and reporting errors.

678. By Thierry Carrez on 2011-02-15

Fixes tarball contents by adding missing scripts and files to setup.py / MANIFEST.in

677. By justinsb on 2011-02-15

When re-throwing an exception, use "raise", not "raise e". This way we don't lose the stack trace.

676. By Salvatore Orlando on 2011-02-15

The proposed fix puts a VM which fails to spawn in a (new) 'FAILED' power state. It does not perform a clean-up.
This because the user needs to know what has happened to the VM he/she was trying to run. Normally, API users do not have access to log files. In this case, the only way for the user to know what happened to the instance is to query its state (e.g.: doing euca-describe-instances). If we perform a complete clean-up, no information about the instance which failed to spawn will be left.

For the NO_HOST_AVAILABLE error, which occurs when there is not enough RAM left on the host, the amount of available memory is now checked at the beginning of the spawn process. This way, if there is not enough RAM left on the host, the spawn process returns immediately.
A test case (spawn_not_enough_memory) has been added as well.

I understand adding a new value to the power_state enumeration might not be acceptable. In that case I will propose for merge a different branch in which we perform a complete clean-up.
However, I reckon we still to provide a way to inform the user the spawn process has failed.

675. By termie on 2011-02-15

Some trivial cleanups in context.py, mostly just a test of using the updated git-bzr-ng

674. By Soren Hansen on 2011-02-15

Use eventlet.green.subprocess instead of standard subprocess

673. By Soren Hansen on 2011-02-15

Fix sporadically failing unittests.