Skip to content

Seed the test-time artifacts and re-check failed lookups in CI #5

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
gortazar wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Seed the test-time artifacts and re-check failed lookups in CI #5

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 36 additions & 2 deletions .github/workflows/maven-build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ jobs:

while read -r p; do
echo "::group::$p"
if ! mvn -B -ntp -f "$p/pom.xml" \
if ! mvn -B -ntp -U -f "$p/pom.xml" \
-Dmaven.wagon.http.retryHandler.count=5 \
dependency:go-offline; then
echo "::warning title=Cache warming::No se pudieron resolver las dependencias de $p"
Expand All @@ -99,6 +99,36 @@ jobs:
# veredicto lo da la matriz. Un fallo aqui no corta el flujo.
echo "Proyectos sin resolver: $fallos" >> "$GITHUB_STEP_SUMMARY"

# go-offline no resuelve lo que surefire pide en tiempo de test: el
# proveedor de JUnit Platform y sus nueve artefactos lo bajaba cada job
# de build por su cuenta, y uno de ellos (common-java5) es el que fallo
# en el run 40870887229. Ejecutar los tests de un proyecto los deja en
# la cache, y a partir de ahi ningun job tiene que pedirlos.
#
# Se siembra con tema_4.2/ejemplo-2 porque es el unico proyecto sin
# Spring ni contenedores: un test, decimas de segundo. El proveedor que
# deja es el mismo que necesitan los otros 85.
- name: Seed the test-time artifacts
run: |
set -euo pipefail

mvn -B -ntp -U -f parte_4/tema_4.2/ejemplo-2/pom.xml \
-Dmaven.wagon.http.retryHandler.count=5 \
test
Comment on lines +115 to +117

# A diferencia del paso anterior, aqui si hay una expectativa
# concreta que comprobar, asi que un fallo corta: si la siembra deja
# de sembrar en silencio, volvemos a tener 86 jobs pidiendo lo mismo
# a Central a la vez y nadie se entera hasta que falle.
proveedor=$HOME/.m2/repository/org/apache/maven/surefire
if ! find "$proveedor" -name 'surefire-junit-platform-*.jar' | grep -q .; then
echo "::error title=Cache seeding::La siembra no dejo el proveedor de surefire en la cache"
exit 1
fi

echo "Artefactos de surefire en la cache:" >> "$GITHUB_STEP_SUMMARY"
find "$proveedor" -name '*.jar' | sed "s|$proveedor/|- |" >> "$GITHUB_STEP_SUMMARY"

build:
name: Build - ${{ matrix.project }}
needs: [discover, warm-cache]
Expand All @@ -124,8 +154,12 @@ jobs:
distribution: 'temurin'
cache: maven

# -U obliga a reintentar las resoluciones que fallaron antes. Sin el,
# una ausencia anotada en la cache compartida se convierte en un "no
# existe" permanente para los 86 jobs, aunque el artefacto este en
# Central. Con la cache completa no cuesta ninguna peticion extra.
- name: Build with Maven
run: mvn -B -ntp -Dmaven.wagon.http.retryHandler.count=3 clean verify
run: mvn -B -ntp -U -Dmaven.wagon.http.retryHandler.count=3 clean verify
working-directory: ${{ matrix.project }}

result:
Expand Down
Loading