As per Huawei's advice, enable acpi=force to force ACPI and
ipmi_si.trydmi=0 to disable IPMI detection to avoid intermittent
kernel initialisation failures due to some firmware issues.

Signed-off-by: Guillaume Tucker <guillaume.tucker@collabora.com>

When a TestCase object has no action_metadata associated with it, the
object attribute is None.  Check if this is the case before calling
dict methods on it, to fix exceptions like this one:

  [...]
    File "/usr/lib/python3/dist-packages/lava_results_app/dbutils.py", line 352, in walk_actions
      build_action(action, testdata, submission)
    File "/usr/lib/python3/dist-packages/lava_results_app/dbutils.py", line 332, in build_action
      if 'level' in case.action_metadata:
  TypeError: argument of type 'NoneType' is not iterable

Signed-off-by: Guillaume Tucker <guillaume.tucker@collabora.com>

Sjoerd Simons authored Sep 28, 2018 and Guillaume Tucker committed Dec 12, 2018

If the server does not provide extra dispatcher configuration check for
a local file to include in the runs. Useful to e.g. configure the IP to
use for communicating with the duts.

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

Denis Pynkin authored Mar 27, 2018 and Guillaume Tucker committed Dec 12, 2018

Allow to specify any path in `lava_test_results_dir` for deployments.

Originally guest image for Qemu is mounted to target mount point but in the same
time path inside the image skip only 1 directory in hierarchy doubling
the rest of the path in Qemu VM, for instance:
`/var/lib/lava-5/lib/lava-5/bin/...` instead of `/var/lib/lava-5/bin/...`.
This behavior causes the issues in `lava-test-shell` test:
 lava-test: # lava-test: # sh: 9: /var/lib/lava-4/bin/lava-test-runner: not found

Signed-off-by: Denis Pynkin <denis.pynkin@collabora.com>

Some Ubuntu-derived distros such as Apertis have a read-only root file
system, and it can't be re-mounted as rw.  So to be able to store LAVA
test files, move the directories to /var/lib/ instead as this has
write permissions.

Signed-off-by: Guillaume Tucker <guillaume.tucker@collabora.com>

Retired devices often have no worker defined.
Devices with other health states need to warn of a
missing worker host setting.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Add notes on bootloader testing and recovery

Signed-off-by: Neil Williams <neil.williams@linaro.org>

When the device dictionary is not valid yaml, the lava-masster service was crashing.

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Although releases are prepared from git tag downloads, preparation
of the release and testing of the changes needs to have a tarball
and the simplest way is to allow setup.py to run the sdist argument.
version.py tries to pass that to git describe which fails.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

To process the aarch64 stretch packages, the deploy must
depend on the task which built the aarch64 stretch packages.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Update the comment in pkg-debian-9 script to the
actual behaviour.
Fix issue in both aarch64 scripts to only build
binary packages (for the native arch) so that
lava-common comes from the amd64 build when
including into reprepro.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

If the flasher command does contain whitespaces, then calling cmd.split(' ')
will split the command in two.
For instance, a flasher using "'/bin/flasher command' kernel.bin" is not going
to work without this patch.

Closes #129

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Developers can benefit from the script and the supporting
data.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Closes: #154

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Several devices need time to drain power at poweroff and if
admins configure a wait in the power off command, this timeout
will be triggered.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

There is a build directory in .gitlab-ci so changes there
get ignored. Change the scripts to use _build which is
already ignored due to the documentation build.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Caught with pylint3 - if run returns None instead of
the connection, other later steps can fail.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Apply https://review.linaro.org/#/c/lava/lava-lab/+/28967/


to upstream. Power off needs a longer timeout to allow the
device to completely discharge when running a lot of jobs
in sequence. NFS needs to be forced to version 3 to prevent
kernel panics.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

The output is still a valid shell command.

This will help to debug issues with whitespaces in command lines.

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Closes: #155 - provide information on errors even if the
error has to be fixed elsewhere.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

This caused a failure in !181

Signed-off-by: Neil Williams <neil.williams@linaro.org>

This helper allows to run a given command on the dispatcher itself while
sending the stdout and stderr stream to the logger. The output will be visible
in real time.
If the command fails (returning non 0), then an exception will be raised.

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

Not all commands provide sane exit codes but when the output of
the command must be parsed, ensure that the string match is
short and case-insensitive to protect against some of the
likely optimisations in the tool used in the command.
Add a new parsed_command function to Action to help
future debugging.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

sendline is already adding a newline at the end of the string.

This is only crashing with invalid device dictionary.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Fix some security check comments which got moved by
black in the lava_scheduler_app/tests files.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Signed-off-by: Neil Williams <neil.williams@linaro.org>

We can now do "./version.py origin/master" to get the current master version.

Closes: #152

Signed-off-by: Neil Williams <neil.williams@linaro.org>

Signed-off-by: Neil Williams <neil.williams@linaro.org>

This will make sorting tags easier as:
"2018.10" < "2018.10.0001.g123123456-master" < "2018.11"

Which is not true with "master-2018.10.0001.g123123456"

Signed-off-by: Rémi Duraffort <remi.duraffort@linaro.org>

The conditional can cause the list to be empty, so the
[0] is out of range. Some versions of fastboot capitalise
the word "Finished", so compare with lower().
Report a fail if no match found.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

During testing, arm64 packages were not available for
buster. Uncomment lines excluded during debug.

Signed-off-by: Neil Williams <neil.williams@linaro.org>

The later is deprecated and *might* be removed in the future