Hello,

My patch [1] to fix it in Neutron is now merged. Thx Ihar and Kevin for
so fast review :)
It looks that now functional tests are working fine.

[1] https://review.openstack.org/#/c/426550/

-- 
Best regards / Pozdrawiam
Sławek Kapłoński
sla...@kaplonski.pl

On Sun, 29 Jan 2017, Sławek Kapłoński wrote:

> Hello,
> 
> I made small patch to Neutron to fix [1] this issue. It looks that
> functional tests are fine with it [2].
> Please review it if it makes sense for You :)
> 
> [1] https://review.openstack.org/#/c/426550/
> [2] 
> http://logs.openstack.org/50/426550/1/check/gate-neutron-dsvm-functional-ubuntu-xenial/4b56e71/console.html
> 
> -- 
> Best regards / Pozdrawiam
> Sławek Kapłoński
> sla...@kaplonski.pl
> 
> On Sun, 29 Jan 2017, Sławek Kapłoński wrote:
> 
> > Hello,
> > 
> > Thanks for explaining that. I'm don't know this part well and that's why
> > I asked here if it's oslo or neutron issue.
> > So it looks like it is issue in Neutron nor Oslo.context, right?
> > 
> > -- 
> > Best regards / Pozdrawiam
> > Sławek Kapłoński
> > sla...@kaplonski.pl
> > 
> > On Sat, 28 Jan 2017, Joshua Harlow wrote:
> > 
> > > Yup, all that should be needed is to read the docs and adjust it.
> > > 
> > > "once" - print only the first occurrence of matching warnings, regardless 
> > > of
> > > location
> > > 
> > > "always" - always print matching warnings
> > > 
> > > Changing it to 'once' should gather less repeated logs; IMHO 'always' 
> > > should
> > > never be used, so that was probably just a mistake for whoever committed
> > > that fixture with it set like that.
> > > 
> > > -Josh
> > > 
> > > Davanum Srinivas wrote:
> > > > Graham, Sławek,
> > > > 
> > > > Have you seen this?
> > > > 
> > > > Neutron has/uses a WarningFixture:
> > > > http://git.openstack.org/cgit/openstack/neutron/tree/neutron/tests/base.py#n161
> > > > http://git.openstack.org/cgit/openstack/neutron/tree/neutron/tests/tools.py#n70
> > > > 
> > > > Which uses filterwarnings:
> > > > http://git.openstack.org/cgit/openstack/neutron/tree/neutron/tests/tools.py#n80
> > > > 
> > > > Where "always" is being used:
> > > > https://docs.python.org/2/library/warnings.html
> > > > 
> > > > Which prints warnings every single time.
> > > > 
> > > > So, can we please "fix" it in there quickly?
> > > > 
> > > > Thanks,
> > > > Dims
> > > > 
> > > > On Sat, Jan 28, 2017 at 6:46 PM, Sławek Kapłoński<sla...@kaplonski.pl>  
> > > > wrote:
> > > > > Hello,
> > > > > 
> > > > > Thanks. I just filled bug report to oslo project:
> > > > > https://bugs.launchpad.net/oslo.context/+bug/1660088
> > > > > 
> > > > > --
> > > > > Best regards / Pozdrawiam
> > > > > Sławek Kapłoński
> > > > > sla...@kaplonski.pl
> > > > > 
> > > > > On Sat, 28 Jan 2017, Hayes, Graham wrote:
> > > > > 
> > > > > > On 28/01/17 20:54, Sławek Kapłoński wrote:
> > > > > > > Hello,
> > > > > > > 
> > > > > > > I pushed today patch in Neutron to gerrit [1] and I noticed that 
> > > > > > > functional
> > > > > > > tests are failing. In logs there is huge number of deprecation 
> > > > > > > warnings
> > > > > > > displayed [2]. Tests are failing because of global timeout is 
> > > > > > > reached and IMO
> > > > > > > reason of this timeout is this huge amount of deprecation 
> > > > > > > warnings.
> > > > > > > I checked that those warnings are disaplyed when 
> > > > > > > oslo.context==2.12.0 is
> > > > > > > installed. With oslo.context==2.11.0 all is fine.
> > > > > > > Should it be reported as bug in Neutron or in Oslo project? Or 
> > > > > > > maybe You
> > > > > > > already know about it and there is some patch to fix it but I 
> > > > > > > couldn't find it?
> > > > > > Its an oslo.context bug - I can't see one filed for it yet.
> > > > > > > [1] https://review.openstack.org/#/c/426429/
> > > > > > > [2] 
> > > > > > > http://logs.openstack.org/29/426429/3/check/gate-neutron-dsvm-functional-ubuntu-xenial/7079dc5/console.html
> > > > > > > 
> > > > > > It looks like the warnings function was miss used - this should be 
> > > > > > set
> > > > > > as a once only message. The stack trace level seems suspect, as it
> > > > > > points as neutron/context.py as the one sending out the notice, not
> > > > > > oslo.contexts base class.
> > > > > > 
> > > > > > [3] seems to the cause - I would suggest reverting this commit and
> > > > > > doing a 2.13.0 release. This is going to be *really* noisy. 
> > > > > > (logstash
> > > > > > has 2.5 million items for this string in the last 24 hours)
> > > > > > 
> > > > > > 3 -
> > > > > > https://github.com/openstack/oslo.context/commit/f25543fcc792ebf155728a91fde06e8dc4e96cea
> > > > > > 
> > > > > > __________________________________________________________________________
> > > > > > 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
> > > > > __________________________________________________________________________
> > > > > 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
> > > > > 
> > > > 
> > > > 
> > > > 
> > > 
> > > __________________________________________________________________________
> > > 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
> 
> 
> 
> > __________________________________________________________________________
> > 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
> 



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

Attachment: signature.asc
Description: PGP signature

__________________________________________________________________________
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