| Message ID | 1455894797-88379-1-git-send-email-cardoe@cardoe.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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.
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
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.
> 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
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
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
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?
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..
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
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.
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.
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="
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(+)