xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [PATCH for-4.7] travis: drop broken LLVM repos
@ 2016-06-07 16:08 Doug Goldstein
  2016-06-07 16:17 ` Ian Jackson
                   ` (3 more replies)
  0 siblings, 4 replies; 11+ messages in thread
From: Doug Goldstein @ 2016-06-07 16:08 UTC (permalink / raw)
  To: xen-devel
  Cc: Doug Goldstein, Ian Jackson, Wei Liu, Jan Beulich, Andrew Cooper

LLVM repos are currently down so drop them from being installed so we
can get some testing back.
---
 .travis.yml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index d2e1bec..5bae60d 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -40,7 +40,6 @@ addons:
     apt:
         sources:
             - ubuntu-toolchain-r-test
-            - llvm-toolchain-precise
         packages:
             - zlib1g-dev
             - libncurses5-dev
@@ -70,7 +69,6 @@ addons:
             - gcc-aarch64-linux-gnu
             - gcc-5
             - g++-5
-            - clang-3.8
             - seabios
 # we must set CXX manually instead of using 'language: cpp' due to
 # travis-ci/travis-ci#3871
-- 
2.7.4 (Apple Git-66)


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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-07 16:08 [PATCH for-4.7] travis: drop broken LLVM repos Doug Goldstein
@ 2016-06-07 16:17 ` Ian Jackson
  2016-06-07 17:07   ` Doug Goldstein
  2016-06-07 16:20 ` Wei Liu
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 11+ messages in thread
From: Ian Jackson @ 2016-06-07 16:17 UTC (permalink / raw)
  To: Doug Goldstein; +Cc: Andrew Cooper, Wei Liu, Jan Beulich, xen-devel

Doug Goldstein writes ("[PATCH for-4.7] travis: drop broken LLVM repos"):
> LLVM repos are currently down so drop them from being installed so we
> can get some testing back.

This is unfortunate churn to be having in our tree.  I'm happy for
this removal commit to go in, but isn't there a way to deal with this
that doesn't involve updating a metadata file in xen.git ?

(This kind of external repo dependency is one reason why osstest likes
to fetch its inputs from a very small number of different places, with
many files and trees copied to xenbits.)

Ian.

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-07 16:08 [PATCH for-4.7] travis: drop broken LLVM repos Doug Goldstein
  2016-06-07 16:17 ` Ian Jackson
@ 2016-06-07 16:20 ` Wei Liu
  2016-06-07 17:09   ` Doug Goldstein
  2016-06-08 11:59 ` Jan Beulich
  2016-06-08 12:28 ` Doug Goldstein
  3 siblings, 1 reply; 11+ messages in thread
From: Wei Liu @ 2016-06-07 16:20 UTC (permalink / raw)
  To: Doug Goldstein
  Cc: Ian Jackson, Andrew Cooper, Wei Liu, Jan Beulich, xen-devel

On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
> LLVM repos are currently down so drop them from being installed so we
> can get some testing back.
> ---
>  .travis.yml | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index d2e1bec..5bae60d 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -40,7 +40,6 @@ addons:
>      apt:
>          sources:
>              - ubuntu-toolchain-r-test
> -            - llvm-toolchain-precise
>          packages:
>              - zlib1g-dev
>              - libncurses5-dev
> @@ -70,7 +69,6 @@ addons:
>              - gcc-aarch64-linux-gnu
>              - gcc-5
>              - g++-5
> -            - clang-3.8
>              - seabios
>  # we must set CXX manually instead of using 'language: cpp' due to
>  # travis-ci/travis-ci#3871
> -- 
> 2.7.4 (Apple Git-66)
> 

Can we just wait until llvm sorts itself out?

I don't think the breakage is permanent, right?

Wei.

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-07 16:17 ` Ian Jackson
@ 2016-06-07 17:07   ` Doug Goldstein
  0 siblings, 0 replies; 11+ messages in thread
From: Doug Goldstein @ 2016-06-07 17:07 UTC (permalink / raw)
  To: Ian Jackson; +Cc: Andrew Cooper, Wei Liu, Jan Beulich, xen-devel


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

On 6/7/16 12:17 PM, Ian Jackson wrote:
> Doug Goldstein writes ("[PATCH for-4.7] travis: drop broken LLVM repos"):
>> LLVM repos are currently down so drop them from being installed so we
>> can get some testing back.
> 
> This is unfortunate churn to be having in our tree.  I'm happy for
> this removal commit to go in, but isn't there a way to deal with this
> that doesn't involve updating a metadata file in xen.git ?
> 
> (This kind of external repo dependency is one reason why osstest likes
> to fetch its inputs from a very small number of different places, with
> many files and trees copied to xenbits.)
> 
> Ian.
> 

You can see it two ways. The tests for that specific version are always
available since they're part of the repo. But if external dependencies
go away then those tests are broken.

-- 
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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-07 16:20 ` Wei Liu
@ 2016-06-07 17:09   ` Doug Goldstein
  2016-06-07 17:34     ` Wei Liu
  0 siblings, 1 reply; 11+ messages in thread
From: Doug Goldstein @ 2016-06-07 17:09 UTC (permalink / raw)
  To: Wei Liu; +Cc: Andrew Cooper, Ian Jackson, Jan Beulich, xen-devel


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

On 6/7/16 12:20 PM, Wei Liu wrote:
> On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
>> LLVM repos are currently down so drop them from being installed so we
>> can get some testing back.
>> ---
>>  .travis.yml | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/.travis.yml b/.travis.yml
>> index d2e1bec..5bae60d 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -40,7 +40,6 @@ addons:
>>      apt:
>>          sources:
>>              - ubuntu-toolchain-r-test
>> -            - llvm-toolchain-precise
>>          packages:
>>              - zlib1g-dev
>>              - libncurses5-dev
>> @@ -70,7 +69,6 @@ addons:
>>              - gcc-aarch64-linux-gnu
>>              - gcc-5
>>              - g++-5
>> -            - clang-3.8
>>              - seabios
>>  # we must set CXX manually instead of using 'language: cpp' due to
>>  # travis-ci/travis-ci#3871
>> -- 
>> 2.7.4 (Apple Git-66)
>>
> 
> Can we just wait until llvm sorts itself out?
> 
> I don't think the breakage is permanent, right?
> 
> Wei.
> 

Its not permanent but it is now unknown when they will return. I figured
I'd toss this patch out there if the community would rather have some
build tests that pass instead of no build tests that run.

-- 
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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-07 17:09   ` Doug Goldstein
@ 2016-06-07 17:34     ` Wei Liu
  2016-06-08  9:28       ` Jan Beulich
  0 siblings, 1 reply; 11+ messages in thread
From: Wei Liu @ 2016-06-07 17:34 UTC (permalink / raw)
  To: Doug Goldstein
  Cc: Ian Jackson, Andrew Cooper, Wei Liu, Jan Beulich, xen-devel

On Tue, Jun 07, 2016 at 01:09:04PM -0400, Doug Goldstein wrote:
> On 6/7/16 12:20 PM, Wei Liu wrote:
> > On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
> >> LLVM repos are currently down so drop them from being installed so we
> >> can get some testing back.
> >> ---
> >>  .travis.yml | 2 --
> >>  1 file changed, 2 deletions(-)
> >>
> >> diff --git a/.travis.yml b/.travis.yml
> >> index d2e1bec..5bae60d 100644
> >> --- a/.travis.yml
> >> +++ b/.travis.yml
> >> @@ -40,7 +40,6 @@ addons:
> >>      apt:
> >>          sources:
> >>              - ubuntu-toolchain-r-test
> >> -            - llvm-toolchain-precise
> >>          packages:
> >>              - zlib1g-dev
> >>              - libncurses5-dev
> >> @@ -70,7 +69,6 @@ addons:
> >>              - gcc-aarch64-linux-gnu
> >>              - gcc-5
> >>              - g++-5
> >> -            - clang-3.8
> >>              - seabios
> >>  # we must set CXX manually instead of using 'language: cpp' due to
> >>  # travis-ci/travis-ci#3871
> >> -- 
> >> 2.7.4 (Apple Git-66)
> >>
> > 
> > Can we just wait until llvm sorts itself out?
> > 
> > I don't think the breakage is permanent, right?
> > 
> > Wei.
> > 
> 
> Its not permanent but it is now unknown when they will return. I figured
> I'd toss this patch out there if the community would rather have some
> build tests that pass instead of no build tests that run.

OK. I'm fine with taking this patch.

Wei.

> 
> -- 
> Doug Goldstein
> 




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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-07 17:34     ` Wei Liu
@ 2016-06-08  9:28       ` Jan Beulich
  2016-06-08 10:09         ` Dario Faggioli
  2016-06-08 11:44         ` Wei Liu
  0 siblings, 2 replies; 11+ messages in thread
From: Jan Beulich @ 2016-06-08  9:28 UTC (permalink / raw)
  To: Doug Goldstein, Wei Liu; +Cc: Andrew Cooper, Ian Jackson, xen-devel

>>> On 07.06.16 at 19:34, <wei.liu2@citrix.com> wrote:
> On Tue, Jun 07, 2016 at 01:09:04PM -0400, Doug Goldstein wrote:
>> On 6/7/16 12:20 PM, Wei Liu wrote:
>> > On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
>> >> LLVM repos are currently down so drop them from being installed so we
>> >> can get some testing back.
>> >> ---
>> >>  .travis.yml | 2 --
>> >>  1 file changed, 2 deletions(-)
>> >>
>> >> diff --git a/.travis.yml b/.travis.yml
>> >> index d2e1bec..5bae60d 100644
>> >> --- a/.travis.yml
>> >> +++ b/.travis.yml
>> >> @@ -40,7 +40,6 @@ addons:
>> >>      apt:
>> >>          sources:
>> >>              - ubuntu-toolchain-r-test
>> >> -            - llvm-toolchain-precise
>> >>          packages:
>> >>              - zlib1g-dev
>> >>              - libncurses5-dev
>> >> @@ -70,7 +69,6 @@ addons:
>> >>              - gcc-aarch64-linux-gnu
>> >>              - gcc-5
>> >>              - g++-5
>> >> -            - clang-3.8
>> >>              - seabios
>> >>  # we must set CXX manually instead of using 'language: cpp' due to
>> >>  # travis-ci/travis-ci#3871
>> >> -- 
>> >> 2.7.4 (Apple Git-66)
>> >>
>> > 
>> > Can we just wait until llvm sorts itself out?
>> > 
>> > I don't think the breakage is permanent, right?
>> > 
>> > Wei.
>> > 
>> 
>> Its not permanent but it is now unknown when they will return. I figured
>> I'd toss this patch out there if the community would rather have some
>> build tests that pass instead of no build tests that run.
> 
> OK. I'm fine with taking this patch.

Question then is - do we want this on just 4.7, or also unstable?

Jan


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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-08  9:28       ` Jan Beulich
@ 2016-06-08 10:09         ` Dario Faggioli
  2016-06-08 11:44         ` Wei Liu
  1 sibling, 0 replies; 11+ messages in thread
From: Dario Faggioli @ 2016-06-08 10:09 UTC (permalink / raw)
  To: Jan Beulich, Doug Goldstein, Wei Liu
  Cc: Andrew Cooper, Ian Jackson, xen-devel


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

On Wed, 2016-06-08 at 03:28 -0600, Jan Beulich wrote:
> >>> On 07.06.16 at 19:34, <wei.liu2@citrix.com> wrote:
> > On Tue, Jun 07, 2016 at 01:09:04PM -0400, Doug Goldstein wrote:
> 
> >> Its not permanent but it is now unknown when they will return. I
> figured
> >> I'd toss this patch out there if the community would rather have
> some
> >> build tests that pass instead of no build tests that run.
> > 
> > OK. I'm fine with taking this patch.
> 
> Question then is - do we want this on just 4.7, or also unstable?
> 
Both, I think?

So that, AFAIU the issue, people using travis will have both their 4.7
_and_ unstable build staring being tested again?

Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-08  9:28       ` Jan Beulich
  2016-06-08 10:09         ` Dario Faggioli
@ 2016-06-08 11:44         ` Wei Liu
  1 sibling, 0 replies; 11+ messages in thread
From: Wei Liu @ 2016-06-08 11:44 UTC (permalink / raw)
  To: Jan Beulich
  Cc: Wei Liu, xen-devel, Doug Goldstein, Ian Jackson, Andrew Cooper

On Wed, Jun 08, 2016 at 03:28:56AM -0600, Jan Beulich wrote:
> >>> On 07.06.16 at 19:34, <wei.liu2@citrix.com> wrote:
> > On Tue, Jun 07, 2016 at 01:09:04PM -0400, Doug Goldstein wrote:
> >> On 6/7/16 12:20 PM, Wei Liu wrote:
> >> > On Tue, Jun 07, 2016 at 12:08:52PM -0400, Doug Goldstein wrote:
> >> >> LLVM repos are currently down so drop them from being installed so we
> >> >> can get some testing back.
> >> >> ---
> >> >>  .travis.yml | 2 --
> >> >>  1 file changed, 2 deletions(-)
> >> >>
> >> >> diff --git a/.travis.yml b/.travis.yml
> >> >> index d2e1bec..5bae60d 100644
> >> >> --- a/.travis.yml
> >> >> +++ b/.travis.yml
> >> >> @@ -40,7 +40,6 @@ addons:
> >> >>      apt:
> >> >>          sources:
> >> >>              - ubuntu-toolchain-r-test
> >> >> -            - llvm-toolchain-precise
> >> >>          packages:
> >> >>              - zlib1g-dev
> >> >>              - libncurses5-dev
> >> >> @@ -70,7 +69,6 @@ addons:
> >> >>              - gcc-aarch64-linux-gnu
> >> >>              - gcc-5
> >> >>              - g++-5
> >> >> -            - clang-3.8
> >> >>              - seabios
> >> >>  # we must set CXX manually instead of using 'language: cpp' due to
> >> >>  # travis-ci/travis-ci#3871
> >> >> -- 
> >> >> 2.7.4 (Apple Git-66)
> >> >>
> >> > 
> >> > Can we just wait until llvm sorts itself out?
> >> > 
> >> > I don't think the breakage is permanent, right?
> >> > 
> >> > Wei.
> >> > 
> >> 
> >> Its not permanent but it is now unknown when they will return. I figured
> >> I'd toss this patch out there if the community would rather have some
> >> build tests that pass instead of no build tests that run.
> > 
> > OK. I'm fine with taking this patch.
> 
> Question then is - do we want this on just 4.7, or also unstable?
> 

Both IMO.

Wei.

> Jan
> 

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-07 16:08 [PATCH for-4.7] travis: drop broken LLVM repos Doug Goldstein
  2016-06-07 16:17 ` Ian Jackson
  2016-06-07 16:20 ` Wei Liu
@ 2016-06-08 11:59 ` Jan Beulich
  2016-06-08 12:28 ` Doug Goldstein
  3 siblings, 0 replies; 11+ messages in thread
From: Jan Beulich @ 2016-06-08 11:59 UTC (permalink / raw)
  To: Doug Goldstein; +Cc: Andrew Cooper, Wei Liu, Ian Jackson, xen-devel

>>> On 07.06.16 at 18:08, <cardoe@cardoe.com> wrote:
> LLVM repos are currently down so drop them from being installed so we
> can get some testing back.
> ---

I was about to commit this, but you didn't provide an S-o-b.

Jan


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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH for-4.7] travis: drop broken LLVM repos
  2016-06-07 16:08 [PATCH for-4.7] travis: drop broken LLVM repos Doug Goldstein
                   ` (2 preceding siblings ...)
  2016-06-08 11:59 ` Jan Beulich
@ 2016-06-08 12:28 ` Doug Goldstein
  3 siblings, 0 replies; 11+ messages in thread
From: Doug Goldstein @ 2016-06-08 12:28 UTC (permalink / raw)
  To: xen-devel; +Cc: Ian Jackson, Wei Liu, Jan Beulich, Andrew Cooper


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

On 6/7/16 12:08 PM, Doug Goldstein wrote:
> LLVM repos are currently down so drop them from being installed so we
> can get some testing back.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>

-- 
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

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2016-06-08 12:28 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-07 16:08 [PATCH for-4.7] travis: drop broken LLVM repos Doug Goldstein
2016-06-07 16:17 ` Ian Jackson
2016-06-07 17:07   ` Doug Goldstein
2016-06-07 16:20 ` Wei Liu
2016-06-07 17:09   ` Doug Goldstein
2016-06-07 17:34     ` Wei Liu
2016-06-08  9:28       ` Jan Beulich
2016-06-08 10:09         ` Dario Faggioli
2016-06-08 11:44         ` Wei Liu
2016-06-08 11:59 ` Jan Beulich
2016-06-08 12:28 ` Doug Goldstein

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).