openstack/nova - nova - OpenDev: Free Software Needs Free Tools

Since reno is the new tool for Relnotes [1], we need to add it
to Nova and provide some Sphinx docs for Liberty and Mitaka.
Change-Id: Ibd74e62b6f36076dbec17dd146632fd42aad6eb2

This adds the CORS support middleware to Nova, allowing a deployer
to optionally configure rules under which a javascript client may
break the single-origin policy and access the API directly.
The paste.ini method of deploying the middleware was
chosen, because it needs to be able to annotate responses created
by keystonemiddleware. If the middleware were explicitly included,
keystone would reject the request before the cross-domain headers
could be annotated, resulting in an error response that was
unreadable by the user agent.
The test suite was expanded, to permit the inclusion of
HTTP headers, and to add the ability to send an OPTIONS request.
OpenStack CrossProject Spec:
 http://specs.openstack.org/openstack/openstack-specs/specs/cors-support.html
Oslo_Middleware Docs:
 http://docs.openstack.org/developer/oslo.middleware/cors.html
OpenStack Cloud Admin Guide:
 http://docs.openstack.org/admin-guide-cloud/cross_project_cors.html
bp:nova-cors 
Change-Id: Ibc95df0860799ff36f682da7d032d5422b0c39c1

The 'force_config_drive' option provides a way to force injection of
configuration drives on instances. Previously, this option supported
three values: 'always', 'True' and 'False'. However, the 'always'
value was deprecated in Liberty and can now be removed.
Remove this value and replace the existing 'StrOpt' with a 'BoolOpt'.
This also requires changing some unit tests to reflect the new value
for this parameter.
UpgradeImpact: This change removes support for the 'always' option.
 Configuration files providing this value will need to be updated
 accordingly.
DocImpact: This option is now a BoolOpt and documentation will need to
 be updated accordingly.
Change-Id: I0b4030ec1336fb1f2e721570c99a27a979e2847c

Change-Id: Ie1da4c73c492c4cbb4db7d53bce4743691153f74

Almost all of our unit tests for the _poll_shelved_intances() periodic
task were not actually running any code because they didn't set the config
properly. Some were also not setting the instance host either, which means
the test instances wouldn't be returned from the filter query. Since the
tests used stubs and mox, they weren't checking that the stubbed methods
were actually running and thus didn't notice.
This patch makes them use mock, check their work, and set the config
properly. It also does a little object and mock cleanup for the bits that
were touched.
Change-Id: If890eb61e42af3e1495da7db9033b19fa7811aa1

The _poll_shelved_instances() task can race with other API operations
because it does not filter instances with a task_state!=None, and because
it does not use expected_task_state=(None,) when it attempts to lock the
instance for offloading. This patch adds those checks as well as tests to
make sure they're honored going forward.
Change-Id: I7b1e235a4d09085cebf3d6d71d71757f15f69e29

I9229d882712df1ad57afbda7477fa4d72f4fd83c introduced _safe_get_floating_ips
which returns an empty list of floating ips in cases where neutron returns
a 404. It also switched a check for NeutronClientException with NotFound.
The NotFound Exception isn't used by neutron client so allow for both cases.
Closes-bug: #1513879
Change-Id: I1d37a0dadd4c9055ba545906535bb3e40e70a0f0

Fix an issue where when local conductor is used, the DB driver for
servicegroup will not handle transient DB problems gracefully. The
patch makes the behavior consistent with messaging timeouts if remote
conductor is used.
Change-Id: Ie736e7b64ea4f60f78878c8713ce826702f9f05e
Closes-Bug: 1505471

Live-migration multinode job should check different configurations.
To implement this new hook in nova should be used. Current patch adds
execution of live-migration specific use-cases.
Change-Id: I3ad9287b8f103a874f88100df82fa9e1f8617d6f

Since obj_relationships was removed in
Iea8b6348e3b01aaa335ec5408f07826f1e758d20 because manifest backports are
now used, most parent object versions will no longer change when a child
object is changed. Because of this, some objects that have a child
version tied to a parent version no longer need to be tied. The versions
can progress individually on their own. The child is all grown up now,
he's moving out of his parents' basement, buying a car, and going on to
make his parents proud.
Change-Id: Ie0c7bd419e3490d01f73c7a39cd2666eafcf9539

There is a locking regression in the 2.8.0 release that is
causing nova unit/functional tests to time out.
Depends-On: #I37eeb57e99db5f77a11eed471ac6c55c4547e821
Change-Id: I3ec6a6429bb1032e47fb0317ce9ca4b769666370
Closes-Bug: #1514876 

The current regex contains a lot of slightly off regex
definitions. Most of them don't hurt, and do still match the correct
way, but they make people believe that things like the * at the end of
matches is important (which it is not).
This cleans this up, plus adds a few comments to help people going
forward.
Change-Id: Ica7fc32fd5a756214db987b95d2305bc74cfb6f1

The cells architecture of doing db replay makes this call
fundamentally racey for cells. The contract is that if you request an
action on a server and it is locked, you get a 409 at the API, and are
done. If you get a 2xx, the resource is not locked, and life is good.
Except in cells, the API db will be updated, and the API will return
all is well. Except the db replay hasn't happened for the cell, so the
additional check in the nova compute for locked blows up.
The fix probably involves the nova-compute check doing a timed retry
under cells because of replication lags.
Change-Id: Ic18407b8372c286d8e258860f3e2bf76eceef2aa
Related-Bug: #1466696 

In I8d58be7f3cb1bdefc3b783a256c21ed216cdbad7, we added
"compute:delete", "compute:soft_delete" and "compute:force_delete",
So let's remove the older lines that have the an empty "" target.
Change-Id: I239ca596cca4631e311d7afe45de7ffcc60f70b9

Added a LOG.info nova.policy:enforce method to print
what exactly we are testing in our gate jobs (see
I62d8572e36b8ebb5ece218d858679e93ca0f8c7e) and found
these two rules were missing.
Change-Id: I94780b5173a72c93cf3f7e0dabd429f40fcda46f