On Thu, 2010-09-09 at 03:15 -0400, Kamil Paral wrote:
----- "James Laska" <jlaska(a)redhat.com> wrote:
>
> No objections here. Some additional data ... there are a few other
> issues lurking which I've not managed to triage/diagnose yet. They
> include ...
>
> 1. rpmguard test is sending emails even when no previous package
> is
> found -
>
>
https://fedorahosted.org/pipermail/autoqa-results/2010-September/032313.html
That is correct. Since AutoQATest parent class now handles email sending,
we send out test result every time. We can argue whether we want to have some
option to disable sending emails in certain cases, but I would say it's just
fine this way. Certainly in this case - the test passed and we want to know it.
Just summary and highlights are mixed up, I'll fix that in my forthcoming
patchset.
Ah okay ... that's the issue, that the subject and body of the message
are mixed. Something seemed weird and I couldn't pinpoint it :)
> 2. rpmguard run_once() crash -
>
>
http://autoqa.fedoraproject.org/results/581-autotest/qa04.c.fedoraproject...
A typo (not mine:)). I'll fix that in my patchset.
Thanks :)
> 5. repoclosure crashes -
>
>
http://autoqa.fedoraproject.org/results/554-autotest/qa04.c.fedoraproject...
That's this one -
https://fedorahosted.org/autoqa/ticket/214. I wonder what
we can do about it to fix this faster. It may take a long time until
new upstream yum-utils are releases and they get into Fedora and EPEL.
Do we want to create our own devel build from VCS?
I spoke to Seth on #fedora-devel regarding plans for a newer yum-utils
build. He isn't anticipating one soon, and recommended doing a
custom-build that includes the referenced upstream patch. I've filed a
ticket to remind me to take care of this ...
https://fedorahosted.org/autoqa/ticket/225
> I'll try to pinpoint these this week and send patches for
review.
> Once
> those are knocked out, maybe we can see where the multihook patchset
> is ... and adjust accordingly?
Don't bother with those rpmguard issues, I'll handle them. Thanks.
Okay, thanks Kamil!
Thanks,
James