Skip to content

Commit 75002d5

Browse files
committed
align proof.json naming
1 parent 49b049d commit 75002d5

File tree

2 files changed

+31
-16
lines changed

2 files changed

+31
-16
lines changed

crates/host-bench/src/lib.rs

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,18 @@ pub struct HostArgs {
152152
pub app_proofs: Option<PathBuf>,
153153
}
154154

155+
fn write_versioned_proof(
156+
output_dir: &Path,
157+
block_number: u64,
158+
versioned_proof: VersionedVmStarkProof,
159+
) -> eyre::Result<()> {
160+
let proof_path = output_dir.join(format!("{}_proof.json", block_number));
161+
let json = serde_json::to_vec_pretty(&versioned_proof)?;
162+
fs::write(&proof_path, json)?;
163+
println!("wrote proof json to {}", proof_path.display());
164+
Ok(())
165+
}
166+
155167
pub fn reth_vm_config(_app_log_blowup: usize) -> SdkVmConfig {
156168
unimplemented!("only for openvm logic")
157169
// let mut config = toml::from_str::<AppConfig<SdkVmConfig>>(include_str!(
@@ -509,9 +521,7 @@ pub async fn run_ceno_reth_benchmark(args: HostArgs) -> eyre::Result<()> {
509521
//
510522
if let Some(output_dir) = args.output_dir.as_ref() {
511523
let versioned_proof = VersionedVmStarkProof::new(vm_stark_proof)?;
512-
let json = serde_json::to_vec_pretty(&versioned_proof)?;
513-
fs::write(output_dir.join("proof.json"), json)?;
514-
println!("wrote proof json to {}", output_dir.display());
524+
write_versioned_proof(output_dir, args.block_number, versioned_proof)?;
515525
}
516526
}
517527
BenchMode::ProveStarkOnly => {
@@ -775,9 +785,7 @@ pub async fn run_reth_benchmark(args: HostArgs, openvm_client_eth_elf: &[u8]) ->
775785

776786
if let Some(output_dir) = args.output_dir.as_ref() {
777787
let versioned_proof = VersionedVmStarkProof::new(proof)?;
778-
let json = serde_json::to_vec_pretty(&versioned_proof)?;
779-
fs::write(output_dir.join("proof.json"), json)?;
780-
println!("wrote proof json to {}", output_dir.display());
788+
write_versioned_proof(output_dir, args.block_number, versioned_proof)?;
781789
}
782790
}
783791
#[cfg(feature = "evm-verify")]

server/prove_block.sh

Lines changed: 17 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -93,9 +93,15 @@ else
9393
BLOCK_NUMBER=$(( BLOCK_NUMBER - remainder ))
9494
echo "[prove_block.sh] Latest block number: $raw_block_number (rounded down to $BLOCK_NUMBER)" >&2
9595

96-
proof_s3_uri="s3://${S3_BUCKET}/${S3_PREFIX}/${PROOF_UUID}/${BLOCK_NUMBER}_proof.json"
97-
echo "[prove_block.sh] Checking for existing proof at ${proof_s3_uri}" >&2
98-
if s5cmd ls "$proof_s3_uri" >/dev/null 2>&1; then
96+
fi
97+
98+
PROOF_FILENAME="${BLOCK_NUMBER}_proof.json"
99+
CANONICAL_PROOF_S3_URI="s3://${S3_BUCKET}/${S3_PREFIX}/${PROOF_FILENAME}"
100+
JOB_PROOF_S3_URI="s3://${S3_BUCKET}/${S3_PREFIX}/${PROOF_UUID}/${PROOF_FILENAME}"
101+
102+
if [[ -z "$BLOCK_NUMBER_OVERRIDE" ]]; then
103+
echo "[prove_block.sh] Checking for existing proof at ${CANONICAL_PROOF_S3_URI}" >&2
104+
if s5cmd ls "$CANONICAL_PROOF_S3_URI" >/dev/null 2>&1; then
99105
echo "[prove_block.sh] Found existing proof for block ${BLOCK_NUMBER}; sleeping 300s then exiting" >&2
100106
sleep 300
101107
exit 0
@@ -185,7 +191,7 @@ if [[ -n "$CENO_STATUS_API_BASE_URL" ]]; then
185191
fi
186192

187193
start_ts_ms=$(date +%s%3N)
188-
PROOF_JSON="$job_dir/proof.json"
194+
PROOF_JSON="$job_dir/${PROOF_FILENAME}"
189195

190196
OUTPUT_PATH="$job_dir/metrics.json"
191197

@@ -288,17 +294,18 @@ fi
288294
# Post-processing hook: customize as needed
289295
echo "[prove_block.sh] Proof finished with status=$status in ${duration_ms}ms at $(date -Is)" >&2
290296

291-
# Upload proof.json to S3 (best-effort)
297+
# Upload proof file to S3 (best-effort)
292298
if [[ -f "$PROOF_JSON" ]]; then
293299
set +e
294-
s5cmd cp "$PROOF_JSON" "s3://${S3_BUCKET}/${S3_PREFIX}/${PROOF_UUID}/${BLOCK_NUMBER}_proof.json"
295-
upload_rc=$?
296-
if [[ $upload_rc -ne 0 ]]; then
297-
echo "[prove_block.sh] Warning: failed to upload proof.json to S3 (rc=$upload_rc)" >&2
300+
if ! s5cmd cp "$PROOF_JSON" "$JOB_PROOF_S3_URI"; then
301+
echo "[prove_block.sh] Warning: failed to upload proof file to ${JOB_PROOF_S3_URI}" >&2
302+
fi
303+
if ! s5cmd cp "$PROOF_JSON" "$CANONICAL_PROOF_S3_URI"; then
304+
echo "[prove_block.sh] Warning: failed to upload proof file to ${CANONICAL_PROOF_S3_URI}" >&2
298305
fi
299306
set -e
300307
else
301-
echo "[prove_block.sh] Warning: proof.json not found at $PROOF_JSON" >&2
308+
echo "[prove_block.sh] Warning: proof file not found at $PROOF_JSON" >&2
302309
fi
303310

304311
if [[ -f "$OUTPUT_PATH" ]]; then

0 commit comments

Comments
 (0)