Skip to content

Fix solana programs admin key naming#3013

Merged
isaacsolo merged 4 commits into
masterfrom
is-fix-solana-start
May 2, 2022
Merged

Fix solana programs admin key naming#3013
isaacsolo merged 4 commits into
masterfrom
is-fix-solana-start

Conversation

@isaacsolo

Copy link
Copy Markdown

Description

Currently, solana-programs container is broken and doesn't seed the first solana transactions. This change fixes that but leaves the rest of the naming aside for https://github.com/AudiusProject/audius-protocol/pull/3011/files.

Tests

solana-programs start works.

How will this change be monitored? Are there sufficient logs?

No monitoring.

@isaacsolo isaacsolo requested a review from jowlee May 2, 2022 16:41
Comment thread solana-programs/start.sh
"anchorAdminPrivateKey": "$admin_keypair_privatekey",
"anchorAdminStoragePublicKey": "$admin_storage_keypair_publickey",
"anchorAdminStoragePrivateKey": "$admin_storage_keypair_privatekey",
"anchorAdminPublicKey": "$admin_authority_keypair_publickey",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

are there references to these values that we have to propagate? in local setup for DP etc.

@isaacsolo isaacsolo May 2, 2022

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

i think so! i've got a follow up PR that will rename even more but wanted to unblock first.
#3011

@isaacsolo isaacsolo force-pushed the is-fix-solana-start branch from 03add7b to 1ae1811 Compare May 2, 2022 18:12
@isaacsolo isaacsolo merged commit ed4fa05 into master May 2, 2022
@isaacsolo isaacsolo deleted the is-fix-solana-start branch May 2, 2022 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants