Linting & Improved workflows

- Improved isotovideo example workflow, it can be easily extended
  with a matrix as per GitHub Actions docs [0].

  It brings:
  - A perl script for checking the test results, to remove `jq`
    usages.
  - Pipeline Artifacts, it's possible to inspect the test
    results after a run.

  And now the example workflow is green and not repeated.

- Added `use warnings` to perl scripts as per good practices.

[0]: https://docs.github.com/en/actions/using-jobs/using-a-matrix-for-your-jobs#example-expanding-configurations

.github/workflows/commit-message-checker.yml

@@ -0,0 +1,15 @@
+---
+name: 'Commit message check'
+
+on:
+  pull_request:
+  push:
+    branches:
+      # we must not fix commit messages when they already reached main
+      - '!main'
+
+jobs:
+  check-commit-message:
+    secrets:
+      accessToken: "${{ secrets.GITHUB_TOKEN }}"
+    uses: os-autoinst/os-autoinst-common/.github/workflows/base-commit-message-checker.yml@master

.github/workflows/isotovideo.yml

@@ -1,11 +1,48 @@
 ---
 name: isotovideo
 # yamllint disable-line rule:truthy
-on: [push, pull_request]
+on:
+  - pull_request
 jobs:
-  test:
-    runs-on: ubuntu-latest
+  isotovideo:
+    strategy:
+      matrix:
+        # An example matrix, it'll create a derivate job for each 
+        # item specified here. For simplicity sake the example below
+        # has the full var line since it's easy enough.
+        #
+        # For more complex setups it'll be worth exploring advance matrix setups
+        # see: https://docs.github.com/en/actions/using-jobs/using-a-matrix-for-your-jobs#example-expanding-configurations
+        isotovideo-args:
+          - "qemu_no_kvm=1 casedir=."
+    runs-on: ubuntu-20.04
     steps:
-      - uses: actions/checkout@v2
-      - name: Run isotovideo against test code in happy-path scenario
-        run: podman run --rm -it -v .:/tests:Z registry.opensuse.org/devel/openqa/containers/isotovideo:qemu-x86 qemu_no_kvm=1 casedir=/tests
+      - uses: actions/checkout@v4
+
+      - name: Run isotovideo against test code
+        uses: "docker://registry.opensuse.org/devel/openqa/containers/isotovideo:qemu-x86"
+        with:
+          args: ${{ matrix.isotovideo-args }} ${{ env.ACTIONS_STEP_DEBUG && '--debug' || '' }}
+          entrypoint: 'isotovideo'
+
+      - name: Validate test results
+        uses: "docker://registry.opensuse.org/devel/openqa/containers/isotovideo:qemu-x86"
+        with:
+          args: ./testresults/
+          entrypoint: "./scripts/validate-test-results"
+
+      - name: Archive test artifcats
+        if: always()
+        uses: actions/upload-artifact@v3
+        with:
+          name: isotovideo-artifacts
+          path: |
+            ./testresults/
+            ./video.ogv
+            ./video_time.vtt
+            ./qemu_state.json
+            ./serial_terminal.txt
+            ./virtio_console.log
+            ./virtio_console1.log
+            ./virtio_console_user.log
+          retention-days: 7

main.pm

@@ -2,6 +2,8 @@
 # SPDX-License-Identifier: GPL-2.0-or-later
 
 use strict;
+use warnings;
+
 use testapi;
 use autotest;
 

scripts/validate-test-results

@@ -0,0 +1,31 @@
+#!/usr/bin/env perl
+
+# Copyright 2023 SUSE LLC
+# SPDX-License-Identifier: GPL-2.0-or-later
+
+use strict;
+use warnings;
+
+use Mojo::JSON qw(decode_json);
+use Mojo::Path;
+use Mojo::File qw(path);
+
+my @args = @ARGV;
+my @test_results_to_check = ();
+
+while(@args){
+	my $arg = shift @args;
+	@args = (@args, glob("$arg/result-*.json")) if -d $arg;
+	next unless $arg =~ m/result-.+\.json$/;
+	push @test_results_to_check, $arg if -f $arg;
+}
+
+while(@test_results_to_check){
+	my $path = Mojo::Path->new(shift @test_results_to_check)->canonicalize;
+	my $result = decode_json(path($path)->slurp)->{result};
+
+	printf "Test result [%s] -> [%s]\n", $path->to_string, $result;
+	exit 1 unless $result eq 'ok';
+}
+
+exit 0;

tests/boot.pm

@@ -3,6 +3,9 @@
 
 use base 'basetest';
 use strict;
+use warnings;
+
+use base 'basetest';
 use testapi;
 
 sub run {