On 06/22/2016 12:55 PM, Lucas Alvares Gomes wrote:
Hi,
On Wed, Jun 22, 2016 at 10:53 AM, Sam Betts (sambetts)
<[email protected]> wrote:
This patch https://review.openstack.org/#/c/324909/ merged last night and
has broken the IPA functional tests.
To verify pull master and run "tox -r -e func" and it¹ll fail to run. If
you git checkout the commit before that one merged the same thing passes
successfully.
Seeing this error has made me realise that we don¹t have a CI job to run
these functional tests on IPA so this isn¹t caught and highlighted in
gerrit for reviewers, is this on purpose or should we add a new one to
prevent this happening again?
I would say we should add a job to the IPA gate to verify the
functional tests, just like swift does [0].
We may also need to revert/fix that patch that broke those tests.
I'm -1 to reverting anything until we have the test in the gate.
[0]
https://github.com/openstack-infra/project-config/blob/bd54f0127ee1a8da985f7fc6644e91b11f8f5f09/zuul/layout.yaml#L12108
Cheers,
Lucas
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: [email protected]?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: [email protected]?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
