Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

These are ostensibly JSON, but what they actually are is Jinja
templates whose *output* is JSON. In particular, they can contain
{# comments in this syntax #}, causing syntax-check to fail.

lqa will check that their output is valid JSON anyway, so I think
the ability to have comments is worth more than the syntax check.

Reviewed-by: Luis Araujo <luis.araujo@collabora.co.uk>
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2156

Mathieu Duponchelle authored 9 years ago and Sjoerd Simons committed 9 years ago

We no longer ship chaiwala-btrfs.

Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2158

Mathieu Duponchelle authored 9 years ago and Sjoerd Simons committed 9 years ago

mc-tool (among other things), wasn't installed on the LAVA instance.

This package is provided by telepathy-mission-control-5 , which
is correctly listed here:

https://wiki.apertis.org/QA/Test_Cases/apparmor-folks



As such, this commit adds it, and other dependencies listed on
the wiki, but absent in the yaml file.

Reviewed-by: Luis Araujo <luis.araujo@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2132

Emilio Pozuelo Monfort authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Emilio Pozuelo Monfort <emilio.pozuelo@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2131

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2117

Simon McVittie authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2116

Mathieu Duponchelle authored 9 years ago and Sjoerd Simons committed 9 years ago

And check the id could actually be looked up.

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2115

Luis Araujo authored 9 years ago and Sjoerd Simons committed 9 years ago

Signed-off-by: Luis Araujo <luis.araujo@collabora.co.uk>

Luis Araujo authored 9 years ago and Sjoerd Simons committed 9 years ago

This commit removes the {{arch}} variable from the image
names in the job template files.

Signed-off-by: Luis Araujo <luis.araujo@collabora.co.uk>

Luis Araujo authored 9 years ago and Sjoerd Simons committed 9 years ago

This commit removes the changes of an optional board
and image type for the minimal template.

Signed-off-by: Luis Araujo <luis.araujo@collabora.co.uk>

Luis Araujo authored 9 years ago and Sjoerd Simons committed 9 years ago

Added the following attributes to the job templates: image.date,
image.version, image.board and image.type.

These attributes can be used to filter LAVA tests results based
on their values.

Signed-off-by: Luis Araujo <luis.araujo@collabora.co.uk>

Luis Araujo authored 9 years ago and Sjoerd Simons committed 9 years ago

This commit reverts a previous change about testing for an optional
image type and board to work around the target wayland image url not
matching the image name. The issue is now properly fixed in the image
generation scripts.

Signed-off-by: Luis Araujo <luis.araujo@collabora.co.uk>

Luis Araujo authored 9 years ago and Sjoerd Simons committed 9 years ago

This commit changes the template-dir variable to the relative path
apertis-tests/templates/ , so the templates files can be found from
the same directory this repository is checked out.

Luis Araujo authored 9 years ago and Sjoerd Simons committed 9 years ago

This commit updates all the test templates and profiles for the new
images urls. It includes the following changes:

- Add two new optional variables opt_image_type/opt_board and check
if they are defined before using them as a work-around for the amd64
wayland image url.
- Move some common profiles variables to the parent profile.
- Include the new minnowboard-max-minimal profile.

Xavier Claessens authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D1309

Luis Araujo authored 9 years ago and Sjoerd Simons committed 9 years ago

This commits adds the package dependency 'telepathy-mission-control-5'
for the telepathy-ring test.

Luis Araujo authored 9 years ago and Sjoerd Simons committed 9 years ago

Folks packages were dropped from the images by default so now the tests
need to explicitly install these packages as dependencies.

Jonny Lamb authored 9 years ago and Sjoerd Simons committed 9 years ago

On fast platforms (amd64) these fail with a "Conflicting UIDs found in
added contacts" which is very weird, so add a few sleeps here which
ensure the tests pass.

Bug: https://bugs.apertis.org/show_bug.cgi?id=610



Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D1304

Using an alpha value of 0 makes the stage transparent if the device
happens to have an ARGB visual on X (or always on wayland)

Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Reviewed-by: Mathieu Duponchelle <meh@collabora.co.uk>

Differential Revision: https://phabricator.apertis.org/D1154

Philip Withnall authored 9 years ago and Sjoerd Simons committed 9 years ago

See the previous commit.

Bug: https://bugs.apertis.org/show_bug.cgi?id=602



Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D1170

Philip Withnall authored 9 years ago and Sjoerd Simons committed 9 years ago

For the 3.18 kernel release, the vivi v4l test driver was dropped, and
replaced by the vivid driver. This does much the same thing, but has a
different set of options, and outputs at a different resolution by
default.

Update the v4l/automated/run-test.sh test to use vivid instead of vivi.

Bug: https://bugs.apertis.org/show_bug.cgi?id=602



Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D1169

Philip Withnall authored 9 years ago and Sjoerd Simons committed 9 years ago

Otherwise the folks-ofonotest.sh file tries to set the libfolks backend
configuration to a non-existent file when the folks-ofono-backend test
is run, and folks-inspect ends up showing all contacts instead of just
contacts from the oFono backend.

Bug: https://bugs.apertis.org/show_bug.cgi?id=605



Signed-off-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D1164

Philip Withnall authored 9 years ago and Sjoerd Simons committed 9 years ago

Because test-sysprof.sh was running sysprof-cli under sudo, the
SYSPROF_PID it later tried to kill was sudo, not sysprof-cli. This meant
that sysprof-cli was not actually terminated, meaning the test waited on
it forever.

Fix that by no longer running sysprof-cli under sudo — the whole test
must be run as root instead. Add a whoami check for this and re-exec
under sudo if not running as root. This works on passwordless sudo
systems.

Bug: https://bugs.apertis.org/show_bug.cgi?id=558



Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D1160

Héctor Orón Martínez authored 9 years ago and Sjoerd Simons committed 9 years ago

Signed-off-by: Héctor Orón Martínez <hector.oron@collabora.co.uk>

Xavier Claessens authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D923

Xavier Claessens authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D962

Xavier Claessens authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D926

Xavier Claessens authored 9 years ago and Sjoerd Simons committed 9 years ago

That file was known to have issues. Also it verify files under
/home/shared gets indexed/thumbnailed.

Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D925

Xavier Claessens authored 9 years ago and Sjoerd Simons committed 9 years ago

Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D924

Xavier Claessens authored 9 years ago and Sjoerd Simons committed 9 years ago

There are rare case of timeout probably in
TrackerIndexerMixin.__init__(). Hopefully this will
help understanding exactly where.

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D909

Xavier Claessens authored 9 years ago and Sjoerd Simons committed 9 years ago

It got removed since the grilo rewrite, merged into the main grilo
tests.

Differential Revision: https://phabricator.apertis.org/D891


Reviewed-by: Luis Araujo <luis.araujo@collabora.co.uk>