Age | Commit message (Collapse) | Author | Files | Lines |
|
Sometime lavacli wait exit with an error.
Until the LAVA/lavacli bug is fixed, let's try to handle the situation.
Since the error seems to happen when the job end, let's just check if
the job ended, in the other case retry to wait.
Change-Id: I7b7b1aee0dcc2ecc2ef971cc40c29374307f6b0f
Bug-AGL: SPEC-3705
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: If44134e2b036493db636571dc89b51be4ff8f53d
|
|
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: Ie7955e06765a72ce6ef3f30d590df34a07dff226
|
|
Change-Id: Ic30c9838a307cb074a669c2373a76d4f250533bd
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
Change-Id: I6cb8ce991e03ce7656d76eb7b82832f6bdd82a7c
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
This reverts commit d212724c5b1fbf323b71ad09ca2d069b15b27995.
Reason for revert: this broke other target boards result evaluation.
Change-Id: I01ddd2b6790eedd767e34630ba587ad507a3d8d0
|
|
The RPI4 fail often in uboot phase due to network timeouts.
This patchs adds a way to restart failled jobs for rpi4.
For the moment we restart all failed jobs, a subsequent patch will add a
check on error case.
Bug-AGL: SPEC-3004
Change-Id: I556ec30d1af4adfa57ce3553f79f4152d335e158
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
Lavacli wait has a bug and returns
Submitted.
Running......................................................
Unable to call 'jobs.wait': [Errno 2] No such file or directory
Avoid this failing the job as the following evaluation will still work.
Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: If8fa942b7ddc49857d2f010f02ae3f53d01476ac
|
|
lava.read-feedback is currently returned by default on qemu jobs.
This is a known issue upstream and fixed in a later version.
Thus ignore a single fail of 'lava.read-feedback'.
Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I2e6c3c779c87ec539d631b129a452998a0ec70f3
|
|
If GERRIT_HOST is empty fallback to gerrit.automotivelinux.org
Bug-AGL: SPEC-2903
Change-Id: Iced5435293c9fc0c3d260e3e27b2f15e03d9ea89
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
This will allow to post more detailed messages back to gerrit.
Note the double quotes used and the eval before the ssh to enforce the expansion.
Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I1c0d618796ef1cb697529575f7b9925ae4926a97
|
|
This fixes the application builds where we have no skiplist for most.
Before this we bailed out on return 0 and did never parse the result.
Now the result parsing is it's own function and always being called.
Bug-AGL: SPEC-2882
.
Change-Id: I58c05fa48dd9121e0eeb494975513dcc88290dc5
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
Bug-AGL: SPEC-2703
Change-Id: I223a4242942ac9e751f0b6dd3caab0309f2a566a
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
This patchs adds a link to the console logs when sending result.
Bug-AGL: SPEC-2703
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
Change-Id: Idb95e5cdf80072c4cfdce95ecc4facabffb43caf
|
|
TESTNAME was mispelled in the debug output.
Change-Id: I068da47c52626672f6e307349e95e03d3b7b20c3
Bug-AGL: SPEC-2703
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
The raw testline contains an * which breaks the parsing of the testname.
This patchs adds quote to prevent that.
Change-Id: I8f11b3d9ee5444983dd886954a5e2712025a7314
Bug-AGL: SPEC-2703
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
The existence test is inverted, fix it.
Change-Id: Ida3104e34b3475c80b206091464b1e9a88d8be49
Bug-AGL: SPEC-2703
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
The lavacli results does not provide the identity argument and so does
not work.
This patch adds the missing "-i $lab" for made it works.
While at it, this patch converts two hardcoded "-i agl" call to use the correct
lab variable.
As asked during the meeting, let's add a sanity check that this lab
variable is set.
Bug-AGL: SPEC-2703
Change-Id: I9432f8b27aa7157e08b48f5caeb905a18152fa6a
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
|
|
This patch move the grepping test inside the if for not failling due to
set -xe
For the same reason, harden the code checking if git clone was
successfull.
Change-Id: Idb0619c4e90c9a3a0654a024fbd2166e49df98c6
Bug-AGL: SPEC-2703
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
Change-Id: I1ec20a66dfc4a427642806011f6de8d4e7c75408
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
This patch fix the following problems:
- {{GERRIT_CHANGE_NUMBER}},${{GERRIT_PATCHSET_NUMBER}}: bad substitution
A copy paste from jenkins commands, but I forgot to remove the double {}
- missing grep -v '\*', removed by ec3e1dc57d7a6a454f9ed1f0210b07b8270a1fbb ("Follow up fix")
- for apps, no dedicated skipdir as asked in SPEC-2703
Change-Id: I8dfdc6ff4d03fc946e0b8ad35f092e217afd685b
Bug-AGL: SPEC-2703:
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
Change-Id: I13c0e82a53a19d4343eb2659d1c86eea399d94c3
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
do not fail on CIBT result evaluation.
Change-Id: I91e84cc9e30a149f8e56c218b1f97375c76c3d95
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
The service weston does not start on qemu/qemuarm64.
for having this service check skipped only for thoses two machine, we
need to have a per-machine skiplist.
In the process, we add the possibiity to have a per GERRIT_PROJECT
skiplist.
Furthermore, I "migrate" the skiplist for apps in a skiplists directory.
Change-Id: I989b2bdf2463272ee1f407a933be49a2f4219d5f
Bug-AGL: SPEC-2703
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
This patchs implement pass/fail analysis of job tests.
All failling test are compared to a skiplist for removing known failling
tests.
If it exists any remaining failling test after the skiplist phase, the
job is set as fail in gerrit.
Bug-AGL: SPEC-2703
Change-Id: I4af8f54caaeef616c529b626ca8010e7159d8206
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
Instead of using a timeout to manage the polling part, this patch
replace it with a lavacli command which is 'lavacli jobs wait'that will
wait for the job to finish, but after 2 hours the 'job wait' will be
stopped even the job if not finished, we are estimating after a 2 hours
of waiting, a problem is occurring in the job.
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>
Change-Id: Ic81a635b0cc2dc5dda3028aefb78f88be0d79a31
|
|
This is needed to avoid jenkins polling until the timeout when a job
was aborted.
Change-Id: Ibcf0fffaeaac86a1198e4f0411e784ff74ab9f5b
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
Remove the set -x and print JOB_URL prominently.
Change-Id: I81b2283557e5b2ca5ca7815adb43f4e930250357
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
After managing to send the email test report after all the job have been
finished, this patch remove the part to send email when submitting the job.
Bug-AGL: SPEC-2177
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>
Change-Id: I477ae5513cc1d9c5dcba9b2b71bb6bc38cbbbb99
|
|
Remove the leading space from the email recipient when requesting an
email report. The leading space was causing the SMTP handling on the
backend to reject the request.
Change-Id: Id176728a7c6eea8f54128d74cf658b13b6be81cb
Signed-off-by: Kevin Hilman <khilman@baylibre.com>
|
|
This patch fixes the KCI_API link, it should be an 'http' not 'httpis'.
Change-Id: Iad52da5de124289ac15563b3bf10d652ef76bbe1
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>
|
|
Fix the logic to check if the email auth token is present.
While at it, make output more verbose for easier debug.
Signed-off-by: Kevin Hilman <khilman@baylibre.com>
(cherry picked from commit b9f7f8928ab26f15f0950d00a7f2417ffa204755)
Change-Id: Iceb00e5617cbfb787353c7caf05e5d00e9512d70
|
|
This patch avoid to fail the job if the sending email report section is
not working well.
Change-Id: I9bc192e1ac8f3937fc06be15233b0a697d16ff15
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>
|
|
The 'LAVA_MASTER_URI' used is wrong, this patch fixes the url.
It adds also a check on the output of lavacli command before to send the
test email report.
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>
(cherry picked from commit d891414424913e7bcf2845740cd5c532a858877e)
Change-Id: Iff4b984593abb5ccb495764246826bcf173e6e6e
|
|
|
|
and adapt the common job to not use '' .
Change-Id: I15b4fa1a4ab181c55c53885fb746cfd56403c698
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
When using --yaml option to show the job information the 'Health' is no
longer with capital 'H'. This patch change the 'Health" to 'health'.
Change-Id: Ifd463b4aee9ca90bbaa017e2f409a5105cec3e93
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>
|
|
Change-Id: I10c292096dae34936b8dcbfbc8d383a21ed02bfd
Bug-AGL: SPEC-2267
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
TODO: rewrite to run under set -e !
Change-Id: I4f26d8e5de03d2a16a6b8c8825ce7bc9df8da5d4
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
|
|
this patch adds a missing "then"
Change-Id: Iaddfc12597864e3ba35a12baa6b0716660d969b3
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
|
|
In order to move totally to lavacli, this patch:
- Apply changes to submit jobs using lavacli. As the blocking call
(--block) no longer exists with lavacli, so had to replace that with
a polling loop.
- Replace lava-tool with lavacli in the polling part in the labs-prepare
file, which checks the device status and availability.
Different changes tested at Baylibre.
Change-Id: I25b1ad4b47ae09a158e1e1cf42e30da5e6aa8b57
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>
|
|
Add new feature to the ci-managmenet that will request to send test
report email to the specified email addresses in the automotive-
testreports list after a certain delay (1 hour=3600 sec).
This commit is accomplished based on the work of Corentin Labbe
<clabbe@baylibre.com>.
Change-Id: I14350b13b5b6cf2550303999ced56055a392212b
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>
|
|
Fix "raspberrypi3-uboot" -> "raspberrypi3-uboot-uboot" type of
substitutions.
Ensure that the full string (up to EOL) is matched before trying to
replace.
Change-Id: I2026beb964b12aff1f60441257ceec82b706bab7
Signed-off-by: Kevin Hilman <khilman@baylibre.com>
|
|
|
|
Do a second grep on the lava log to include lava error message if there
was any. This is usefull in case a job fails. The error can be read
directly out of the jenkins log.
JIRA: SPEC-1178
Change-Id: Id3333df9c56dd67f5d664ae0d234e33e3300c366
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>
|
|
when using grep + tr to find board availability, status, a race
condition happens sometimes. When it happens the job is stuck and
timeout fails.
Fix it by calling sequentially grep then tr and not piping.
Change-Id: Ib1322e280900b62abdd827992b42183024cd094c
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>
|
|
Fix the query of the log in jenkins log. Use the lava website url and not the
lava API url.
JIRA: SPEC-1178
Change-Id: I9ddc48b13e91a91aa2e53c768aa02f2ae12e5a99
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>
|
|
lava-tool submit-jobs go back to --block as wait-for-jobs isn't working
as expected. Revert the change until wait-for-jobs works.
But keep this version of the tool to have a link to the lava-job posted
in the log when using submit-job.
Change-Id: Ie7d045d198b00b19d198404fc67b2bcfe992e9a6
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>
|
|
The previous version would fail to find some boards if the device names
were different between lava, AGL, releng.
This version creates a device alias table. And uses this table with the
correct name for each component (AGL, lava, releng).
Change-Id: Ief064c5ff9edb2d824e80549148ee0ab40ebcddf
Signed-off-by: Loys Ollivier <lollivier@baylibre.com>
|