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.

Sign up for GitHub

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

Jump to bottom

feat: main, test config: anchor IBC denoms, encon committee addresses #6437

Merged
merged 5 commits into from
Oct 14, 2022
Merged

feat: main, test config: anchor IBC denoms, encon committee addresses #6437

merged 5 commits into from
Oct 14, 2022

Conversation

dckc
Copy link
Member

@dckc dckc commented Oct 11, 2022 •
edited
Loading

closes: #6284
refs: #6435

Description

  • Econ committee addresses for main net
  • PSM anchorAssets configurations
    • local testing: decentral-psm-config.json
      • Keep AUSD for compatibility with ad-hoc integration test scripts in agoric-cli
      • add USDC_axl to test multiple coins, token icons, etc.
    • devnet, ollinet anchorAssets
      • Keep AUSD for compatibility
      • use {USDC,USDT}_{axl,grv} look-alikes for ephemeral nets, without relayer-based denoms
    • test (emerynet) denoms
      • Toy USD to start with
      • Don't use up keywords such as USDC_axl; vote them in as relayers become available
    • main denoms
      • USDC_axl
      • USDC_grv
      • USDT_axl
      • USDT_grv

Security Considerations

  • anchor assets designated by these denoms rely on availability of relayers
  • the usual cosmos wallet key management risks apply to econ committee addresses

Documentation Considerations

Perhaps the Validator Guide should say something about choosing between main-psm and test-psm configs?

Testing Considerations

For the anchor assets:

  • IBC denoms are the results of actual transactions
  • booting a local chain with this anchorAssets configuration has been tested (though not trading and such)

We plan to continue to use test gov addresses for emerynet.

@dckc dckc changed the title feat: mainnet, emerynet config: anchor IBC denoms, encon committee addresses feat: main, test config: anchor IBC denoms, encon committee addresses Oct 11, 2022
@dckc dckc force-pushed the dc-config-anchors-voters branch 8 times, most recently from d2ed0f1 to 85349fe Compare October 13, 2022 05:26
dckc added 3 commits October 13, 2022 00:38
@dckc
test(vats): use {USDC,USDT}_{axl,grv} look-alikes for ephemeral nets
e96396b 
Where we don't expect to have actual relayers, include production
names for screenshots and such.

Keep the AUSD keyword for compatibility with agoric-cli integration
test scripts.
@dckc
feat(vats): test config doesn't use up production token keywords
ade318b 
For our long-lived testnet, we expect to have relayers, but they're
not in place yet, so we don't know the final IBC denoms yet.
We can start with just a toy token and vote the other tokens in
as relayers become available.
@dckc
feat: anchorAssets config for {USDC,USDT}_{axl,grv} on main
b938a79
@dckc dckc force-pushed the dc-config-anchors-voters branch from 85349fe to 419385c Compare October 13, 2022 05:38
@dckc dckc marked this pull request as ready for review October 13, 2022 05:38
@dckc
Copy link
Member Author

dckc commented Oct 13, 2022

@arirubinstein @Tartuffo let's cut emerynet relayers out of the critical path by putting only Toy USD in the emerynet config (decentral-test-psm-config.json). As emerynet relayers for Axelar and Gravity come online, we can add those tokens by governance vote (#6021).

@dckc dckc force-pushed the dc-config-anchors-voters branch from 419385c to 548c096 Compare October 13, 2022 06:10
Tartuffo
Tartuffo approved these changes Oct 14, 2022
View reviewed changes
Copy link
Contributor

@Tartuffo Tartuffo left a comment

Choose a reason for hiding this comment

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

LGTM!

michaelfig
michaelfig approved these changes Oct 14, 2022
View reviewed changes
Copy link
Member

@michaelfig michaelfig left a comment

Choose a reason for hiding this comment

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

Initial mainnet PSM assets verified, both IBC denom traces and decimal places.

@dckc dckc added the automerge:rebase Automatically rebase updates, then merge label Oct 14, 2022
@mergify mergify bot merged commit 3c37418 into master Oct 14, 2022
@mergify mergify bot deleted the dc-config-anchors-voters branch October 14, 2022 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tartuffo Tartuffo Tartuffo approved these changes

@michaelfig michaelfig michaelfig approved these changes

@arirubinstein arirubinstein Awaiting requested review from arirubinstein

@dtribble dtribble Awaiting requested review from dtribble

@btulloh btulloh Awaiting requested review from btulloh

@rowgraus rowgraus Awaiting requested review from rowgraus

Assignees
No one assigned
Labels
automerge:rebase Automatically rebase updates, then merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

PSM Deployment for Mainnet
3 participants
@dckc @michaelfig @Tartuffo