Move nova-cells-v1 to experimental queue

Cells v1 has been deprecated since Pike. CERN
has been running with cells v2 since Queens.
The cells v1 job used to be the only thing that
ran with nova-network, but we switched the job
to use neutron in Rocky:

  I9de6b710baffdffcd1d7ab19897d5776ef27ae7e

The cells v1 job also suffers from intermittent
test failures, like with snapshot tests.

Given the deprecated nature of cells v1 we should
just move it to the experimental queue so that it
can be run on-demand if desired but does not gate
on all nova changes, thus further moving along its
eventual removal.

This change also updates the cells v1 status doc
and adds some documentation about the different
job queues that nova uses for integration testing.

Change-Id: I74985f1946fffd0ae4d38604696d0d4656b6bf4e
Closes-Bug: #1807407
This commit is contained in:
Matt Riedemann 2018-12-07 10:59:37 -05:00
parent 4d8baef8dc
commit e02fbb53d5
4 changed files with 26 additions and 3 deletions

View File

@ -205,7 +205,6 @@
# code; we don't need to run this on all changes, nor do we run # code; we don't need to run this on all changes, nor do we run
# it in the gate. # it in the gate.
- ^(?!nova/network/.*)(?!nova/virt/libvirt/vif.py).*$ - ^(?!nova/network/.*)(?!nova/virt/libvirt/vif.py).*$
- nova-cells-v1
- nova-live-migration - nova-live-migration
- nova-lvm - nova-lvm
- nova-multiattach - nova-multiattach
@ -225,7 +224,6 @@
jobs: jobs:
- neutron-grenade: - neutron-grenade:
irrelevant-files: *dsvm-irrelevant-files irrelevant-files: *dsvm-irrelevant-files
- nova-cells-v1
- nova-live-migration - nova-live-migration
- nova-multiattach - nova-multiattach
- nova-tox-functional - nova-tox-functional
@ -255,6 +253,7 @@
irrelevant-files: *dsvm-irrelevant-files irrelevant-files: *dsvm-irrelevant-files
- tempest-pg-full: - tempest-pg-full:
irrelevant-files: *dsvm-irrelevant-files irrelevant-files: *dsvm-irrelevant-files
- nova-cells-v1
- nova-tempest-full-oslo.versionedobjects: - nova-tempest-full-oslo.versionedobjects:
irrelevant-files: *dsvm-irrelevant-files irrelevant-files: *dsvm-irrelevant-files
- legacy-tempest-dsvm-nova-libvirt-kvm-apr: - legacy-tempest-dsvm-nova-libvirt-kvm-apr:

View File

@ -85,7 +85,16 @@ Integration tests
----------------- -----------------
Today, our integration tests involve running the Tempest test suite on a Today, our integration tests involve running the Tempest test suite on a
variety of Nova deployment scenarios. variety of Nova deployment scenarios. The integration job setup is defined
in the ``.zuul.yaml`` file in the root of the nova repository. Jobs are
restricted by queue:
* ``check``: jobs in this queue automatically run on all proposed changes even
with non-voting jobs
* ``gate``: jobs in this queue automatically run on all approved changes
(voting jobs only)
* ``experimental``: jobs in this queue are non-voting and run on-demand by
leaving a review comment on the change of "check experimental"
In addition, we have third parties running the tests on their preferred Nova In addition, we have third parties running the tests on their preferred Nova
deployment scenario. deployment scenario.

View File

@ -57,6 +57,12 @@ Because of this, there are a few restrictions placed on cells v1:
tests are added to Tempest which trigger a latent bug in cells v1 it may not tests are added to Tempest which trigger a latent bug in cells v1 it may not
be fixed. However, regressions in working function should be tracked with be fixed. However, regressions in working function should be tracked with
bugs and fixed. bugs and fixed.
#. Changes proposed to nova will not be automatically tested against a Cells v1
environment. To manually trigger Cells v1 integration testing on a nova
change in Gerrit, leave a review comment of "check experimental" and the
*nova-cells-v1* job will run on it, but the job is non-voting, meaning if
it fails it will not prevent the patch from being merged if approved by the
core team.
**Suffice it to say, new deployments of cells v1 are not encouraged.** **Suffice it to say, new deployments of cells v1 are not encouraged.**

View File

@ -0,0 +1,9 @@
---
other:
- |
CI testing of Cells v1 has been moved to the ``experimental`` queue meaning
changes proposed to nova will not be tested against a Cells v1 setup unless
explicitly run through the ``experimental`` queue by leaving a review
comment on the patch of "check experimental". Cells v1 has been deprecated
since the 16.0.0 Pike release and this is a further step in its eventual
removal.