[8/2/13 11:14:04 AM] Matthew Mosesohn: Andrey, I forgot to add info about
yum makecache progress
[8/2/13 11:14:30 AM] Matthew Mosesohn: you know makecache has started when
/var/cache/yum/x86_64/6Server is created
[8/2/13 11:15:15 AM] Matthew Mosesohn: then subfolder rhel-6-server-rpms
will grow first

[8/2/13 11:15:20 AM] Matthew Mosesohn: not yet
[8/2/13 11:15:26 AM] Matthew Mosesohn: just wanted to mention this part
[8/2/13 11:18:52 AM] Matthew Mosesohn: rhel-6-server-rpms will grow first
to 125mb
[8/2/13 11:19:03 AM] Matthew Mosesohn: optional 14mb, and then the others
are 10mb or under
[8/2/13 11:19:30 AM] Matthew Mosesohn: it's not a small amount of data and
it takes up almost 1/2 the download time

Can be made by separated pull request.


On Fri, Aug 2, 2013 at 11:36 AM, Evgeniy L <notifications@github.com> wrote:

> In lib/astute/logparser.rb:
>
> > +          end
> > +      end
> > +
> > +      private
> > +      def recursive_size(path, opts={})
> > +        path << '/' unless path.end_with?('/')
> > +
> > +        File.size?(path) if not File.directory?(path)
> > +
> > +        total_size = 0
> > +        Dir["#{path}**/*"].each do |f|
> > +          # Option :files_only used when you want to calculate total size of
> > +          # regular files only. The default :files_only is false, so the function will
> > +          # includes to total value inode size of each dir (4096 bytes in mos cases) as
> > +          # unix util 'du' does it.
> > +          total_size += File.size?(f).to_i if File.file?(f) || ! opts[:files_only]
>
> Why not?
> s/File.size?(f).to_i/File.size(f)/
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Mirantis/astute/pull/43/files#r5548035>
> .
>



-- 
Mike Scherbakov