Skip to content

Commit 11d0228

Browse files
Evrard-Nilclaude
andauthored
Fix CI: Docker build missing benches dir and audit workflow permissions (#44)
The benchmarks PR added a [[bench]] target in Cargo.toml but the Dockerfile only copied src/, causing cargo to fail parsing the manifest. Also grant issues:write permission to the security audit workflow so rustsec/audit-check can create issues for advisories. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent bfc8166 commit 11d0228

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

.github/workflows/security-audit.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,9 @@ on:
99
# Run weekly on Mondays at 9am UTC
1010
- cron: "0 9 * * 1"
1111

12+
permissions:
13+
issues: write
14+
1215
jobs:
1316
audit:
1417
name: cargo audit

Dockerfile

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,16 @@ ENV SOURCE_DATE_EPOCH=${SOURCE_DATE_EPOCH}
1212
# Cache dependencies: copy manifests first, then do a dummy build
1313
COPY Cargo.toml Cargo.lock ./
1414
RUN mkdir src && echo "fn main() {}" > src/main.rs && echo "" > src/lib.rs \
15+
&& mkdir -p benches && echo "fn main() {}" > benches/hot_path.rs \
1516
&& cargo build --release --locked 2>/dev/null || true \
16-
&& rm -rf src \
17+
&& rm -rf src benches \
1718
&& rm -f target/release/deps/*vllm_proxy_rs* \
1819
&& rm -f target/release/vllm-proxy-rs* \
1920
&& rm -rf target/release/.fingerprint/vllm-proxy-rs-*
2021

2122
# Copy real source and build — touch to ensure cargo detects changes
2223
COPY src/ src/
24+
COPY benches/ benches/
2325
RUN find src -name '*.rs' -exec touch {} + && cargo build --release --locked
2426

2527
# Stage 2: Runtime image

0 commit comments

Comments
 (0)