[libvirt PATCH] ci: integration: Rename all Avocado standard stream log files to *.log

Erik Skultety eskultet at redhat.com
Tue Mar 22 12:11:22 UTC 2022


By default, stdout/stderr Avocado test log files do not have any file
extension which confuses GitLab's web UI to mangle the MIME type for
these and so the browser will never offer the option to open such file
from in a text editor rather than dowloading it.
Since GitLab sets a proper MIME for .txt and .log file extensions,
rename all Avocado log files without an extension to *.log . This pairs
nicely with the coredumpctl info file which we already name as
'coredumpctl.txt' because of this.

Signed-off-by: Erik Skultety <eskultet at redhat.com>
---

Here are 2 artifact web UI views on a failed job which you can try yourself in
your browser:
    Before this patch:
        https://gitlab.com/eskultety/libvirt/-/jobs/2232852413/artifacts/browse/logs/avocado/02-._scripts_hooks_052-domain-hook.t/
    After this patch:
        https://gitlab.com/eskultety/libvirt/-/jobs/2234111527/artifacts/browse/logs/avocado/02-._scripts_hooks_052-domain-hook.t/


 ci/integration.yml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/ci/integration.yml b/ci/integration.yml
index 519494cfd5..2808e829ef 100644
--- a/ci/integration.yml
+++ b/ci/integration.yml
@@ -36,6 +36,9 @@
     - sudo coredumpctl info --no-pager > logs/coredumpctl.txt
     - sudo mv /var/log/libvirt logs/libvirt
     - sudo chown -R $(whoami):$(whoami) logs
+      # rename all Avocado stderr/stdout logs to *.log so that GitLab's web UI doesn't mangle the MIME type
+    - find logs/avocado/ -type f ! -name "*.log" -exec
+        sh -c 'DIR=$(dirname {}); NAME=$(basename {}); mv $DIR/$NAME{,.log}' \;
   variables:
     SCRATCH_DIR: "/tmp/scratch"
   artifacts:
-- 
2.34.1



More information about the libvir-list mailing list