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.

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

Chain writer config for Solana plugin oracle creator logic #15848

Open
wants to merge 37 commits into
base: solana-offchain-plugin
Choose a base branch
from

Conversation

huangzhen1997
Copy link
Contributor

@huangzhen1997 huangzhen1997 commented Jan 6, 2025

Jira ticket https://smartcontract-it.atlassian.net/browse/NONEVM-934

This PR remove EVM specific dependency, and add Solana support for chain writer config in plugin logic. At the moment, the Solana program address required for the plugin are not yet available. They will be configured later in a different PR, and an empty place holder will be used now for unblocking E2E test. https://chainlink-core.slack.com/archives/C0779R3AH8R/p1736289787018579?thread_ts=1736269231.525509&cid=C0779R3AH8R

Copy link
Contributor

github-actions bot commented Jan 7, 2025

AER Report: CI Core ran successfully ✅

aer_workflow , commit

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

}

solConfig := chainwriter.ChainWriterConfig{
Programs: map[string]chainwriter.ProgramConfig{
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm assuming this will be filled in sometime in the future with a full config referencing all the accounts needed for each method similar to the ccip_config_example?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes we will update once again in the future, I guess I can add available config now like the system program etc.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, the config is very much in flux at the moment, I'm working with Aaron to get it to be accurate with the current on-chain code, but there are still some open questions I have. I'll let you know when it's ready but you can see the latest version in this PR smartcontractkit/chainlink-solana#1009

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.

5 participants