yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #74157
[Bug 1785425] Re: tempest-full-py3 runs on test-only changes
Reviewed: https://review.openstack.org/589039
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=65cdcf4aaa9aec2fbeb29c9734b07efff32a2c49
Submitter: Zuul
Branch: master
commit 65cdcf4aaa9aec2fbeb29c9734b07efff32a2c49
Author: ghanshyam <gmann@xxxxxxxxxxxxxxxxx>
Date: Mon Aug 6 07:23:16 2018 +0000
Define irrelevant-files for tempest-full-py3 job
tempest-full-py3 job runs on nova check and gate pipeline
as part of 'integrated-gate-py35' template.
Unlike tempest-full or other job, nova job definition list on
project-config side[1] does not define the irrelevant-files for
tempest-full-py job which leads to run this job on doc/test file
only change also.
This commit defines the irrelevant-files for tempest-full-py3 on
nova's zuul.yaml.
Closes-Bug: #1785425
[1] https://github.com/openstack-infra/project-config/blob/57907fbf04c3f9a69d390efee800d42697faae16/zuul.d/projects.yaml#L9283
Change-Id: I887177c078a53c53e84289a6b134d7ea357dfbef
** Changed in: nova
Status: In Progress => Fix Released
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1785425
Title:
tempest-full-py3 runs on test-only changes
Status in OpenStack Compute (nova):
Fix Released
Bug description:
I'm not sure if this is due to the tempest-full rename to tempest-
full-py3 but it seems like this didn't used to be an issue. But we now
run tempest-full-py3 even on test-only changes, like this change:
https://review.openstack.org/#/c/588935/
My guess is we had this handled for tempest-full because of the
irrelevant-files list in project-config:
https://github.com/openstack-infra/project-
config/blob/3d9c5399fdb9b16d3d0391f1d4f32e904db16388/zuul.d/projects.yaml#L9369
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1785425/+subscriptions
References