This patch ports floating_ip_pools 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: I0b34358db08a29e76a59b22a0992abc88296058d
The modifications to add joins for fixed_ip_requests allows us to
condense three network calls into one for both start and finish
migration.
Change-Id: I61ca03661f722c6406d39021b79a648f4374dfcd
Partial-bug: #1365606
Many of the fixed ip related calls are extremely inefficient. This
joins a couple of calls so that we can optimize queries that use
them.
Change-Id: Ifa2e4c6ac517286f7ae952052d29801b45a6b6b2
Partial-bug: #1365606
Right now any test that fails spews *all* of the migration steps at INFO
level along with the test output (assuming it did migrations). This patch
squelches the INFO-level messages from migrate, which don't add anything
useful anyway. Messages look like this:
...
2014-10-10 09:51:41,967 INFO [migrate.versioning.api] 264 -> 265...
2014-10-10 09:51:41,984 INFO [migrate.versioning.api] done
2014-10-10 09:51:41,984 INFO [migrate.versioning.api] 265 -> 266...
2014-10-10 09:51:42,003 INFO [migrate.versioning.api] done
Change-Id: I424bbb25c04b5e327c6d5d2ff8bad266bc7d51d0
This revises the servers, images, and flavors schemas to avoid a
duplicate definition of the top-level tags in those schemas. Instead,
of defines each schema to allow either a list of tags with only
name/id (as in the index listings), or a list of fully-defined
servers/images/flavors. It is not permitted to mix and match.
It also adds three kinds of unit tests for the API schemas:
* A basic check to ensure that the schema files are valid RelaxNG;
* A check to ensure that the collected body of schema files does not
contain duplicate top-level tag definitions;
* Explicit test XML documents.
Closes-Bug: 885025
Change-Id: I542b420de34fd82bb2a2813f36637aef2c2d5c92
Using the flavor extra_specs property "hw_watchdog_action" was broken.
Scheduling of a new instance always failed with NoValidHostFound error
because of ComputeCapabilitiesFilter, which treated this property as a
host capability to be checked.
Commit f0ff4d5105 caused the problem.
To fix this watchdog_action property is put into 'hw:' scope, so
that it will be ignored by ComputeCapabilitiesFilter in scheduler and
handled in libvirt driver. The doc must be fixed accordingly.
Now driver accepts both 'hw_watchdog_action' and
'hw:watchdog_action', tests were edited for these cases.
Were added TODO items to delete the compat code in the next release.
DocImpact
Closes-Bug: #1367344
Change-Id: Ic5344ec34a130ee5a0ed2c7348af0b9d79e3508e
Most of the API code was using instance objects already except for the
listing, so this updates that.
Updates the test to use a fake instance object rather than a dict to
enforce the change.
Also adds a note on the slightly weird call to common.get_instance
in the show method when the instance isn't used for anything, it's
really a check to make sure the instance exists.
Part of blueprint kilo-objects
Change-Id: Ief17c57afbd81e2b6aef9586b448e8797aaa6544
commit a8a5d44c8a add pci_request_id
as one item for the request_network tuple. But the icehouse code
assume only three items in the tuple.
This patch filters pci_request_id out from the tuple.
Change-Id: I991e1c68324fe92fac647583f3ec8f6aec637913
Closes-Bug: #1377447
Adds support to enable guestfs logging.
Change-Id: I46c5ec6606e4a2eb8277b081d0c773ceb6d36c4c
DocImpact: New group of option has been introduced "guestfs"
Closes-Bug: #1279857
This patch ports couldpipe extension 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: Id40c637c0fc207e97b1d094805eb0cee6e9cd9c9
This patch is prepare for porting floating_ips to v2.1,
and make v2.1 and v2 share same test cases.
Change-Id: I7ba202adeb7d65f1e04c36e8db647565c1fcaeb2
The list comprehension used to create the overs list was really hard to
read, so this moves that code into it's own helper method and breaks out
the logic from the list comprehension, and adds unit tests.
Partial-Bug: #1369605
Change-Id: I44236c7a206b2a9bd3843c9ca2ce56591a66352f
Commit 5aef264967 added the log message to
give some idea when we're unnecessarily refreshing quota usage.
Logstash only collects INFO+ level messages, so bump this to an INFO
level message so we can track gate runs to see if this is actually a
problem.
Related-Bug: #1353962
Change-Id: I60ffa9dbda22e252f290bf316fa2e472d3e92f2f
This moves the over-quota processing out of quota_reserve and into a
private method.
This completes the refactor of the quota_reserve method to isolate the
large chunks of logic into separate private methods.
Closes-Bug: #1369605
Change-Id: Ic910ea042dfe50209f79928e61b3b34b4c734e73
Cert revocation was broken by
32b0adb591. os.chdir() never returns
anything, so this method would always raise an exception. The proper
way to handle an error from os.chdir() is to catch OSError.
There were existing tests for this code, but they conveniently mocked
os.chdir() to return values that are never actually returned. The
tests were fixed to match the real behavior.
Change-Id: I7549bb60a7d43d53d6f81eecea31cbb9720cc8b6
Closes-bug: #1376368
This patch removes the baremetal virt driver from nova, which was
deprecated in Juno. It does not aim to perform any of the refactoring
we need to do now that the driver is gone, but is intended as a clean
removal of all the baremetal code itself.
Of specific note are the api_samples tests. We didn't have samples to
test the ironic proxy code, so I removed the baremetal ones for show
and index, resetting that extension back to "untested" status as if it
was new.
Tests that stub out ironic interactions and re-introduce those tests
should be added after this patch.
Change-Id: Ia76e41a8a3b7230701872ae7a1975edc3d9ea847
Jenkins