From 17b3bae64ca7aa4c1a6d296860cce6c0bfbb38a1 Mon Sep 17 00:00:00 2001 From: Matt Riedemann Date: Thu, 28 Jun 2018 12:12:53 -0400 Subject: [PATCH] Use ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa in tree ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa is defined in the ironic repo but nova uses it because it's defined in the project-config repo with it's own irrelevant-files list. Due to a recent change in zuul [1], the definition from project-config overrides the irrelevant-files from the job definition in ironic and results in us running this job on things like docs-only changes in nova, which is not awesome. This change moves the job usage in-tree from project-config and updates the irrelevant-files list to match nova-dsvm-base. Since project-config is branchless, we'll have to backport this in-tree change to nova's stable branches to continue running this job on nova stable branches once the override is removed from project-config. [1] http://lists.openstack.org/pipermail/openstack-dev/2018-June/131304.html Change-Id: I9435fe84c8dc8a6f35ffb195ddbf71ca34edb242 Closes-Bug: #1776981 --- .zuul.yaml | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/.zuul.yaml b/.zuul.yaml index 2d6a2abb8ae5..dca9da0c1cce 100644 --- a/.zuul.yaml +++ b/.zuul.yaml @@ -169,6 +169,26 @@ # Please try to keep the list of job names sorted alphabetically. check: jobs: + # We define our own irrelevant-files so we don't run the job + # on things like nova docs-only changes. + - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa: + voting: false + # This irrelevant-files list should match nova-dsvm-base. + # TODO(mriedem): Define a node anchor to deduplicate this. + irrelevant-files: + - ^(placement-)?api-.*$ + - ^(test-|)requirements.txt$ + - ^.*\.rst$ + - ^.git.*$ + - ^doc/.*$ + - ^nova/hacking/.*$ + - ^nova/locale/.*$ + - ^nova/tests/.*$ + - ^releasenotes/.*$ + - ^setup.cfg$ + - ^tests-py3.txt$ + - ^tools/.*$ + - ^tox.ini$ - nova-cells-v1 - nova-live-migration - nova-lvm