Correct exception handling when create aggregate

when create an aggregate, InvalidAggregateAction might be raised,
this patch translates the InvalidAggregateAction to HTTPBadRequest.
Also, 2 unnecessary logs was removed becuase log is not output in
most BadRequest.

Change-Id: I3dac808d8edab6118e347eb5d78b96f96b9d40ca
libvirt: Remove dated docstring

The docstring is out of date, we can update it or remove it.
The libvirt_opts includes all the details, so remove it.

Change-Id: Ia903ff71509b4228faf0580893c4720b5d8b9c61
Move set_admin_password tests from test_compute.py to api/mgr modules

To prepare for converting the set_admin_password compute API flow to use
objects consistently, i.e. no direct database updates, use dot notation
to access instance object attributes, etc, the unit tests need to be
refactored and moved from test_compute.py into test_compute_api.py and
test_compute_mgr.py to properly unit test the compute api and manager
changes, respectively.

As part of the move, the tests are re-written with mock.

Part of blueprint compute-manager-objects-juno

Change-Id: I5815f7dec3105440f1b211d7bac98888fde7a576
Remove duplicate method

Duplicate method(_do_test_create_instance_above_quota)
are found in compute/test_servers.py. Remove it.

Change-Id: I0753992cadc40adce40999bb5ae8ecd503bd53bc
Merge "Remove duplicate method"
Merge "Correct exception handling when create aggregate"
Merge "libvirt: Remove dated docstring"
Merge "Move set_admin_password tests from test_compute.py to api/mgr modules"