> I can't say I'm too deeply versed in the code,  but it's enough to make
me wonder if we want to go that direction and avoid the issues altogether?

It's the nature of python that methods and modules can be added in runtime
and pylint can't do full analysis. That's why the best use of it - limited
list of checks applied to the last commit only. This is a way how nova and
sahara have it implemented. Non-voting gate job helps to find silly
mistakes that could barely be found by other ways.

Thanks,
Andrew.

On Fri, Oct 3, 2014 at 10:09 AM, Neal, Phil <phil.n...@hp.com> wrote:

> > From: Dina Belova [mailto:dbel...@mirantis.com]
> > On Friday, October 03, 2014 2:53 AM
> >
> > Igor,
> >
> > Personally this idea looks really nice to me, as this will help to avoid
> > strange code being merged and not found via reviewing process.
> >
> > Cheers,
> > Dina
> >
> > On Fri, Oct 3, 2014 at 12:40 PM, Igor Degtiarov
> > <idegtia...@mirantis.com> wrote:
> > Hi folks!
> >
> > I try too guess do we need in ceilometer checking new patches for
> > critical errors with pylint?
> >
> > As far as I know Nova and Sahara and others have such check. Actually
> > it is not checking of all project but comparing of the number of
> > errors without new patch and with it, and if diff is more then 0 then
> > patch are not taken.
>
> Looking a bit deeper it seems that Nova struggled with false positives and
> resorted to https://review.openstack.org/#/c/28754/ , which layers some
> historical checking of git on top of pylint's tendency to check only the
> latest commit. I can't say I'm too deeply versed in the code,  but it's
> enough to make me wonder if we want to go that direction and avoid the
> issues altogether?
>
> >
> > I have taken as pattern Sahara's solution and proposed a patch for
> > ceilometer:
> > https://review.openstack.org/#/c/125906/
> >
> > Cheers,
> > Igor Degtiarov
> >
> > _______________________________________________
> > OpenStack-dev mailing list
> > OpenStack-dev@lists.openstack.org
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >
> >
> >
> >
> > --
> > Best regards,
> > Dina Belova
> > Software Engineer
> > Mirantis Inc.
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev@lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
_______________________________________________
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to