[ovs-dev] [PATCH] ovs-bugtool: Add ability to prioritize files by date.

Raju Subramanian rsubramanian at nicira.com
Fri Mar 23 21:58:24 UTC 2012


On Fri, Mar 23, 2012 at 2:39 PM, Ben Pfaff <blp at nicira.com> wrote:

> That's fine with.  Omitting such files would also be fine, I guess.
> But the new behavior appeared, at a glance, to be that we don't catch
> exceptions in os.stat() at all.  No?
>
>
No. I have added that check in the latest patch I sent.


> On Fri, Mar 23, 2012 at 02:37:21PM -0700, Raju Subramanian wrote:
> > The old behaviour was if os.stat() failed, it would still add the file to
> > the report but just wouldn't update the counter tracking the sizes. Do
> you
> > want to maintain this behaviour?
> >
> > On Fri, Mar 23, 2012 at 2:26 PM, Ben Pfaff <blp at nicira.com> wrote:
> >
> > > On Fri, Mar 23, 2012 at 02:22:23PM -0700, Raju Subramanian wrote:
> > > > When size limit is reached in the middle of processing a dir,
> > > > the report ends up containing oldest files. This change adds
> > > > an optional param in the plugin to prioritize newer files.
> > > >
> > > > Feature #9937
> > > > Requested-by: Ronald Lee <rlee at nicira.com>
> > > > Signed-off-by: Raju Subramanian <rsubramanian at nicira.com>
> > >
> > > The new version still doesn't catch exceptions in os.stat.
> > >
> > > Thanks,
> > >
> > > Ben.
> > >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20120323/d2f453a5/attachment-0003.html>


More information about the dev mailing list