Re: [Minios-devel] automation: Creating a patchwork instance to improve pre-commit build testing

[Lars Kurth <lars.kurth@xxxxxxxxxx>]

On 24/07/2018, 11:50, "Julien Grall" <julien.grall@xxxxxxx> wrote:

    Hi Lars,
    
    On 24/07/18 11:33, Lars Kurth wrote:
    > 
    > On 24/07/2018, 11:19, "Wei Liu" <wei.liu2@xxxxxxxxxx> wrote:
    >      On Tue, Jul 24, 2018 at 04:04:05AM -0600, Jan Beulich wrote:
    >      > I'm afraid my personal bar for any such automation is pretty
    >      > high: There must not ever be any negative effect from such an
    >      > addition. Positive effects would of course be very welcome. I
    >      > realize this is an unrealistic goal, but it should at least come
    >      > close (perhaps after some initial learning phase). But this implies
    >      > that at least in theory it is possible to come close in the first
    >      > place, which I can't take for given with the information I've been
    >      > provided so far.
    >      
    >      Then I'm afraid the only suggestion I get for you at the moment is to
    >      add a filter to dump those emails to /dev/null -- you already 
realised
    >      that's an unrealistic goal (at least at the beginning).
    >      
    >      Wei.
    >      
    > First of all, there should only be mail (aka spam) if there was a failure.
    
    This seems a little strange to only send e-mail on failure. How do you 
    differentiate between the bot has successfully tested that series and 
    the series is still in queue then?
    
Yes, that would be a trade-off to minimize "spam"

It seems to me there are a number of options we have and thus some decisions
that need to be made.
 
1: Do we trigger a CI cycle for *every* patch?
2: Do we have an opt-in or op-out (e.g. through a tag, a specific CC, etc.) for 
patches
3: Do we report results back to xen-devel or to a separate list
Looking at Linux 0 day, they report failures to a separate list - see 
https://lists.01.org/pipermail/kbuild-all/2018-July/thread.html
They also only seem to report failures

I am not quite sure what QEMU does. But I can't see any bot messages on their 
list archives

4: Who else, besides the author should get a mail
The patch submitters should definitely get a mail, the question is whether 
people on the CC list should also get one

5: Do we report back on success or only on failure?
See question by Julien

6: What exactly do we report back
Aka what is in the actual mail


 


_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel