qpismont/herald
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases 4 Wiki Activity

1.2: Sandboxing #7

Merged
qpismont merged 17 commits from 1.2 into main 2026-09-20 16:58:36 +02:00
Conversation 18 Commits 17 Files Changed 43
+16 -2
1 changed files with 16 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 6a21c7d6c3 - Show all commits
.woodpecker/tests.yml
+16 -2
View File
@@ -3,13 +3,27 @@ when:
- push
steps:
- name: fmt
qpismont marked this conversation as resolved
Herald commented 2026-09-17 18:21:45 +02:00
Review
Copy Link
Copy Source

Pipeline CI : vérifier que cargo fmt --check, cargo clippy -- -D warnings et cargo test --all-features couvrent bien le nouveau workspace multi-crates (utiliser --workspace), et que les tests du sandbox qui nécessitent Docker/root sont soit exécutés, soit explicitement ignorés avec justification.

Pipeline CI : vérifier que `cargo fmt --check`, `cargo clippy -- -D warnings` et `cargo test --all-features` couvrent bien le nouveau workspace multi-crates (utiliser `--workspace`), et que les tests du sandbox qui nécessitent Docker/root sont soit exécutés, soit explicitement ignorés avec justification.
image: rust:1.97
commands:
- rustup component add rustfmt
- cargo fmt --all -- --check
- name: clippy
image: rust:1.97
commands:
- rustup component add clippy
- cargo clippy
- cargo clippy --workspace --all-targets --all-features -- -D warnings
- name: test
image: rust:1.97
commands:
- cargo test
- cargo test --workspace --all-targets
- name: container-build
qpismont marked this conversation as resolved
Herald commented 2026-09-17 17:43:35 +02:00
Review
Copy Link
Copy Source

Le job container-build compile l'image mais ne l'exécute jamais : il ne détectera donc pas l'absence de git/runtime conteneur dans l'image finale. Ajouter un smoke test (lancer l'image et vérifier la présence de git et docker/podman), et noter que le pipeline ne se déclenche que sur push, pas sur les pull requests.

Le job `container-build` compile l'image mais ne l'exécute jamais : il ne détectera donc pas l'absence de `git`/runtime conteneur dans l'image finale. Ajouter un smoke test (lancer l'image et vérifier la présence de `git` et `docker`/`podman`), et noter que le pipeline ne se déclenche que sur `push`, pas sur les pull requests.
image: quay.io/buildah/stable
privileged: true
volumes:
- /data/woodpecker-builds:/data
commands:
- buildah bud -f Containerfile -t herald-ci .