Oh, no I didn't. By the time I got around to it I saw 264 and figured that
would cover the issue.

On Mon, Jul 13, 2015 at 7:33 PM, Robert Collins <robe...@robertcollins.net>
wrote:

> On 10 July 2015 at 22:07, Kevin Benton <blak...@gmail.com> wrote:
> > No prob. The fixes for Neutron were relatively trivial.
> > https://review.openstack.org/#/c/200420/
> >
> > The only one that was a bit surprising was the failure of autospec in
> this
> > file:
> >
> https://review.openstack.org/#/c/200420/4/neutron/tests/unit/services/metering/agents/test_metering_agent.py
> >
> > It was coming back with argument count mismatch failures. It seems like
> the
> > log decorator[1] trips up autospec in the new version.
> >
> > 1.
> >
> https://github.com/openstack/neutron/blob/master/neutron/services/metering/drivers/noop/noop_driver.py#L22
>
> Did you file a bug for that on mock? I don't recall seeing one.
>
> https://github.com/testing-cabal/mock/issues/264 has a patch upstream
> and it works on mock itself, so I'm applying that to cPython and
> backporting now.
>
> -Rob
>
> __________________________________________________________________________
> 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
>



-- 
Kevin Benton
__________________________________________________________________________
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

Reply via email to