Also remove calls to sys.(stdout|stderr).flush that we flusing lava-run
stdout/stderr instead of the subprocess stdout/stderr.

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

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

Helper script:

for file in $(git log --stat --since=2018-12-31 --name-only | grep -v -e "^commit" -e "^Author:" -e "^Date:" | grep -E '^\S' | sort -u | grep \\.py$); do if (grep -q Copyright $file); then if (grep -q Copyright.*2019 $file) ; then :; else echo "$file needs updating"; fi; fi; done | less

Signed-off-by: Steve McIntyre <steve.mcintyre@linaro.org>

Closes: #147 Only force a prompt if force_prompt is
already set. Declare the use of force_prompt in debug
logs.

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

Also fix some warnings

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

Exclude action.py and parser.py which need some
formatting changes.
Change the black task to exclude the remaining files.

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

Borrow from the Lxc support to use a custom ShellSession class to
allow use of poweroff for all QEMU instead of needing connection_tags.
Ensure following test actions remount the overlay image.

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

In python3, all classes are inheriting from object by default.

Change-Id: Ic5c1b533c3479b4fdb8cbfe56d051022d1efa779

Change-Id: Iab02aa99a0e2fc53f4184d535aa7882490d0b60b

This export is no longer required and plays
awkwardly with second UART support.

Change-Id: I2cd74adfdb1b49544727c6175e377193384f8739

This argument is not used anymore.

Change-Id: I9cc98ed14003e79110c820149570cc1248549535

spawnwindowsize limits the amount of buffer searched without
changing the size of the buffer, so prompts were missed.
Use maxread - within the platform/tty restrictions - to
slightly improve the buffer size. Keep searchwindowsize
at the default of searching the entire buffer.

Change-Id: I99fa570c6537a1b499d34533bc214ae939084a1a

Change-Id: Icae93ccd1949c8ccde656371dbbb9b0a66a3d03d

Make a clear separation between lava_server and lava_dispatcher
binary packages so that a master does not need all the dependencies
of lava-dispatcher installed (and does not necessarily have a
local worker).
Update documentation on how lava-common can be used.

Change-Id: I040a44c7ce6d853524ccc33e08df593b0d14c54f

Change-Id: I2fa71e84007a7ca7ced8f9f078027c85fd874660

Change-Id: I8bc55876f82817533dfef42128aefea15c904311

Change-Id: I6288743094584110b7815cae41caaa994b702a66

The complex string / list handling of the connection prompts is a
pexpect behaviour. Clarify our code handling to avoid TypeError
when adding a list to an existing prompt string by first converting
to a list with a single string entry.

Change-Id: I570f2ad51b436a4a7bfcac1e642bb2533b788f16

Ensure that pexpect is able to replace invalid UTF8 characters
received over the serial connection without losing input or
raising an exception. Fix pylint warning in power.py and
output the full exception instead of just the summary message.

Change-Id: I1cb7e0da0b015a50d99bdea58ec22d6ac3788226

This also fix the name when calling parent __init__ function.

Change-Id: I302c3920eed9ef51232e22d75d555f0d34abf510

https://staging.validation.linaro.org/scheduler/job/202322#L1126

Change-Id: I2068405054d2f01d319dd51b194b9ff992cd1400

Requires the new connection command support from the second UART
changes to support tags for each connection command.
telnet and ssh can be supported at this time.

Change-Id: Ia787ba3e0d971dd8cdae6fed341cc845d191a4c2

prospector error message:
  lost-exception / return statement in finally block may swallow exception

For instance, calling foo() will return 1, without raising the exception.

def foo():
  try:
    raise Exception()
  finally:
    return 1

Change-Id: Ic3ae097a24d56af366cf3c2c1ca1c5544efd4255

Change-Id: If23943e7e8e9cf285c4c43d0b9c9eeeeea5ef1b6

Ignore empty reads or single newline reads.

Change-Id: I15a9ec4ef9a483a62ee717233cd5f3514765a4e8

Change-Id: I098edd9edfeb968b303eaedc6afb6654e43b4b98

Also remove the need for KeyboardInterrupt by using a specific exception class.

Change-Id: Icfa76a2d2bc8410bbd2f963084adf43d0ae891c0

Needs python-pexpect and python3-pexpect from
jessie-backports

Change-Id: I84f21216b37e90be62818fc9e0226935018fe4ec

Change-Id: Ic46a90c1638b6ad7a711122fcf2ab2635cfb9a7b

Add retrieval of content from other namespaces by default and log as
feedback interleaved into the test log.

Change-Id: I064ba71795e56e95506ea2521276818187510a62

All these exceptions are inheriting from LAVAException. Hence it's easy to
classify and catch them.

Change-Id: Ib99db6145efdeedbfc03755a2787158469aa6fac

- Remove expect-bootloader-connection because no other bootloader
classes have it.
- Remove the extra newline from the interrupt string that is sent

Change-Id: I20e865e7cb5a1b96a21b1d75842d1017fe49a709

- Only have a delay between characters sent on serial if a
delay is defined in the device yaml
- All devices were mistakenly forcing send_char on, this was
adding around 30 seconds to the bootloader-commands action duration
- Corresponding lava-server device-types review incoming

Change-Id: I55e98a8c86257e22763b681b924ecf86ae0ce1c2

Change-Id: Ib05c98c3b45659d353f100fdf20b552160be3fb7

Remove CommandRunner and let actions decide whether to simply
wait or use force_wait_prompt.

Change-Id: I83033f48e3639596b3eff788be8244fc9f334748

Change-Id: Id4a5b3a6c0a37cfa72320988bec9ff2a00b8bcd2

Undo review: https://review.linaro.org/#/c/14586/
Fix issue: https://projects.linaro.org/browse/TICKET-544

Set the default linesep back to '\n', because if only UEFI
needs it changed we shouldn't have changed it for every other
bootloader.

Change the connection linesep for UEFI.

Auto_login_action sets the linesep from the deployment_data
for the OS that has been booted.

Removed the SHELL_SEND_DELAY constant because it was not used
anywhere.

Change-Id: I912a39f70e6ce0549cb73b9bf6a6473bf5b44d06