Skip to content

Conversation

@dkijania
Copy link
Member

@dkijania dkijania commented Nov 19, 2025

Implementing CI test for block race test. In order to implement it i have to slightly adjust scripts .

@dkijania
Copy link
Member Author

!ci-build-me

2 similar comments
@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania dkijania force-pushed the ci_for_rosetta_block_race_test branch from b32555d to b3582c4 Compare November 19, 2025 23:57
@dkijania
Copy link
Member Author

!ci-build-me

1 similar comment
@dkijania
Copy link
Member Author

!ci-build-me

@dkijania dkijania force-pushed the ci_for_rosetta_block_race_test branch from c5b84d2 to 0fc75a4 Compare November 20, 2025 20:38
@dkijania
Copy link
Member Author

!ci-build-me

10 similar comments
@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania
Copy link
Member Author

!ci-build-me

@dkijania dkijania force-pushed the ci_for_rosetta_block_race_test branch from 72c42de to 7733e53 Compare November 25, 2025 21:32
@dkijania
Copy link
Member Author

!ci-build-me

@dkijania dkijania marked this pull request as ready for review November 25, 2025 21:34
@dkijania dkijania requested review from a team as code owners November 25, 2025 21:34
@dkijania dkijania force-pushed the ci_for_rosetta_block_race_test branch from 7733e53 to 3973a55 Compare November 25, 2025 21:36
@dkijania
Copy link
Member Author

!ci-build-me

1 similar comment
@dkijania
Copy link
Member Author

!ci-build-me

@dkijania dkijania force-pushed the ci_for_rosetta_block_race_test branch from 59d82b9 to b375200 Compare November 26, 2025 12:53
@dkijania
Copy link
Member Author

!ci-build-me

@dkijania dkijania force-pushed the ci_for_rosetta_block_race_test branch from b375200 to e665e82 Compare November 26, 2025 14:35
@dkijania
Copy link
Member Author

!ci-build-me

LEDGER_ARCHIVE="ledger.tar"
LEDGER_DIR="ledger"

echo "Downloading ledger archive from $LEDGER_URL ..."
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest fusing this so we're not wasting I/O to store the tar itself.

echo "Downloading and extracting ledger archive from $LEDGER_URL ..."
mkdir -p "$LEDGER_DIR"
curl -L "$LEDGER_URL" | tar -x -C "$LEDGER_DIR"


echo "Running warmup mina advanced test submit-to-archive (1 block, 1 payment, 0 zkapp)..."
TMP=tmp MINA_PRIVKEY_PASS="$MINA_EXE" advanced test submit-to-archive \
TMP=tmp MINA_PRIVKEY_PASS="" "$MINA_EXE" advanced test submit-to-archive \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is TMP? Is this AI hallucination?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was in the original script, done by George. I think this is passing TMP down to inner script as we are not running export TMP

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was in the original script, done by George. I think this is passing TMP down to inner script as we are not running export TMP

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you point to me where it's used? Can't find it at all

@glyh
Copy link
Member

glyh commented Dec 3, 2025

!ci-build-me

1 similar comment
@dkijania
Copy link
Member Author

dkijania commented Dec 4, 2025

!ci-build-me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants