Eric Fried <openst...@fried.cc> writes: > For my part, I tried it [1] and it doesn't seem to have worked. (The > functional test failure is what the dep is supposed to have fixed.) Did > I do something wrong? > > [1] https://review.openstack.org/#/c/533821/12
If you examine the "items:" section in this file: http://logs.openstack.org/21/533821/12/check/openstack-tox-functional/9066bb2/zuul-info/inventory.yaml You will see that Zuul collected the following changes to test together: 526541,19 533808,6 521098,29 521187,29 535463,3 536624,3 536625,4 537648,5 533821,12 All on the master branch of nova. The change you specified, "https://review.openstack.org/#/c/536545/" is not present. The reason is that, contrary to earlier replies in this thread, the /#/c/ version of the change URL does not work. I'm sure we can fix that, but for the moment, we'll need to use the permalink form. -Jim __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev