[openstack-dev] [ironic] [ironic-python-agent] Broken functional tests

Dmitry Tantsur dtantsur at redhat.com
Wed Jun 22 11:01:45 UTC 2016

On 06/22/2016 12:55 PM, Lucas Alvares Gomes wrote:
> Hi,
> On Wed, Jun 22, 2016 at 10:53 AM, Sam Betts (sambetts)
> <sambetts at cisco.com> 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: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

More information about the OpenStack-dev mailing list