On 06/10/2023 3:05 am, Marek Marczykowski-Górecki wrote: > grep+sleep message every 1s makes job log unnecessary hard to read. > > Signed-off-by: Marek Marczykowski-Górecki <marma...@invisiblethingslab.com> > --- > I know I can download serial log file, but that's 3 more clicks... > --- > automation/scripts/qubes-x86-64.sh | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/automation/scripts/qubes-x86-64.sh > b/automation/scripts/qubes-x86-64.sh > index 1e84e40a4afc..5464d10fc343 100755 > --- a/automation/scripts/qubes-x86-64.sh > +++ b/automation/scripts/qubes-x86-64.sh > @@ -222,10 +222,12 @@ if [ -n "$wait_and_wakeup" ]; then > ssh $CONTROLLER wake > fi > > +set +x > until grep "^Welcome to Alpine Linux" smoke.serial || [ $timeout -le 0 ]; do > sleep 1; > : $((--timeout)) > done > +set -x > > tail -n 100 smoke.serial >
This wants repeating in dom0_check= when looking for "Welcome to Alpine" in guest-domU.log because the scrool is still visible in https://gitlab.com/xen-project/people/marmarek/xen/-/jobs/5235487317 Happy to fix on commit, as this is the only comment I have on the series. Alternatively, would it be worth writing a "wait_until $msg $file" function to abstract this away? ~Andrew