Mailing-List: contact dev-help@ant.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Ant Developers List" <dev@ant.apache.org>
From: =?utf-8?Q?Nicolas_Lalev=C3=A9e?= <nicolas.lalevee@hibnet.org>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
Subject: Re: Ivy - Buildjobs/PreCommit
Date: Wed, 24 May 2017 15:12:03 +0200
References: <002101d2cee0$dd7f2860$987d7920$@de>
 <002201d2cee6$4581cc60$d0856520$@de>
 <FBE56A25-30F8-4EDA-A715-B8C84F7F77F7@gmail.com>
 <4855C9F1-EA94-40EC-878B-ABECD06B60EE@gmail.com>
 <002901d2cef9$15d64c30$4182e490$@de>
 <464CA5A1-BA4A-4140-8FFF-C01AAB731563@gmail.com>
 <002a01d2cf06$c69d0ba0$53d722e0$@de>
 <AE0536D5-83E3-4A3A-A6D5-26F10A1B88D6@hibnet.org>
 <572A4ADA-12FA-40BF-B2C5-11EF74403715@gmail.com>
 <6E627468-0085-4F43-BB4F-3AEA085AEFCE@hibnet.org>
 <7BD92912-71E5-4897-8854-CE0C06EFD615@gmail.com>
 <41D64985-6B5D-4F23-8F7E-693EE30F76D9@hibnet.org>
 <003d01d2d2c8$56513360$02f39a20$@de>
 <A9AE9A62-B2AF-417D-9BE9-9A6F1E547A84@gmail.com>
 <005d01d2d47e$c32056d0$49610470$@de>
To: Ant Developers List <dev@ant.apache.org>
In-Reply-To: <005d01d2d47e$c32056d0$49610470$@de>
Message-Id: <6C6EAEE6-09FE-49F3-9ED8-EDEB64D4ED22@hibnet.org>
archived-at: Wed, 24 May 2017 13:12:18 -0000


> Le 24 mai 2017 =C3=A0 13:13, Jan Mat=C3=A8rne (jhm) =
<apache@materne.de> a =C3=A9crit :
>=20
> Not sure about this expertise because my sources were old, but I think =
the github-plugin does not support matrix jobs.
> https://github.com/KostyaSha/github-integration-plugin/issues/53
>=20
> Maybe a newer plugin version would do, but I could not see the version =
stamp.
> I'll ask on builds@a.o for help.
>=20
> For the meanwhile I deactivated the matrix-PR-job and reactivated the =
'old' plain-PR-job.
>=20
> If we can't get matrix support we should be to create our own 'matrix' =
by copying that job and sticking to certain configurations.
> While a matrix would scale this approach won't. So we should define =
only 2-4 most interesting configs like
> - Java7@Unix
> - Java7@Windows
> - Java8@Unix

I think we can even stick to only two configs. Testing on two different =
OS is the real pain and I don=E2=80=99t remember having big issues with =
jvm versions.

We could even for now stick to Windows if most committers are on Unix =
machines. At least I am.

And in the end we still have the more exhaustive matrix build which runs =
after the commit being merged into the asf repo.

Nicolas

>=20
>=20
> Jan
>=20
>> -----Urspr=C3=BCngliche Nachricht-----
>> Von: J Pai [mailto:jai.forums2013@gmail.com]
>> Gesendet: Montag, 22. Mai 2017 14:45
>> An: Ant Developers List
>> Betreff: Re: Ivy - Buildjobs/PreCommit
>>=20
>> It looks like there might be some issue with the matrix job(s)
>> https://builds.apache.org/job/Ivy-GithubPR-Matrix/2/console
>>=20
>> =46rom what I understand of that log, it seems like the parent matrix =
job
>> (the one above) is correctly checking out the commit from the =
submitted
>> PR:
>>=20
>>> Checking out Revision c1765c71ba4394597409de790bf4529ff97e866e
>> (master)  > git config core.sparsecheckout # timeout=3D10  > git =
checkout
>> -f c1765c71ba4394597409de790bf4529ff97e866e
>>> git rev-parse origin/master^{commit} # timeout=3D10  > git rev-list
>> 239bc0b04a95d6c0f82e59c513de36d68c033c6a # timeout=3D10
>>=20
>> However, it then hands off the job execution to individual
>> Windows/Linux jobs and those jobs https://builds.apache.org/job/Ivy-
>> GithubPR-Matrix/jdk=3DJDK%201.7%20(latest),os=3Dubuntu/2/console =
don=E2=80=99t seem
>> to be able to checkout the commit(s) from the PR and fail with error
>> like:
>>=20
>>> Fetching upstream changes from https://github.com/apache/ant-ivy/
>>=20
>>> git --version # timeout=3D10
>>> git fetch --tags --progress
>> https://github.com/apache/ant-ivy/
>> +refs/heads/*:refs/remotes/origin/*
>> Checking out Revision c1765c71ba4394597409de790bf4529ff97e866e =
(master)
>>> git config core.sparsecheckout # timeout=3D10  > git checkout -f
>> c1765c71ba4394597409de790bf4529ff97e866e
>> FATAL: Could not checkout c1765c71ba4394597409de790bf4529ff97e866e
>>=20
>> hudson.plugins.git.GitException
>> : Command "git checkout -f c1765c71ba4394597409de790bf4529ff97e866e"
>> returned status code 128:
>> stdout:
>> stderr: fatal: reference is not a tree:
>> c1765c71ba4394597409de790bf4529ff97e866e
>>=20
>> 	at
>> =
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPI
>> Impl.java:1866)
>>=20
>>=20
>> -Jaikiran
>>=20
>> On 22-May-2017, at 12:25 PM, Jan Mat=C3=A8rne (jhm) =
<apache@materne.de>
>> wrote:
>>=20
>>>> On a related note, while we are at this - does Apache infra allow
>> the
>>> jobs to be run against Windows OS Jenkins agents as well? There are =
a
>>> few issues specifically reported against Windows OS and having the
>> job
>>> run against linux and Windows OS should give a decent coverage for
>> the
>>> upstream code.
>>>=20
>>> In Jenkins there is the concept of =C2=AB Matrix =C2=BB job.
>>> Here is the ones for Ant:
>>> =
https://builds.apache.org/view/A/view/Ant/job/Ant-Build-Matrix-1.9.x-
>>> Windows/ <https://builds.apache.org/view/A/view/Ant/job/Ant-Build-
>>> Matrix-1.9.x-Windows/>
>>> =
https://builds.apache.org/view/A/view/Ant/job/Ant-Build-Matrix-1.9.x-
>>> Linux/
>>> <https://builds.apache.org/view/A/view/Ant/job/Ant-Build-Matrix-
>>> 1.9.x-Linux/>
>>> AFAIR, it used to be able to do on several different OS. I don=E2=80=99=
t know
>>> why they are split.
>>>=20
>>> And I don=E2=80=99t know if it is possible to do it while pulling a =
PR from
>>> github.
>>=20
>>=20
>> New buildjob "Ivy-GithubPR-Matrix". I deactivated the old (so we =
could
>> easily switch back).
>> https://builds.apache.org/view/A/view/Ant/job/Ivy-GithubPR-Matrix/
>> Basically a copy of the old job:
>> * discard old build jobs (keep 10)
>> * git merge support: fail the build if push fails
>> * github project: git://github.com/apache/ant-ivy.git
>> * git
>> -- repository: git://github.com/apache/ant-ivy.git
>> -- branches: */master (=3D default)
>> -- clean before checkout
>> -- recursively update submodules
>> -- build commits submitted for validated merge (=3D PR support)
>> * start build: build pull request to the repository (=3D PR support)
>> * configuration matrix
>> -- JDK: JDK 1.7 (latest), JDK 1.8 (latest), JDK 9 b156 with jigsaw
>> -- os: label expression =3D ["ubuntu", "Windows"]
>> * abort the build if it's stuck: absolute 20 minutes
>> * Lock: Ant
>> * build
>> -- ant(standard): clean coverage-report
>> * publish junit: build/test-report/**
>>=20
>>=20
>> Jan
>>=20
>>=20
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org For additional
>> commands, e-mail: dev-help@ant.apache.org
>>=20
>>=20
>>=20
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org For additional
>> commands, e-mail: dev-help@ant.apache.org
>=20
>=20
>=20
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
>=20


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org