This is really because I wanted to be able to copy/paste from
object_hashes.txt in a subsequent patch and didn't want to introduce
unrelated ordering changes.
Change-Id: I064a52ebd17488334f4ecb88eaae69703a101ae6
This addresses review comments from the following changes:
I61a3e8902a891bac36911812e4e7c080570e3850
I48e6db9693e470b177bf4c75211d8b883c768433
Ic70d2bb781b6a844849a5cf2fe4d271b5a81093d
I5a956513f3485074023e027430cc52ee7a3f92e4
Ica6152ccb97dce805969d964d6ed032bfe22a33f
Part of blueprint bandwidth-resource-provider
Change-Id: Idffaa6d206cda3f507e6be095356537f22302ad7
Add a new microversion that removes support for the aforementioned
argument, which cannot be adequately guaranteed in the new placement
world.
Change-Id: I2a395aa6eccad75a97fa49e993b0300bdcfc7258
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
Implements: blueprint remove-force-flag-from-live-migrate-and-evacuate
APIImpact
This patch lays the underground work for supporting the
``all-tenants`` filter.
Related to blueprint handling-down-cell
Change-Id: I7dcef20aed0178c81d6580aa9534288eaa383dab
This patch adds the plumbing required to ignore the value of the
``[api]/list_records_by_skipping_down_cells`` config from the new
microversion if cell_down_support is True. This config will be
considered only if cell_down_support is False in which case we
look at the [api]/list_records_by_skipping_down_cells and
accordingly skip the records from the down cells or generate an
API exception as the response.
The description of list_records_by_skipping_down_cells is updated
in Id9f12532897912b39093f63e9286540d9029edeb when the microversion
is added.
Related to blueprint handling-down-cell
Change-Id: Icbe27c941c9b934f8f1894e9b9da1d34f047e942
1) Change it to accommodate querying it for 'None' project_ids
in the "--all-tenats" case.
2) If the online data migration for populating queued_for_delete
has not been run for some reason, the the values could be NULL
in the database for instance_mapping.queued_for_delete. Under
such circumstances, we assume that such mappings with NULL
queued_for_delete have *not* being queued_for_deletion.
Related to blueprint handling-down-cell
Change-Id: I80a65bc026e26a272a9dc041b27f9839511db765
Now that os_traits provides symbols for each trait, we can refer to
traits by symbol rather than by string. This gives us compile-time
checking for free, in order to ensure that nova's use of traits lines
up with what os_traits provides.
Change-Id: Id1461c444f0f67b29e0a6a10181267ef1d1d8bc0
This patch adds two new fields to the RequestGroup ovo, requester_id and
provider_uuids. These two fields are needed to be able to hold and
communicate the mapping between the requester of the RequestGroup (e.g.
Neutron port) and the resource providers that are fulfilling the request
(e.g. network device RPs). If the RequestGroup represents the un-numbered
group then more than one RP can fulfill the request hence provider_uuids
is a list.
These new fields later in the series will be populated based on some
logic in the nova-conductor. However in the long run we expect that
these fields will be populated from the Placement allocation
candidates response.
blueprint bandwidth-resource-provider
Change-Id: Ic4735f92542e5e0ca36b459874dc486f6b360317
This patch collects the resource requests from each neutron port
involved in a server create request. Converts each request to
a RequestGroup object and includes them in the RequestSpec.
This way the requests are reaching the scheduler and there
they are included in the generation of the allocation_candidates
query.
This patch only handles the happy path of a server create request. But
it adds couple of TODOs to places where the server move operations
related code paths need to be implemented. That implementation will be
part of subsequent patches.
Note that this patch technically makes it possible to boot server with
one neutron port that has resource request. But it does not handle
multiple such ports or SRIOV ports where two PFs are supporting the
same physnet as well as many server lifecycle operations like resize,
migrate, live-migrate, unshelve. To avoid possible resource allocation
inconsistencies due to the partial support nova rejects any requests
that involves such ports. See the previous patches in this patch
series for details.
Also note that the simple boot cases are verified with functional tests
and in those tests we need to mock out the above described logic that
reject such requests. See a more background about this approach on the
ML [1].
[1] http://lists.openstack.org/pipermail/openstack-discuss/2018-December/001129.html
blueprint bandwidth-resource-provider
Change-Id: Ica6152ccb97dce805969d964d6ed032bfe22a33f
This resolves the following deprecation warning:
b'/home/zuul/src/git.openstack.org/openstack/nova/.tox/functional-py35/
lib/python3.5/site-packages/oslo_service/threadgroup.py:193:
DeprecationWarning: Calling add_timer() with arguments to the callback
function is deprecated. Use add_timer_args() instead.'
The add_timer_args method was added in 1.34.0:
Ib2791342263e2b88c045bcc92adc8160f57a0ed6
So the required version of oslo.service is also updated.
Change-Id: Id54226dc926839686906d04ecf8d791c0881f82a
Partial-Bug: #1813147
This adds a warnings filter to ignore the "inspect.getargspec"
warnings from sqlalchemy-migrate until bug 1814288 is fixed
in that library.
Change-Id: Id8c02e745742980d3dd0a24e6e2b7a5859f3cb88
Related-Bug: #1813147
Since change I81301eeecc7669a169deeb1e2c5d298a595aab94 in
devstack, nova-cpu.conf is a copy of nova.conf minus
database access. Grenade jobs also run nova-compute with
nova-cpu.conf anyway so we can just drop the conditional
which was otherwise messing up the config file that the
ceph script would write rbd configuration which is why
live block migration tests with ceph were failing.
While in here, the zuul job configuration is updated so
that changes to nova/tests/live_migration/ can be
self-testing.
Change-Id: I902e459093af9b82f9033d58cffcb2a628f5ec39
Closes-Bug: #1813216
The first section in the doc mentions the properties of
a flavor but didn't mention description which was added
in 2.55 so this adds it.
Change-Id: Id73a2665e7fa914e00dc60a085a7cd9f47655a73
This fixes the link, re-words it a bit, moves it to the main
description (since it applies to PUT also) and drops the note
since we don't need note formatting for linking in reference
material.
Closes-Bug: #1814953
Change-Id: Ia24cda353bdcadf3fe8405aac588e8abf1100608
.pyc files are ignored by nova's .gitignore, as a result
if you change branches after the .pyc files are generated
for example from stable/ocata to master it does not remove
the .pyc files.
This can lead to the _validate_bytecode_header function that
is invoked as part of sphinx-build to fail as the .pyc
files no longer match the contents of the checked out files.
This change adds {[testenv]commands} to the docs envs
to clean up stale .pyc files before running sphinx-build
Change-Id: I536a260da639d32afbc998be220305f9489db375
- This change updates the admin flavor docs
to reflect the use of osc to update flavor
descriptions
- This change documents that modifcations to
flavor extra_specs are not reflected in an
instance's embedded flavor.
Change-Id: I0f206d9db70465d8ce6b1404f546f3e00eeb6e23
This object has a default=False setting for queued_for_delete, but never
actually sets that value. All newly created records should have a non-NULL
value for this field, and we have a migration to fix them, so this
change explicitly forces that =False, unless the object is being created
with a value set.
Closes-Bug: #1814913
Change-Id: I99c5cc24c7e9bf5e2e72ffc868990b87b0e8e3f8
os-vif got bumped to 1.14.0 in https://review.openstack.org/572082 and
Nova is touching the VIFPortProfileOVSRepresentor class in a supported
fashion until at least Train.
However, that commit contributes to triggering bug 1813147 in check and
gate. The warning is safe to silence and should be converted to an error
in Train (once the older path is removed).
Change-Id: I9d8175c78f068cf9bdad4aeba0941f6ff49a9acc
Signed-off-by: Jan Gutter <jan.gutter@netronome.com>
Related-Bug: #1813147
blueprint: generic-os-vif-offloads
Zuul