Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev: script for relayer local devnet #2465

Merged
merged 2 commits into from
May 3, 2023
Merged

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented May 3, 2023

Adds a repeatable script for debugging and testing the relayer, which attempts to build a path between the preview deployment and a local devnet, ignoring testnet entirely. This allows us to test the very latest versions of the Penumbra code, and rules out any variation caused by proto definition changes or other code drift.

Renames the run-relayer to build-path"to describe accurately what it does. After path is built, then one can run rly start but we're not quite ready for that yet.

Also updates the configure-relayer script to target devnet & preview, rather than preview & testnet. N.B. the directionality is important here: we want to be able to inspect the local devnet logs and see the relayer's events in pd's logs.

Maybe worth editing the scripts to allow for this customization ad-hoc, not bothering right now.

Adds a repeatable script for debugging and testing the relayer,
which attempts to build a path between the preview deployment and a
local devnet, ignoring testnet entirely. This allows us to test the very
latest versions of the Penumbra code, and rules out any variation
caused by proto definition changes or other code drift.

Renames the `run-relayer` to `build-path"`to describe accurately what it
does. After path is built, then one can run `rly start` but we're not
quite ready for that yet.

Also updates the `configure-relayer` script to target devnet & preview,
rather than preview & testnet. N.B. the directionality is important here:
we want to be able to inspect the local devnet logs and see the relayer's
events in pd's logs.

Maybe worth editing the scripts to allow for this customization ad-hoc,
not bothering right now.
@conorsch conorsch temporarily deployed to smoke-test May 3, 2023 22:09 — with GitHub Actions Inactive
@conorsch conorsch marked this pull request as ready for review May 3, 2023 22:10
@conorsch conorsch requested a review from avahowell May 3, 2023 22:11
@conorsch
Copy link
Contributor Author

conorsch commented May 3, 2023

We've been using this to spin up testing environments while chasing #2330, submitting to main so we don't have to keep rebasing it.

Follow up to #2464, where we forgot the format macro. What we really
want is more spans, as with the connection_open logic, but we'll get to
that shortly.
@conorsch conorsch temporarily deployed to smoke-test May 3, 2023 23:18 — with GitHub Actions Inactive
@conorsch conorsch merged commit 2446bd2 into main May 3, 2023
@conorsch conorsch deleted the relayer-devnet-scripting branch May 3, 2023 23:33
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.

1 participant