Skip to content

script: deploy receptor #602

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

Merged
merged 3 commits into from
Oct 4, 2024
Merged

script: deploy receptor #602

merged 3 commits into from
Oct 4, 2024

Conversation

tserg
Copy link
Collaborator

@tserg tserg commented Oct 3, 2024

This PR adds a script to deploy receptor on mainnet. It does not grant permission yet for the Receptor to call shrine.update_yin_spot_price(), which can be done when the UI is ready.

Initial parameters are ~16.6 mins for update frequency (same as Seer), and 3hrs for TWAP.

I initially wanted to add support for Sepolia, but the prices on Sepolia are too way off (USDC/CASH is ~1.74, but DAI/CASH is ~0.00024) to be meaningful. This can be verified here using the fake USDC (0x07ab0b8855a61f480b4423c46c32fa7c553f0aac3531bbddaa282d86244f7a23) and fake DAI (0x06f2a0dfeff180133de890ad69c6ba294574c5f34a67890fd22464f348c4d03c) by Ekubo.

@tserg tserg requested a review from milancermak October 3, 2024 08:04
Copy link
Contributor

@milancermak milancermak left a comment

Choose a reason for hiding this comment

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

LGTM.

The deploy script is idempotent, right? So running it again should deploy only the Receptor in this case?

@tserg
Copy link
Collaborator Author

tserg commented Oct 3, 2024

The deploy script is idempotent, right? So running it again should deploy only the Receptor in this case?

With the state file, yes.

Anyway, I will proceed with deployment on mainnet and update the state file in this PR.

@tserg
Copy link
Collaborator Author

tserg commented Oct 3, 2024

Deployment is successful! I have updated the state file and address.

@milancermak
Copy link
Contributor

@tserg tserg merged commit 5808049 into main Oct 4, 2024
1 check passed
@tserg tserg deleted the script/receptor branch October 4, 2024 13:27
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