-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[TT-1046] Add Chainlink node test config as TOML #13142
Conversation
b1811a1
|
||
# Chainlink node TOML config | ||
[NodeConfig] | ||
BaseConfigTOML = """ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have to nest this TOML? Depending on the TOML lib, I think it would be possible to use a type for this field which defers parsing (like json.RawMessage
), so that we could handle it as dynamic TOML instead of an opaque string.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do like this idea, but my only doubt is that it will work differently than it does now in CCIP and idk if it's worth diverting.
Quality Gate passedIssues Measures |
What was done?
CLNode
were left untouchedChainlink nodes funding has been increased for some tests, due to default GasEstimator settings that's higher than it was in the past. If that's an issue, please let me know!