[PATCH] travis: add IRC notifications

[PATCH] travis: add IRC notifications

[Doug Goldstein]

This will cause failed builds and when the build flips back to success
to be reported to #xentest on FreeNode. The syntax of the message will
be:
<travis-ci>	xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
<travis-ci>	Change view :
https://github.com/xen-project/xen/compare/RANGE
<travis-ci>	Build details :
https://travis-ci.org/xen-project/xen/builds/BUILDID

The blob was generated with the following command:
travis encrypt -r xen-project/xen 'chat.freenode.net#xentest'

The reason it is encrypted is to prevent people that fork the repo to
spam #xentest. This value will only properly decrypt when running within
the xen-project/xen space.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
---
CC: Ian Campbell <ian.campbell@citrix.com>
CC: Ian Jackson <ian.jackson@eu.citrix.com>
CC: Jan Beulich <jbeulich@suse.com>
CC: Keir Fraser <keir@xen.org>
CC: Tim Deegan <tim@xen.org>
---
 .travis.yml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/.travis.yml b/.travis.yml
index 189d568..3118600 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -79,3 +79,7 @@ script:
       make CC="${CROSS_COMPILE}${CC}" HOSTCC="${CC}" dist )
 after_script:
     - cat xen/.config
+notifications:
+    irc:
+        channels:
+            - secure: "mPIFllF6eW3F3talvccMy55Tfcid66IPkkXZYCxDKRF2DQrMyvmg4qt0xN6gGZsdfOBMNr+/YfO5PxusBCUkVdBGBzd3QhFoIDYZbJZgzVh3yNDQ+x4L7p1cZNrwJ2loMmSX6KxGKZxZX9NRStrTUkVyp0jGZB9xkwT8Rl6jXj7EQkgQ95K1Wqafx0ycLfyDQmzX9bzi/3KIBFKMGmK18AFMh+R30zK0FPUUsS4+VhepIkVqO5puU3OYePd34wRnWlt7hjU2Vj5vYmVXp3UOE+E8/Lf9IGVAhitDi+EC35b8zo2BHJ9z6xZARYPvfSqbXcXV20RycabI+e3ufZJ40eatssly5QjWH+HhKS42C4gV1psmQhkTCNCM62Ty5uf6R1hsZJQuiOZrc8ojdje8ey2MxJk4R+Xz+Igg1/kD6+WX9/Y6Y3iRuj5HL1xCYfpTbK4mC7ofw0SofW2aAGI68jHpCqJdQCDzMl6748PlDMM0eKe0MPKIEenYHcoBnOEC/jciXUDa6wduV75EEip7oq2i+m44MopcsEDTpdliH077GhKapF0ActjvBTLpyoTRSfkKm0NZol/dgwd3PGG/mY8clIoeXWRb4opk93ejPC967KmSNC68SlfwaJmFZS5T9vAgb6k7r6i9G3dmYtrLKzws8IV1CPWqLzk58+v4pRk="
-- 
2.5.4 (Apple Git-61)

Re: [PATCH] travis: add IRC notifications

[Ian Jackson]

Doug Goldstein writes ("[PATCH] travis: add IRC notifications"):
> This will cause failed builds and when the build flips back to success
> to be reported to #xentest on FreeNode. The syntax of the message will
> be:
> <travis-ci>	xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
> <travis-ci>	Change view :
> https://github.com/xen-project/xen/compare/RANGE
> <travis-ci>	Build details :
> https://travis-ci.org/xen-project/xen/builds/BUILDID

I would like to give a little while for the community to comment on
whether this is what they want.  Personally I think it's a reasonable
use of #xentest.

I'm tempted to suggest that osstest reports should go there too :-).

CCing Lars.

Ian.

Re: [PATCH] travis: add IRC notifications

[Andrew Cooper]

On 22/02/16 17:06, Ian Jackson wrote:
> Doug Goldstein writes ("[PATCH] travis: add IRC notifications"):
>> This will cause failed builds and when the build flips back to success
>> to be reported to #xentest on FreeNode. The syntax of the message will
>> be:
>> <travis-ci>	xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
>> <travis-ci>	Change view :
>> https://github.com/xen-project/xen/compare/RANGE
>> <travis-ci>	Build details :
>> https://travis-ci.org/xen-project/xen/builds/BUILDID
> I would like to give a little while for the community to comment on
> whether this is what they want.  Personally I think it's a reasonable
> use of #xentest.

+1

~Andrew

Re: [PATCH] travis: add IRC notifications

[Doug Goldstein]

[-- Attachment #1.1: Type: text/plain, Size: 868 bytes --]

On 2/22/16 11:06 AM, Ian Jackson wrote:
> Doug Goldstein writes ("[PATCH] travis: add IRC notifications"):
>> This will cause failed builds and when the build flips back to success
>> to be reported to #xentest on FreeNode. The syntax of the message will
>> be:
>> <travis-ci>	xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
>> <travis-ci>	Change view :
>> https://github.com/xen-project/xen/compare/RANGE
>> <travis-ci>	Build details :
>> https://travis-ci.org/xen-project/xen/builds/BUILDID
> 
> I would like to give a little while for the community to comment on
> whether this is what they want.  Personally I think it's a reasonable
> use of #xentest.
> 
> I'm tempted to suggest that osstest reports should go there too :-).
> 
> CCing Lars.
> 
> Ian.
> 

I'd like to see osstest reports go there as well.

-- 
Doug Goldstein


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 959 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH] travis: add IRC notifications

[Lars Kurth]

> On 22 Feb 2016, at 18:34, Doug Goldstein <cardoe@cardoe.com> wrote:
> 
> On 2/22/16 11:06 AM, Ian Jackson wrote:
>> Doug Goldstein writes ("[PATCH] travis: add IRC notifications"):
>>> This will cause failed builds and when the build flips back to success
>>> to be reported to #xentest on FreeNode. The syntax of the message will
>>> be:
>>> <travis-ci>	xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
>>> <travis-ci>	Change view :
>>> https://github.com/xen-project/xen/compare/RANGE
>>> <travis-ci>	Build details :
>>> https://travis-ci.org/xen-project/xen/builds/BUILDID
>> 
>> I would like to give a little while for the community to comment on
>> whether this is what they want.  Personally I think it's a reasonable
>> use of #xentest.
>> 
>> I'm tempted to suggest that osstest reports should go there too :-).
>> 
>> CCing Lars.
>> 
>> Ian.
>> 
> 
> I'd like to see osstest reports go there as well.

I don't have an issue with this, but as Ian pointed out, we should leave
some time for comments and then decide.

Lars

Re: [PATCH] travis: add IRC notifications

[Konrad Rzeszutek Wilk]

On Mon, Feb 22, 2016 at 06:39:26PM +0000, Lars Kurth wrote:
> 
> > On 22 Feb 2016, at 18:34, Doug Goldstein <cardoe@cardoe.com> wrote:
> > 
> > On 2/22/16 11:06 AM, Ian Jackson wrote:
> >> Doug Goldstein writes ("[PATCH] travis: add IRC notifications"):
> >>> This will cause failed builds and when the build flips back to success
> >>> to be reported to #xentest on FreeNode. The syntax of the message will
> >>> be:
> >>> <travis-ci>	xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
> >>> <travis-ci>	Change view :
> >>> https://github.com/xen-project/xen/compare/RANGE
> >>> <travis-ci>	Build details :
> >>> https://travis-ci.org/xen-project/xen/builds/BUILDID
> >> 
> >> I would like to give a little while for the community to comment on
> >> whether this is what they want.  Personally I think it's a reasonable
> >> use of #xentest.
> >> 
> >> I'm tempted to suggest that osstest reports should go there too :-).
> >> 
> >> CCing Lars.
> >> 
> >> Ian.
> >> 
> > 
> > I'd like to see osstest reports go there as well.
> 
> I don't have an issue with this, but as Ian pointed out, we should leave
> some time for comments and then decide.

Now that twitter can do more than 140 characters perhaps there as well :-)


> 
> Lars
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel

Re: [PATCH] travis: add IRC notifications

[George Dunlap]

On Mon, Feb 22, 2016 at 7:03 PM, Konrad Rzeszutek Wilk
<konrad.wilk@oracle.com> wrote:
> On Mon, Feb 22, 2016 at 06:39:26PM +0000, Lars Kurth wrote:
>>
>> > On 22 Feb 2016, at 18:34, Doug Goldstein <cardoe@cardoe.com> wrote:
>> >
>> > On 2/22/16 11:06 AM, Ian Jackson wrote:
>> >> Doug Goldstein writes ("[PATCH] travis: add IRC notifications"):
>> >>> This will cause failed builds and when the build flips back to success
>> >>> to be reported to #xentest on FreeNode. The syntax of the message will
>> >>> be:
>> >>> <travis-ci>       xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
>> >>> <travis-ci>       Change view :
>> >>> https://github.com/xen-project/xen/compare/RANGE
>> >>> <travis-ci>       Build details :
>> >>> https://travis-ci.org/xen-project/xen/builds/BUILDID
>> >>
>> >> I would like to give a little while for the community to comment on
>> >> whether this is what they want.  Personally I think it's a reasonable
>> >> use of #xentest.
>> >>
>> >> I'm tempted to suggest that osstest reports should go there too :-).
>> >>
>> >> CCing Lars.
>> >>
>> >> Ian.
>> >>
>> >
>> > I'd like to see osstest reports go there as well.
>>
>> I don't have an issue with this, but as Ian pointed out, we should leave
>> some time for comments and then decide.
>
> Now that twitter can do more than 140 characters perhaps there as well :-)

Just what we need -- the Register retweeting Xen, saying "CORE
DEVELOPERS BUNGLE COMMIT, BREAK BUILD!!!!"

:-)

 -George

Re: [PATCH] travis: add IRC notifications

[Doug Goldstein]

[-- Attachment #1.1.1: Type: text/plain, Size: 1773 bytes --]

On 2/23/16 3:33 AM, George Dunlap wrote:
> On Mon, Feb 22, 2016 at 7:03 PM, Konrad Rzeszutek Wilk
> <konrad.wilk@oracle.com> wrote:
>> On Mon, Feb 22, 2016 at 06:39:26PM +0000, Lars Kurth wrote:
>>>
>>>> On 22 Feb 2016, at 18:34, Doug Goldstein <cardoe@cardoe.com> wrote:
>>>>
>>>> On 2/22/16 11:06 AM, Ian Jackson wrote:
>>>>> Doug Goldstein writes ("[PATCH] travis: add IRC notifications"):
>>>>>> This will cause failed builds and when the build flips back to success
>>>>>> to be reported to #xentest on FreeNode. The syntax of the message will
>>>>>> be:
>>>>>> <travis-ci>       xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
>>>>>> <travis-ci>       Change view :
>>>>>> https://github.com/xen-project/xen/compare/RANGE
>>>>>> <travis-ci>       Build details :
>>>>>> https://travis-ci.org/xen-project/xen/builds/BUILDID
>>>>>
>>>>> I would like to give a little while for the community to comment on
>>>>> whether this is what they want.  Personally I think it's a reasonable
>>>>> use of #xentest.
>>>>>
>>>>> I'm tempted to suggest that osstest reports should go there too :-).
>>>>>
>>>>> CCing Lars.
>>>>>
>>>>> Ian.
>>>>>
>>>>
>>>> I'd like to see osstest reports go there as well.
>>>
>>> I don't have an issue with this, but as Ian pointed out, we should leave
>>> some time for comments and then decide.
>>
>> Now that twitter can do more than 140 characters perhaps there as well :-)
> 
> Just what we need -- the Register retweeting Xen, saying "CORE
> DEVELOPERS BUNGLE COMMIT, BREAK BUILD!!!!"
> 
> :-)
> 
>  -George
> 

So before the Register starts reporting "Xen Project dying due to lack
of email response", does anyone have any +1 or -1 to contribute to this
change?

-- 
Doug Goldstein


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 959 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH] travis: add IRC notifications

[Konrad Rzeszutek Wilk]

On Mon, Feb 29, 2016 at 03:10:48PM -0600, Doug Goldstein wrote:
> On 2/23/16 3:33 AM, George Dunlap wrote:
> > On Mon, Feb 22, 2016 at 7:03 PM, Konrad Rzeszutek Wilk
> > <konrad.wilk@oracle.com> wrote:
> >> On Mon, Feb 22, 2016 at 06:39:26PM +0000, Lars Kurth wrote:
> >>>
> >>>> On 22 Feb 2016, at 18:34, Doug Goldstein <cardoe@cardoe.com> wrote:
> >>>>
> >>>> On 2/22/16 11:06 AM, Ian Jackson wrote:
> >>>>> Doug Goldstein writes ("[PATCH] travis: add IRC notifications"):
> >>>>>> This will cause failed builds and when the build flips back to success
> >>>>>> to be reported to #xentest on FreeNode. The syntax of the message will
> >>>>>> be:
> >>>>>> <travis-ci>       xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
> >>>>>> <travis-ci>       Change view :
> >>>>>> https://github.com/xen-project/xen/compare/RANGE
> >>>>>> <travis-ci>       Build details :
> >>>>>> https://travis-ci.org/xen-project/xen/builds/BUILDID
> >>>>>
> >>>>> I would like to give a little while for the community to comment on
> >>>>> whether this is what they want.  Personally I think it's a reasonable
> >>>>> use of #xentest.
> >>>>>
> >>>>> I'm tempted to suggest that osstest reports should go there too :-).
> >>>>>
> >>>>> CCing Lars.
> >>>>>
> >>>>> Ian.
> >>>>>
> >>>>
> >>>> I'd like to see osstest reports go there as well.
> >>>
> >>> I don't have an issue with this, but as Ian pointed out, we should leave
> >>> some time for comments and then decide.
> >>
> >> Now that twitter can do more than 140 characters perhaps there as well :-)
> > 
> > Just what we need -- the Register retweeting Xen, saying "CORE
> > DEVELOPERS BUNGLE COMMIT, BREAK BUILD!!!!"
> > 
> > :-)
> > 
> >  -George
> > 
> 
> So before the Register starts reporting "Xen Project dying due to lack
> of email response", does anyone have any +1 or -1 to contribute to this
> change?

+1 to the IRC bot!

-1 to the Twitter bot..

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH] travis: add IRC notifications

[George Dunlap]

On Mon, Feb 29, 2016 at 9:10 PM, Doug Goldstein <cardoe@cardoe.com> wrote:
> On 2/23/16 3:33 AM, George Dunlap wrote:
>> On Mon, Feb 22, 2016 at 7:03 PM, Konrad Rzeszutek Wilk
>> <konrad.wilk@oracle.com> wrote:
>>> On Mon, Feb 22, 2016 at 06:39:26PM +0000, Lars Kurth wrote:
>>>>
>>>>> On 22 Feb 2016, at 18:34, Doug Goldstein <cardoe@cardoe.com> wrote:
>>>>>
>>>>> On 2/22/16 11:06 AM, Ian Jackson wrote:
>>>>>> Doug Goldstein writes ("[PATCH] travis: add IRC notifications"):
>>>>>>> This will cause failed builds and when the build flips back to success
>>>>>>> to be reported to #xentest on FreeNode. The syntax of the message will
>>>>>>> be:
>>>>>>> <travis-ci>       xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER)
>>>>>>> <travis-ci>       Change view :
>>>>>>> https://github.com/xen-project/xen/compare/RANGE
>>>>>>> <travis-ci>       Build details :
>>>>>>> https://travis-ci.org/xen-project/xen/builds/BUILDID
>>>>>>
>>>>>> I would like to give a little while for the community to comment on
>>>>>> whether this is what they want.  Personally I think it's a reasonable
>>>>>> use of #xentest.
>>>>>>
>>>>>> I'm tempted to suggest that osstest reports should go there too :-).
>>>>>>
>>>>>> CCing Lars.
>>>>>>
>>>>>> Ian.
>>>>>>
>>>>>
>>>>> I'd like to see osstest reports go there as well.
>>>>
>>>> I don't have an issue with this, but as Ian pointed out, we should leave
>>>> some time for comments and then decide.
>>>
>>> Now that twitter can do more than 140 characters perhaps there as well :-)
>>
>> Just what we need -- the Register retweeting Xen, saying "CORE
>> DEVELOPERS BUNGLE COMMIT, BREAK BUILD!!!!"
>>
>> :-)
>>
>>  -George
>>
>
> So before the Register starts reporting "Xen Project dying due to lack
> of email response", does anyone have any +1 or -1 to contribute to this
> change?

So at this point you had two positive responses (one from AndyC and
one from IanJ) and no negative responses (to the IRC bot) over a week
and a half; people have had the opportunity to object if they want to
now, so according to the "lazy consensus" principle you can just say
"OK, nobody objected, so I'm going to do it now." :-)

 -George

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH] travis: add IRC notifications

[Ian Jackson]

Konrad Rzeszutek Wilk writes ("Re: [Xen-devel] [PATCH] travis: add IRC notifications"):
> On Mon, Feb 29, 2016 at 03:10:48PM -0600, Doug Goldstein wrote:
> > So before the Register starts reporting "Xen Project dying due to lack
> > of email response", does anyone have any +1 or -1 to contribute to this
> > change?
> 
> +1 to the IRC bot!

I'm happy with it too, as I said.  I will queue Doug's patch at the
head of this thread.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH] travis: add IRC notifications

[Ian Jackson]

Doug Goldstein writes ("Re: [PATCH] travis: add IRC notifications"):
> I'd like to see osstest reports go there as well.

I will implement this.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel