Skip to content

Commit b82384a

Browse files
authored
feat: merge-train/spartan (#23817)
BEGIN_COMMIT_OVERRIDE fix: wrap rpc errors (#23794) chore(p2p): BlockTxsRequest comment (#23818) chore: improve pod scheduling (#23820) chore: setup custom testnet coinbase (#23824) fix(bench): upload benchmarks to correct gh repo (#23825) chore: add hi-mem taint (#23828) chore: update artifact registry (#23845) END_COMMIT_OVERRIDE
2 parents 4adc0f8 + c70de5a commit b82384a

37 files changed

Lines changed: 646 additions & 145 deletions

.github/workflows/deploy-network.yml

Lines changed: 49 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,15 @@ on:
1717
description: "Full Aztec docker image (e.g., aztecprotocol/aztec:2.3.4). If not set, constructed from semver."
1818
required: false
1919
type: string
20+
prover_docker_image:
21+
description: "Full Prover docker image URL. If not set defaults to aztec_docker_image."
22+
required: false
23+
type: string
24+
use_internal_docker_registry:
25+
description: "Construct Aztec docker images from INTERNAL_DOCKER_REGISTRY and semver."
26+
required: false
27+
type: boolean
28+
default: false
2029
ref:
2130
description: "Git ref to checkout"
2231
required: false
@@ -62,6 +71,15 @@ on:
6271
description: "Full Aztec docker image (e.g., aztecprotocol/aztec:2.3.4). If not set, constructed from semver."
6372
required: false
6473
type: string
74+
prover_docker_image:
75+
description: "Full Prover docker image URL. If not set defaults to aztec_docker_image."
76+
required: false
77+
type: string
78+
use_internal_docker_registry:
79+
description: "Construct Aztec docker images from INTERNAL_DOCKER_REGISTRY and semver."
80+
required: false
81+
type: boolean
82+
default: false
6583
namespace:
6684
description: "Kubernetes namespace override (optional, defaults to env file value)"
6785
required: false
@@ -123,6 +141,8 @@ jobs:
123141
node-version: 22
124142

125143
- name: Validate inputs
144+
env:
145+
INTERNAL_DOCKER_REGISTRY: ${{ secrets.INTERNAL_DOCKER_REGISTRY }}
126146
run: |
127147
# Validate network
128148
if [[ ! -f "spartan/environments/${{ inputs.network }}.env" ]]; then
@@ -146,28 +166,42 @@ jobs:
146166
fi
147167
fi
148168
149-
# Resolve the docker image
150-
if [[ -n "${{ inputs.aztec_docker_image }}" ]]; then
151-
AZTEC_DOCKER_IMAGE="${{ inputs.aztec_docker_image }}"
152-
else
153-
AZTEC_DOCKER_IMAGE="aztecprotocol/aztec:${{ inputs.semver }}"
169+
if [[ "${{ inputs.use_internal_docker_registry }}" == "true" && -z "${{ inputs.semver }}" ]]; then
170+
echo "Error: semver must be provided when use_internal_docker_registry is true"
171+
exit 1
154172
fi
155-
echo "AZTEC_DOCKER_IMAGE=$AZTEC_DOCKER_IMAGE" >> $GITHUB_ENV
156173
157-
# Use the CRS-baked prover-agent image when it exists; otherwise let the
158-
# deploy script fall back to AZTEC_DOCKER_IMAGE and download CRS on startup.
159-
if [[ -n "${{ inputs.semver }}" ]]; then
160-
PROVER_AGENT_DOCKER_IMAGE="aztecprotocol/aztec-prover-agent:${{ inputs.semver }}"
174+
# Resolve the docker image
175+
AZTEC_DOCKER_IMAGE="${{ inputs.aztec_docker_image }}"
176+
PROVER_AGENT_DOCKER_IMAGE="${{ inputs.prover_docker_image }}"
177+
INTERNAL_REGISTRY_BASE_URL=""
178+
179+
if [[ "${{ inputs.use_internal_docker_registry }}" == "true" ]]; then
180+
INTERNAL_REGISTRY_BASE_URL="${INTERNAL_DOCKER_REGISTRY%/}"
181+
echo "::add-mask::$INTERNAL_REGISTRY_BASE_URL"
182+
fi
161183
162-
echo "Checking if prover agent image exists: $PROVER_AGENT_DOCKER_IMAGE"
163-
if docker manifest inspect "$PROVER_AGENT_DOCKER_IMAGE" > /dev/null 2>&1; then
164-
echo "PROVER_AGENT_DOCKER_IMAGE=$PROVER_AGENT_DOCKER_IMAGE" >> $GITHUB_ENV
184+
if [[ -z "$AZTEC_DOCKER_IMAGE" ]]; then
185+
if [[ -n "$INTERNAL_REGISTRY_BASE_URL" ]]; then
186+
AZTEC_DOCKER_IMAGE="$INTERNAL_REGISTRY_BASE_URL/aztec:${{ inputs.semver }}"
187+
echo "::add-mask::$AZTEC_DOCKER_IMAGE"
165188
else
166-
echo "Prover agent image does not exist: $PROVER_AGENT_DOCKER_IMAGE"
167-
echo "Falling back to AZTEC_DOCKER_IMAGE for prover agents."
189+
AZTEC_DOCKER_IMAGE="aztecprotocol/aztec:${{ inputs.semver }}"
190+
fi
191+
fi
192+
193+
if [[ -z "$PROVER_AGENT_DOCKER_IMAGE" ]]; then
194+
if [[ -n "$INTERNAL_REGISTRY_BASE_URL" ]]; then
195+
PROVER_AGENT_DOCKER_IMAGE="$INTERNAL_REGISTRY_BASE_URL/aztec-prover-agent:${{ inputs.semver }}"
196+
echo "::add-mask::$PROVER_AGENT_DOCKER_IMAGE"
197+
elif [[ -z "${{ inputs.aztec_docker_image }}" && -n "${{ inputs.semver }}" ]]; then
198+
PROVER_AGENT_DOCKER_IMAGE="aztecprotocol/aztec-prover-agent:${{ inputs.semver }}"
168199
fi
169200
fi
170201
202+
echo "AZTEC_DOCKER_IMAGE=$AZTEC_DOCKER_IMAGE" >> $GITHUB_ENV
203+
echo "PROVER_AGENT_DOCKER_IMAGE=$PROVER_AGENT_DOCKER_IMAGE" >> $GITHUB_ENV
204+
171205
- name: Store the GCP key in a file
172206
env:
173207
GCP_SA_KEY: ${{ secrets.GCP_SA_KEY }}

.github/workflows/deploy-staging.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,10 @@ jobs:
3232
tag: ${{ steps.resolve.outputs.tag }}
3333
semver: ${{ steps.resolve.outputs.semver }}
3434
steps:
35-
- name: Checkout v4-next
35+
- name: Checkout v5-next
3636
uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
3737
with:
38-
ref: v4-next
38+
ref: v5-next
3939
token: ${{ secrets.AZTEC_BOT_GITHUB_TOKEN }}
4040
fetch-depth: 0
4141

@@ -82,6 +82,7 @@ jobs:
8282
with:
8383
network: staging
8484
semver: ${{ needs.determine-tag.outputs.semver }}
85+
use_internal_docker_registry: true
8586
source_tag: ${{ needs.determine-tag.outputs.tag }}
8687
deploy_contracts: ${{ inputs.deploy_contracts == true }}
8788
secrets: inherit

.github/workflows/nightly-spartan-bench.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -306,6 +306,7 @@ jobs:
306306
tool: "customSmallerIsBetter"
307307
output-file-path: ./bench-out/bench.json
308308
github-token: ${{ secrets.AZTEC_BOT_GITHUB_TOKEN }}
309+
gh-repository: github.com/AztecProtocol/benchmark-page-data
309310
auto-push: true
310311
ref: ${{ github.event.workflow_run.head_sha || github.sha }}
311312
alert-threshold: "120%"

.github/workflows/weekly-proving-bench.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,7 @@ jobs:
151151
tool: "customSmallerIsBetter"
152152
output-file-path: ./bench-out/bench.json
153153
github-token: ${{ secrets.AZTEC_BOT_GITHUB_TOKEN }}
154+
gh-repository: github.com/AztecProtocol/benchmark-page-data
154155
auto-push: true
155156
ref: ${{ github.event.workflow_run.head_sha || github.sha }}
156157
alert-threshold: "120%"

spartan/aztec-node/scripts/setup-attester-keystore.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,11 @@ for ((v = 0; v < VALIDATORS_PER_NODE; v++)); do
9595
attester="${private_keys[$v]}"
9696
fi
9797

98+
coinbase="${COINBASE:-$attester}"
99+
98100
validators_json+="{
99101
\"attester\": \"$attester\",
100-
\"coinbase\": \"$attester\",
102+
\"coinbase\": \"$coinbase\",
101103
\"feeRecipient\": \"0x0000000000000000000000000000000000000000000000000000000000000000\"
102104
}"
103105
done

spartan/environments/testnet.env

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@ VALIDATOR_PUBLISHERS_PER_REPLICA=8
8080
VALIDATOR_PUBLISHER_MNEMONIC_START_INDEX=5000
8181
VALIDATOR_HA_REPLICAS=1
8282
VALIDATOR_RESOURCE_PROFILE="prod"
83+
VALIDATOR_COINBASE="0x36502A83735ED62671B55858809703898cDE4f95"
8384

8485
PUBLISHERS_PER_PROVER=2
8586
PROVER_PUBLISHER_MNEMONIC_START_INDEX=8000

spartan/scripts/deploy_network.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -569,6 +569,7 @@ VALIDATOR_PUBLISHER_MNEMONIC_START_INDEX = ${VALIDATOR_PUBLISHER_MNEMONIC_START_
569569
VALIDATORS_PER_NODE = ${VALIDATORS_PER_NODE}
570570
VALIDATOR_REPLICAS = ${VALIDATOR_REPLICAS}
571571
VALIDATOR_PUBLISHERS_PER_REPLICA = ${VALIDATOR_PUBLISHERS_PER_REPLICA}
572+
VALIDATOR_COINBASE = $(tf_str "${VALIDATOR_COINBASE:-}")
572573
VALIDATOR_HA_REPLICAS = ${VALIDATOR_HA_REPLICAS}
573574
VALIDATOR_HA_REPLICA_COUNT = ${VALIDATOR_HA_REPLICA_COUNT:-null}
574575
VALIDATOR_HA_OLD_DUTIES_MAX_AGE_H = ${VALIDATOR_HA_OLD_DUTIES_MAX_AGE_H}

0 commit comments

Comments
 (0)