Skip to content

Commit

Permalink
CI: don't store kernel log, only check for coredumps.
Browse files Browse the repository at this point in the history
The kernel log is uploaded automatically by the custom executor, no need
to upload it from inside the VM (and /mnt/artifacts is not mounted in
the new CI scripts).
  • Loading branch information
danzatt authored and mbroz committed Feb 22, 2024
1 parent 193f8ff commit af35c93
Show file tree
Hide file tree
Showing 9 changed files with 10 additions and 12 deletions.
4 changes: 1 addition & 3 deletions .gitlab-ci.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,8 @@
stages:
- test

.dump_kernel_log:
.fail_if_coredump_generated:
after_script:
- sudo dmesg > /mnt/artifacts/dmesg.log
- sudo journalctl > /mnt/artifacts/journalctl.log
- '[ "$(ls -A /var/coredumps)" ] && exit 1 || true'

include:
Expand Down
4 changes: 2 additions & 2 deletions .gitlab/ci/alpinelinux.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
.alpinelinux-dependencies:
variables:
DISTRO: cryptsetup-alpine-edge
after_script:
- '[ "$(ls -A /var/coredumps)" ] && exit 1 || true'
extends:
- .fail_if_coredump_generated
before_script:
- >
sudo apk add
Expand Down
2 changes: 1 addition & 1 deletion .gitlab/ci/annocheck.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
test-main-commit-job-annocheck:
extends:
- .dump_kernel_log
- .fail_if_coredump_generated
tags:
- libvirt
- cryptsetup-rhel-9
Expand Down
2 changes: 1 addition & 1 deletion .gitlab/ci/centos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
variables:
DISTRO: cryptsetup-centos-stream-9
extends:
- .dump_kernel_log
- .fail_if_coredump_generated
before_script:
- >
sudo dnf -y -q install
Expand Down
2 changes: 1 addition & 1 deletion .gitlab/ci/csmock.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
test-commit-job-csmock:
extends:
- .dump_kernel_log
- .fail_if_coredump_generated
tags:
- libvirt
- cryptsetup-rhel-9
Expand Down
2 changes: 1 addition & 1 deletion .gitlab/ci/debian.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
.debian-prep:
extends:
- .dump_kernel_log
- .fail_if_coredump_generated
variables:
DISTRO: cryptsetup-debian-12
before_script:
Expand Down
2 changes: 1 addition & 1 deletion .gitlab/ci/fedora-opal.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
variables:
DISTRO: cryptsetup-fedora-rawhide
extends:
- .dump_kernel_log
- .fail_if_coredump_generated
before_script:
- >
[ -z "$RUN_SYSTEMD_PLUGIN_TEST" ] ||
Expand Down
2 changes: 1 addition & 1 deletion .gitlab/ci/fedora.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
variables:
DISTRO: cryptsetup-fedora-rawhide
extends:
- .dump_kernel_log
- .fail_if_coredump_generated
before_script:
- >
[ -z "$RUN_SYSTEMD_PLUGIN_TEST" ] ||
Expand Down
2 changes: 1 addition & 1 deletion .gitlab/ci/rhel.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
.rhel-openssl-backend:
extends:
- .dump_kernel_log
- .fail_if_coredump_generated
before_script:
- >
sudo yum -y -q install
Expand Down

0 comments on commit af35c93

Please sign in to comment.