From f9a6321c7bffefd7c18d0c4732539e1ae49ccb5c Mon Sep 17 00:00:00 2001 From: Matt Riedemann Date: Thu, 14 Mar 2019 10:05:23 -0400 Subject: [PATCH] Customize irrelevant-files for nova-live-migration job I noticed change Iea6288fe6d341ee92f87a35e0b0a59fe564ab96c was not running the nova-live-migration job even though it was making changes to nova/tests/live_migration/hooks/run_tests.sh. The reason is the nova-live-migration job irrelevant-files were excluding changes to nova/tests/*. This copies the nova-grenade-live-migration irrelevant-files list to the nova-live-migration job and defines it as a variable so it can be re-used in the nova-grenade-live-migration job definition. Change-Id: I753fda1a83b340f4699c049158e6744b099f55d8 --- .zuul.yaml | 37 +++++++++++++++++++------------------ 1 file changed, 19 insertions(+), 18 deletions(-) diff --git a/.zuul.yaml b/.zuul.yaml index dde16ec6939c..83969b600042 100644 --- a/.zuul.yaml +++ b/.zuul.yaml @@ -120,6 +120,24 @@ Config drive is forced on all instances. run: playbooks/legacy/nova-live-migration/run.yaml post-run: playbooks/legacy/nova-live-migration/post.yaml + irrelevant-files: &live-migration-irrelevant-files + # We define our own irrelevant-files because we need to run + # this if there are changes to nova/tests/live_migration/. + # TODO(mriedem): Move the nova/tests/live_migration/ scripts to gate/ + # so this job can use the standard dsvm-irrelevant-files. + - ^api-.*$ + - ^(test-|)requirements.txt$ + - ^.*\.rst$ + - ^.git.*$ + - ^doc/.*$ + - ^nova/hacking/.*$ + - ^nova/locale/.*$ + - ^nova/tests/(?!live_migration/).*$ + - ^releasenotes/.*$ + - ^setup.cfg$ + - ^tests-py3.txt$ + - ^tools/.*$ + - ^tox.ini$ - job: name: nova-lvm @@ -208,24 +226,7 @@ - openstack-dev/grenade - openstack-infra/devstack-gate - openstack/nova - irrelevant-files: - # We define our own irrelevant-files because we need to run - # this if there are changes to nova/tests/live_migration/. - # TODO(mriedem): Move the nova/tests/live_migration/ scripts to gate/ - # so this job can use the standard dsvm-irrelevant-files. - - ^api-.*$ - - ^(test-|)requirements.txt$ - - ^.*\.rst$ - - ^.git.*$ - - ^doc/.*$ - - ^nova/hacking/.*$ - - ^nova/locale/.*$ - - ^nova/tests/(?!live_migration/).*$ - - ^releasenotes/.*$ - - ^setup.cfg$ - - ^tests-py3.txt$ - - ^tools/.*$ - - ^tox.ini$ + irrelevant-files: *live-migration-irrelevant-files - project: # Please try to keep the list of job names sorted alphabetically.