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

Deployment scripts #288

Closed
wants to merge 141 commits into from
Closed

Deployment scripts #288

wants to merge 141 commits into from

Conversation

solanoepalacio
Copy link
Collaborator

@solanoepalacio solanoepalacio commented Mar 8, 2024 •
edited
Loading

Adds EVM deployment scripts to:

  • deploy managers
  • deploy transceivers
  • configure managers (set transceivers and any other configuration property)
  • perform managers and transceivers cross registrations
  • verify managers contracts on scanner
  • verify transceivers contracts on scanner
  • read current transceiver contract configuration
  • perform a test token transfer

To-do list:

  • Transceiver verification still doesn't work properly. The bytecode generated by the contract verification plugin doesn't match the deployed bytecode though upon close inspection the bytecode on the chain does match the bytecode on the build.
  • Document usage
  • Solana deployment scripts

@solanoepalacio solanoepalacio force-pushed the deployment-scripts branch 2 times, most recently from 9ecdc79 to dad0c1e Compare March 11, 2024 21:23
@scnale scnale force-pushed the deployment-scripts branch 2 times, most recently from e03ad78 to d8a0568 Compare March 13, 2024 19:47
@solanoepalacio solanoepalacio marked this pull request as ready for review March 22, 2024 13:36
@solanoepalacio solanoepalacio force-pushed the deployment-scripts branch 2 times, most recently from 8b8303e to b482d31 Compare April 11, 2024 21:17
kcsongor and others added 22 commits April 12, 2024 10:04
@kcsongor
solana: deterministic builds via Docker
92e6d84
@solanoepalacio
deployment scripts: adds deployment scripts boilerplate
d994ad6
@solanoepalacio
adds check-networks script
55849b0
@solanoepalacio
relocate contracts-json type
dc1d151
@solanoepalacio
update gitignore + minor cosmetic changes
07b6647
@solanoepalacio
manager and transceiver basic deployment scripts
7136fac
@solanoepalacio
configure managers script
20768d0
@solanoepalacio
adds testnet contract addresses
f8b4c1c
@solanoepalacio
minor cosmetic changes
34aa877
@solanoepalacio
adds testnet configuration values
ca342b8
@solanoepalacio
fixes json formatting
f9ddc94
@solanoepalacio
fixes script configs directory name
00956d5
@solanoepalacio
fixes scripts missing line-break escaping
93a95e5
@solanoepalacio
improves logs and other small fixes
36ef273
@solanoepalacio
update addresses to latest deployment
a9a88ad
@solanoepalacio
update peer addresses script
f725645
@solanoepalacio
minor refactor on the scripts configuration location
54a60a0
@solanoepalacio
new testnet deployment + verification scripts
8eddbbc
@scnale @solanoepalacio
evm: removes ESM flag for typechain
450db8d
@scnale @solanoepalacio
evm: fixes a couple of details in verification scripts
859275a
@scnale @solanoepalacio
evm: modifies API token field in verification scripts
64cba1b
@solanoepalacio
improves updatePeerAddress to only update when necessary
a7396fe
brianUtn98
brianUtn98 previously approved these changes Jun 6, 2024
View reviewed changes
Copy link

@brianUtn98 brianUtn98 left a comment

Choose a reason for hiding this comment

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

Approved with comments

solana/ts/scripts/config/devnet/quoter.json
@@ -28,27 +28,27 @@
"peerQuotes": {
"Sepolia": {
"maxGasDropoffEth": "0",
"basePriceUsd": "500000",
"nativePriceUsd": "3500000000",
"basePriceUsd": "1",
Copy link

Choose a reason for hiding this comment

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

shouldn't this be 0.5?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it should 🙌

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I actually commited the wrong file, I wanted to commit mainnet

solana/ts/scripts/config/devnet/quoter.json
"basePriceUsd": "500000",
"nativePriceUsd": "3500000000",
"basePriceUsd": "1",
"nativePriceUsd": "3865",
Copy link

Choose a reason for hiding this comment

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

is okay you updated here, but the assistant should take care of this value since sepolia will not be disabled

brianUtn98
brianUtn98 previously approved these changes Jun 6, 2024
View reviewed changes
Copy link

@brianUtn98 brianUtn98 left a comment

Choose a reason for hiding this comment

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

Approved now for Mainnet. Let a comment about this value

      "gasCost": 500000,

brianUtn98
brianUtn98 approved these changes Jun 6, 2024
View reviewed changes
Copy link

@brianUtn98 brianUtn98 left a comment

Choose a reason for hiding this comment

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

GasPrice could be updated as well to 29, but since the assistant should take care of that is fine

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

@barnjamin barnjamin barnjamin left review comments

@brianUtn98 brianUtn98 brianUtn98 requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@solanoepalacio @barnjamin @brianUtn98 @kcsongor @scnale @nonergodic