Skip to content

Conversation

mujahidkay
Copy link
Member

Description

This PR pushes the configuration changes needed for deploying a noble testnet node and relayer b/w agoric and itself

Comment on lines +72 to +73
relayer keys restore "$INTERNAL_CHAIN_NAME" user1 "cinnamon legend sword giant master simple visit action level ancient day rubber pigeon filter garment hockey stay water crawl omit airport venture toilet oppose"
relayer keys restore "$EXTERNAL_CHAIN_NAME" nobleuser1 "stamp later develop betray boss ranch abstract puzzle calm right bounce march orchard edge correct canal fault miracle void dutch lottery lucky observe armed"
Copy link
Member Author

Choose a reason for hiding this comment

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

instead of using mnemonics here I think we should env vars here and pass the actual values via configMapGenerator literals from private overlays

INTERNAL_CHAIN_GAS_DENOM=${INTERNAL_CHAIN_GAS_DENOM:-ubld}
INTERNAL_CHAIN_ID=${INTERNAL_CHAIN_ID:-agoricdev-23}
INTERNAL_CHAIN_NAME=${INTERNAL_CHAIN_NAME:-agoric}
INTERNAL_CHAIN_RPC="https://devnet.rpc.agoric.net:443"
Copy link
Member Author

Choose a reason for hiding this comment

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

can only use devnet rpc as fallback - like other internal chain vars

use_keys
add_path
start_relayer
sleep infinity No newline at end of file
Copy link
Member Author

Choose a reason for hiding this comment

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

not sure if this is still needed

@mujahidkay mujahidkay self-assigned this Feb 10, 2025
}

start_relayer() {
relayer start
Copy link
Contributor

Choose a reason for hiding this comment

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

I am thinking is it possible to be able to use a consistent channel?

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.

2 participants