Updating powervm driver snapshot with update_task_state flow

Fixes bug 1136359

Without this change snapshot still works but instance task state
does not get updated correctly and nova logs will show expected
task state erros from compute manager.

Also added a snapshot test.

Change-Id: I0f4ed24aa82a66fafc4b6813d017618ae07b4751
Adjusts reclaim instance interval of deferred delete tests

All deferred delete tests set reclaim_instance_interval to 1. This is
fine for the test_deferred_delete test that depends on the periodic
task _reclaim_queued_deletes to run.

For the other tests however, in the unlikely event that the periodic
task do run before the tests are done, it will interfere with the tests
by terminating the instance in the middle of it.

By setting the interval longer than the actual test run, this can be
avoided.

Resolves bug 1146510.

Change-Id: I606cd9f3277cc86cf4f9811da2a10efc7433b9d0
Don't traceback in the API on invalid keypair.

When trying to get a keypair that did not exist, a traceback would end
up in the nova-api log.  Handle this exception in the API extension so
that it doesn't make a mess of the log.

Also add a unit test for this particular scenario.

Fix bug 1145545.

Change-Id: Id1647d4ac56629c2931099320d271e34fc2f99b9
Fix issues with re-raising exceptions.

There is complication with re-raising exceptions and our usage of
eventlet.  If the code in the exception handler accesses the db or rpc
in the exception handler, it will no longer be able to re-raise the
exception.  Using excutils.save_and_reraise_exception() works aorund
this issue.

The most common error is calling LOG.error() or LOG.exception(), as it
is possible for these to go access rpc.  There is an option to turn on
notifications for these errors.

Fix bug 845866.

Change-Id: Icfca2af63805711229249aa7abe60a938edd1b35
Remove parameters containing passwords from Notifications.

compute.manager methods often takes password parameters in methods
that are wrapped with exception handlers and notifiers. What can
happen is these passwords will get bundled up and sent outside of
the system.

This patch will strip out any parameter with *_pass* in the name.

The side effect of this is that all notification will have the
error parameters in the 'args' part of the notification payload.
Previously only the positional args were in the 'args' part and
keyword args were placed in the payload directly.

This may affect consumers of the error notifications.

Change-Id: I2e7822eb5416d315ceb690f739e4dba9d52a7954
Merge "Adjusts reclaim instance interval of deferred delete tests"
Fix instance_system_metadata deleted columns

Migration 153 adds instance_type data to sys_meta, but creates entries
with the default value of deleted.. which is NULL.  This means these
entries do not get pulled in on a join when querying an instance, as
there is a condition of deleted=0.

This converts 'deleted' NULL entries to be 0.

Fixes bug 1147839

Change-Id: I3caf92b3992b22089a2653c2e7220585d616a102
Switch to oslo.config.

The oslo-config package has been renamed. Switch to the new
version. (should fix SmokeStack).

Change-Id: I6a0fa803b0076bc5338c996e21bbcef54e1d75cd
Merge "Switch to oslo.config."
Merge "Remove parameters containing passwords from Notifications."
Merge "Updating powervm driver snapshot with update_task_state flow"
Merge "Fix issues with re-raising exceptions."
Merge "Don't traceback in the API on invalid keypair."
Merge "Fix instance_system_metadata deleted columns"
Remove unnecessary setUp() and tearDown() methods

It's unnecessary to implement setUp() and tearDown() when they only call
the parent class method of the same name.

Change-Id: I8a2c7f50e17def87ada8d4f1c1b554eb916afbd1
Tiny typo

virtualizated -> virtualized

Change-Id: Ie5006f2fdd1d6963b35c63e749ffe939d5fb9bcc
Merge "Remove unnecessary setUp() and tearDown() methods"