Skip to content

fix(spartan): export AZTEC_ADMIN_API_KEY in benchmark functions#21887

Merged
alexghr merged 1 commit into
nextfrom
claudebox/investigate-block-capacity-401
Mar 23, 2026
Merged

fix(spartan): export AZTEC_ADMIN_API_KEY in benchmark functions#21887
alexghr merged 1 commit into
nextfrom
claudebox/investigate-block-capacity-401

Conversation

@AztecBot

@AztecBot AztecBot commented Mar 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

The benchmark functions in spartan/bootstrap.sh (block_capacity_bench, network_bench, proving_bench) were missing the AZTEC_ADMIN_API_KEY export, causing 401 Unauthorized errors when tests call the admin RPC endpoint.

Fix

Extracted the admin API key retrieval into a shared export_admin_api_key helper function (replacing 4 duplicate code blocks) and call it from run_network_tests and all three bench functions.

Test plan

  • Re-run the nightly spartan benchmarks and verify no more 401 errors

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Mar 23, 2026
@alexghr alexghr marked this pull request as ready for review March 23, 2026 10:10
@AztecBot AztecBot force-pushed the claudebox/investigate-block-capacity-401 branch from 819d893 to 7991fe5 Compare March 23, 2026 10:13
@alexghr alexghr force-pushed the claudebox/investigate-block-capacity-401 branch from 7991fe5 to 4508c77 Compare March 23, 2026 13:34
@alexghr alexghr force-pushed the claudebox/investigate-block-capacity-401 branch from 4508c77 to 2e00d71 Compare March 23, 2026 13:34
@alexghr alexghr enabled auto-merge March 23, 2026 13:35
@alexghr alexghr removed the claudebox Owned by claudebox. it can push to this PR. label Mar 23, 2026
@alexghr alexghr added this pull request to the merge queue Mar 23, 2026
Merged via the queue into next with commit 82aba13 Mar 23, 2026
20 checks passed
@alexghr alexghr deleted the claudebox/investigate-block-capacity-401 branch March 23, 2026 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants