Remove duplicated comments in virt/storage_users

Change-Id: Ia8f45dc6588b5e456dcfef9c64b5aa14df9365e7
Separate image props filter unit tests

Pull the unit tests for the image properties filters out of
test_host_filters.py and uncruft the unnecessary service_up stubouts.

Change-Id: I5d2cd393fd18de9debff424d9ef37e549c2a7ea2
Separate compute caps filter unit tests

Pulls the unit tests for compute capabilities filters out of
test_host_filters.py and into their own test file.

Change-Id: Ifa941790fdbe872433629424fd96f2781272c60c
Replacement `_` on `_LE` in all LOG.error

oslo.i18n uses different marker functions to separate the
translatable messages into different catalogs, which the translation
teams can prioritize translating. For details, please refer to:
http://docs.openstack.org/developer/oslo.i18n/guidelines.html#guidelines-for-use-in-openstack

There were not marker fuctions some places in directory network.
This commit makes changes:
* Add missing marker functions
* Use ',' instead of '%' while adding variables to log messages

Change-Id: I5a8f381b6f8fdb4e8febe9e6a901f7cdc6846646
Nova should be in charge of its log defaults

Oslo's log module has a set_defaults which Nova can use
to set the context logging format and default log levels.
We should not have to fix default levels in oslo and then
sync across to Nova. This becomes all the more important
when we move to oslo.log library so we should get started
now

Change-Id: Id47099d5a755bc2923b05fe4bef270a7c0f99389
Set logging level for glanceclient to WARN

When capturing a snapshot for example, glanceclient logs
the whole uploaded image into the nova compute logs as
glanceclient's log level is set to DEBUG by default

Change-Id: Ie764517c2a4b91cb02c15096f7f0b4138900c92d
Port floating_ips_bulk extention to v2.1

This patch ports floating_ips_bulk extention from v2 to v2.1,
and have v2 unit test cases shared between v2.1 and v2.
Partially implements blueprint v2-on-v3-api

Change-Id: I9444ff9c394d530cb76dd4ac21b51cd4de7ef6dc
Remove project id in ViewBuilder alternate link

Clearly nobody really uses the "application/vnd.openstack.image"
links in the returned results from GET /v2/{tenant}/images
REST API call, since the link URI returned in it is wrong.

Glance URIs do *not* contain a project or tenant in the URI
structure like Nova's REST API URIs do, but _get_alternate_link()
method of the image ViewBuilder tacks it on improperly:

Change-Id: I4ccb5628569c60026a1772df0794ac0271c48db3
Closes-Bug: #1343080
Add API validation schema for cloudpipe api

Add API schema to validation, require project_id need to be uuid formate.

Partially implements blueprint v2-on-v3-api
Change-Id: I9d95229d9391b419fe633b5b9e6f06f68289d3a0
api: add serial console API calls v2.1/v3

This is a followup patch to:
api: add serial console API calls v2
    (https://review.openstack.org/#/c/113966/)
To add missing v3 APIs

Change-Id: I56a045d9b690ea36753513575a2023582da58e42
Patial-Implements: blueprint v2-on-v3-api
Add API schema for v2.1 "resize a server" API

Some v2.1 APIs(create/update/rebuild server) contains the API
extension feature, it searches extension methods with its method
name and operates them automatically.

It is necessary to change the API schemas also automatically based
on the API extension loading condition, because the API extensions
add API parameters.

This patch adds API schema extensions which are implemented with
the same mechanism (stevedore library) as the API extension for "resize
a server" extensions.

Partially implements blueprint v2-on-v3-api

Change-Id: I3651da8c1d23ee1b5d4710b1b323ecf9337ca70d
Merge "Remove duplicated comments in virt/storage_users"
Merge "Nova should be in charge of its log defaults"
Merge "Set logging level for glanceclient to WARN"
Run build_and_run_instance in a separate greenthread

If we're doing a lot of build operations, we are using a large portion
of the limited rpc worker pool for long periods of time. Since we may wait
on external services (like neutron or glance) during those times, we could
fully deplete that pool.

This patch makes us spawn a new greenthread for that task and return the
rpc worker to the pool. Due to some funkiness with the stack of decorators,
this breaks the inner function out to an object method, which is probably
good anyway, given its size. This also moves the wrap_instance_event
decorator to the inner function so that the start and stop events properly
demarcate the actual task and not just the (now very quick) RPC call.

Change-Id: Ife712c43c5a61424bc68b2f5ab47cefdb46ac168
Closes-Bug: #1372049
Merge "Separate image props filter unit tests"
Merge "Port floating_ips_bulk extention to v2.1"
Merge "Separate compute caps filter unit tests"
Merge "Replacement `_` on `_LE` in all LOG.error"
Merge "Add API validation schema for cloudpipe api"