[Teammetrics-discuss] Phase I: Updates

Sukhbir Singh sukhbir.in at gmail.com
Thu Jun 2 05:26:47 UTC 2011


Hi!

The points I am going to mention were discussed in: [0]. I am going to
take them up one by one.

First, please start by updating your local Git repository.

> So I would rather move the mboxes to /var/cache/teammetrics/liststats
> and the configuration file should be rather reside in  /etc/teammetrics

Done. I have selected /var/cache/teammetrics and /etc/teammetrics.

> I would prefer a better readable config file.

Done. I have implemented support for ConfigParser, which is compliant
to RFC 822 and the Debian standard.

So now you can create the config file in the format of:

[list-name]
url = <base-url>
lists = <either a single list> or <list 1>
    <list 2>
    <list n>

Note that you are not restricted to a single section (list-name). You
can have as many sections as you want, I have implemented support for
handling multiple sections and multiple lists.

You have to manually create the file for now but I plan to fix that
soon after we get other stuff sorted. PS: Suggestions for a config
file name are welcome.

> This is most probably because the list cdd-commits was stopped however,
> I also got this error, when replacing this list by something else - so
> something is broken here.

I have handled the exception but an important thing comes up.

Let's take the example of our own mailing list, teammetrics-discuss
[1]. If you notice, the archive for the month of June (which is the
current month) is in plain text and not a gzip archive. Of course I
can download the archive for the active month also but do we really
want to? Shouldn't we download it when the month completes? So my
question is -- do we measure performance for the active month the
script is run? Note that we will be implementing a system where the
same list is not parsed again, so we can probably parse the active
month later.

> Logfile would be probably a good idea.

Not done. I have plans to finish this by this weekend at most, given
everything else is acceptable to you :-) (I just have to replace the
print statements)

A few other questions:

- when the user runs the script for the first time, the directory and
the config files may not be there. How do you want to handle this?
Should I create the directory and a sample config file or ... ?
- as is expected, the script runs with root privileges. I have put in
a check for that (see `is_root()` in `liststat.py`). Is this
acceptable?

Your testing actually helped us make this a lot better! So please feel
free to test it as rigorously as you like.

I think that should cover up everything for now and I hope it
addresses all the concerns you had.

--
Regards,
Sukhbir.

[0] - http://lists.alioth.debian.org/pipermail/teammetrics-discuss/2011-May/000007.html
[1] - http://lists.alioth.debian.org/pipermail/teammetrics-discuss/



More information about the Teammetrics-discuss mailing list