ci: cap job runtime, dump on test hangs, expand the Linux matrix
Two stalls hit CI in one night with nothing bounding them but the 360-minute default: a deadlocked test host (silent for 2h41m) and a stalled EPEL mirror mid-download in the dnf step (probing the pool showed the first mirror in the current US mirrorlist returning HTTP 500 and sibling mirrors serving mismatched metadata generations). - timeout-minutes: 30 on both jobs. Verified live: the cap killed a stalled job at exactly 30:02. - dotnet test --blame-hang-timeout 10m: a stuck test host is killed and the in-flight tests are named with a full process dump, instead of hanging until the job timeout. Test results (and dumps) upload as artifacts on failure. - EPEL setup follows the EPEL quickstart ordering: dnf-plugins-core, enable CRB, then epel-release. epel-next is not installed: none of the prerequisites need it, EPEL 10 does not have it, and it is one more mirrorlist to fetch. - Matrix: add Ubuntu 26.04 LTS, CentOS Stream 10, and AlmaLinux 10; bump Fedora 42 (EOL) to 44. README badges and Linux prerequisites updated to match. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Kamron Batman
parent
7434ed7ee1
commit
43c218985a
2 changed files with 57 additions and 14 deletions
52
.github/workflows/build-test.yml
vendored
52
.github/workflows/build-test.yml
vendored
|
|
@ -23,6 +23,8 @@ on:
|
|||
jobs:
|
||||
build-macos:
|
||||
runs-on: ${{ matrix.os }}
|
||||
# A hung run otherwise bills the full 360-minute default before GitHub kills it.
|
||||
timeout-minutes: 30
|
||||
name: Build (${{ matrix.name }})
|
||||
strategy:
|
||||
fail-fast: false
|
||||
|
|
@ -52,15 +54,26 @@ jobs:
|
|||
- name: Migration Changes
|
||||
run: git diff --exit-code ./**/Migrations/*.v*.json
|
||||
- name: Test
|
||||
# blame-hang kills a stuck test host after 10 minutes and reports the in-flight
|
||||
# tests plus a process dump instead of hanging until the job timeout.
|
||||
run: |
|
||||
dotnet test --logger trx --results-directory ./TestResults
|
||||
dotnet test --logger trx --results-directory ./TestResults --blame-hang --blame-hang-timeout 10m --blame-hang-dump-type full
|
||||
if [ -z "$(find ./TestResults -name '*.trx' 2>/dev/null)" ]; then
|
||||
echo "::error::No test result files were produced - no test projects ran. Failing to avoid masking failures."
|
||||
exit 1
|
||||
fi
|
||||
- name: Upload test results on failure
|
||||
if: failure()
|
||||
uses: actions/upload-artifact@v4
|
||||
with:
|
||||
name: TestResults-${{ matrix.name }}
|
||||
path: ./TestResults
|
||||
if-no-files-found: ignore
|
||||
|
||||
build-linux:
|
||||
runs-on: ubuntu-latest
|
||||
# A hung run otherwise bills the full 360-minute default before GitHub kills it.
|
||||
timeout-minutes: 30
|
||||
container:
|
||||
image: ${{ matrix.container }}
|
||||
options: --security-opt seccomp=unconfined
|
||||
|
|
@ -69,6 +82,9 @@ jobs:
|
|||
fail-fast: false
|
||||
matrix:
|
||||
include:
|
||||
- container: ubuntu:26.04
|
||||
name: Ubuntu 26
|
||||
packageManager: apt
|
||||
- container: ubuntu:noble
|
||||
name: Ubuntu 24
|
||||
packageManager: apt
|
||||
|
|
@ -81,20 +97,33 @@ jobs:
|
|||
- container: debian:bookworm
|
||||
name: Debian 12
|
||||
packageManager: apt
|
||||
- container: fedora:42
|
||||
name: Fedora 42
|
||||
- container: fedora:44
|
||||
name: Fedora 44
|
||||
packageManager: dnf
|
||||
- container: quay.io/centos/centos:stream9
|
||||
name: CentOS 9 Stream
|
||||
packageManager: dnf
|
||||
epel: true
|
||||
- container: quay.io/centos/centos:stream10
|
||||
name: CentOS 10 Stream
|
||||
packageManager: dnf
|
||||
epel: true
|
||||
- container: almalinux:10
|
||||
name: AlmaLinux 10
|
||||
packageManager: dnf
|
||||
epel: true
|
||||
|
||||
steps:
|
||||
- name: Enable EPEL and CRB for CentOS
|
||||
# Enable CRB before EPEL, per the EPEL quickstart. epel-next is not installed:
|
||||
# none of the prerequisites need it, EPEL 10 does not have it, and it is one more
|
||||
# mirrorlist to fetch.
|
||||
- name: Enable EPEL and CRB
|
||||
run: |
|
||||
dnf upgrade --refresh -y
|
||||
dnf install -y epel-release epel-next-release
|
||||
dnf install -y dnf-plugins-core
|
||||
dnf config-manager --set-enabled crb
|
||||
if: ${{ startsWith(matrix.name, 'CentOS') }}
|
||||
dnf install -y epel-release
|
||||
if: ${{ matrix.epel }}
|
||||
- name: Install Prerequisites using dnf
|
||||
run: dnf makecache --refresh && dnf install -y findutils libicu libdeflate-devel zstd libargon2-devel liburing-devel
|
||||
if: ${{ matrix.packageManager == 'dnf' }}
|
||||
|
|
@ -111,9 +140,18 @@ jobs:
|
|||
- name: Build
|
||||
run: dotnet run --project Projects/BuildTool -- --config Release --skip-prereqs
|
||||
- name: Test
|
||||
# blame-hang kills a stuck test host after 10 minutes and reports the in-flight
|
||||
# tests plus a process dump instead of hanging until the job timeout.
|
||||
run: |
|
||||
dotnet test --logger trx --results-directory ./TestResults
|
||||
dotnet test --logger trx --results-directory ./TestResults --blame-hang --blame-hang-timeout 10m --blame-hang-dump-type full
|
||||
if [ -z "$(find ./TestResults -name '*.trx' 2>/dev/null)" ]; then
|
||||
echo "::error::No test result files were produced - no test projects ran. Failing to avoid masking failures."
|
||||
exit 1
|
||||
fi
|
||||
- name: Upload test results on failure
|
||||
if: failure()
|
||||
uses: actions/upload-artifact@v4
|
||||
with:
|
||||
name: TestResults-${{ matrix.name }}
|
||||
path: ./TestResults
|
||||
if-no-files-found: ignore
|
||||
|
|
|
|||
19
README.md
19
README.md
|
|
@ -18,12 +18,13 @@ ModernUO [](https://www.microsoft.com/en-US/evalcenter/evaluate-windows-server-2022)
|
||||
|
||||
[](https://www.debian.org/distrib/)
|
||||
[](https://ubuntu.com/download/server)
|
||||
[](https://ubuntu.com/download/server)
|
||||
<br/>
|
||||
[](https://alpinelinux.org/downloads/)
|
||||
[](https://getfedora.org/en/server/download/)
|
||||
[](https://access.redhat.com/downloads)
|
||||
[](https://www.centos.org/download/)
|
||||
[](https://getfedora.org/en/server/download/)
|
||||
[](https://access.redhat.com/downloads)
|
||||
[](https://www.centos.org/download/)
|
||||
[](https://almalinux.org/get-almalinux/)
|
||||
[](https://get.opensuse.org/)
|
||||
[](https://www.suse.com/download/sles/)
|
||||
[](https://linuxmint.com/download.php)
|
||||
|
|
@ -77,11 +78,15 @@ Run `./publish.cmd` (Windows) or `./publish.sh` (Linux/macOS) with no arguments
|
|||
- `arm64` - ARM 64-bit
|
||||
|
||||
## Linux Prerequisites
|
||||
### Fedora, CentOS, RHEL, etc
|
||||
### Fedora, CentOS, RHEL, AlmaLinux, Rocky, etc
|
||||
```shell
|
||||
dnf upgrade --refresh -y
|
||||
# CentOS does not come with EPEL enabled
|
||||
dnf install -y epel-release epel-next-release
|
||||
# RHEL-family distributions (CentOS Stream, AlmaLinux, Rocky, RHEL) need CRB and EPEL enabled.
|
||||
# Fedora skips this block.
|
||||
dnf install -y dnf-plugins-core
|
||||
dnf config-manager --set-enabled crb
|
||||
dnf install -y epel-release
|
||||
# Prerequisites
|
||||
dnf install -y findutils libicu libdeflate-devel zstd libargon2-devel liburing-devel
|
||||
```
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue