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

SHIP-1754 Consolidate FinalityDepth config #13702

Open
wants to merge 31 commits into
base: develop
Choose a base branch
from
Open

SHIP-1754 Consolidate FinalityDepth config #13702

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
fbaf4ea
Added missing FinalityDepth config in default toml files.
adityavadhavkar1215 Jun 26, 2024
5f1ede3
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
adityavadhavkar1215 Jun 27, 2024
926e1f0
added changeset to the PR.
adityavadhavkar1215 Jun 27, 2024
5f2b8c8
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
adityavadhavkar1215 Jun 27, 2024
dc4fb58
Updated docs/CONFIG.md
adityavadhavkar1215 Jun 27, 2024
7ff582c
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
adityavadhavkar1215 Jun 27, 2024
1f32b48
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
adityavadhavkar1215 Jun 28, 2024
703c28a
Updated test config to resolve test failure.
adityavadhavkar1215 Jun 28, 2024
d69c25b
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
adityavadhavkar1215 Jun 28, 2024
74457d1
Updated the changeset tag to internal.
adityavadhavkar1215 Jul 5, 2024
b967bc7
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
adityavadhavkar1215 Jul 5, 2024
ea5dc96
Addressed PR comments
adityavadhavkar1215 Jul 10, 2024
5ef0011
Merge branch 'SHIP-1754-Consolidate-all-FinalityDepth-settings' of gi…
adityavadhavkar1215 Jul 10, 2024
9cc0296
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
adityavadhavkar1215 Jul 10, 2024
e384dcd
Updated test values.
adityavadhavkar1215 Jul 10, 2024
7ee2841
Merge branch 'SHIP-1754-Consolidate-all-FinalityDepth-settings' of gi…
adityavadhavkar1215 Jul 10, 2024
39b39d6
Updated default config for Ethereum testnets
adityavadhavkar1215 Jul 10, 2024
963a389
Updated values for chains that did not have default value
adityavadhavkar1215 Jul 11, 2024
d7a75bc
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
adityavadhavkar1215 Jul 11, 2024
38d8b58
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
Madalosso Sep 30, 2024
864bc54
Revision: Testnet value from infra-k8s prod-testnet, mainnet from Nod…
Madalosso Sep 30, 2024
cfc08ec
save WIP
Madalosso Nov 6, 2024
77ade09
Update chain defaults using ccip repo as source of truth
Madalosso Nov 6, 2024
31c567b
gen docs
Madalosso Nov 6, 2024
6c262e6
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
Madalosso Nov 6, 2024
e569490
revert artbitrum finalityDepth values
Madalosso Nov 6, 2024
e6865e1
Undo comment deletion
Madalosso Nov 6, 2024
e3f41f6
fixes
Madalosso Nov 6, 2024
49c5f16
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
Madalosso Nov 7, 2024
9132246
FinalityTagBypass setup for chains with FinalityDepth<1000
Madalosso Nov 8, 2024
660f3ce
Merge branch 'develop' into SHIP-1754-Consolidate-all-FinalityDepth-s…
Madalosso Nov 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/wild-ducks-bake.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"chainlink": patch
---

#internal Added FinalityDepth config where it was missing
1 change: 1 addition & 0 deletions core/chains/evm/config/toml/defaults/Arbitrum_Mainnet.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
# Arbitrum is an L2 chain. Pending proper L2 support, for now we rely on their sequencer
ChainID = '42161'
ChainType = 'arbitrum'
FinalityDepth = 10
adityavadhavkar1215 marked this conversation as resolved.
Show resolved Hide resolved
LinkContractAddress = "0xf97f4df75117a78c1A5a0DBb814Af92458539FB4"
LogPollInterval = '1s'
# Arbitrum only emits blocks when a new tx is received, so this method of liveness detection is not useful
Expand Down
1 change: 1 addition & 0 deletions core/chains/evm/config/toml/defaults/Arbitrum_Sepolia.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
ChainID = '421614'
ChainType = 'arbitrum'
FinalityDepth = 10
adityavadhavkar1215 marked this conversation as resolved.
Show resolved Hide resolved
NoNewHeadsThreshold = '0'
OCR.ContractConfirmations = 1
LogPollInterval = '1s'
Expand Down
1 change: 1 addition & 0 deletions core/chains/evm/config/toml/defaults/Ethereum_Mainnet.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
ChainID = '1'
LinkContractAddress = '0x514910771AF9Ca656af840dff83E8264EcF986CA'
FinalityDepth = 10
adityavadhavkar1215 marked this conversation as resolved.
Show resolved Hide resolved
MinContractPayment = '0.1 link'
OperatorFactoryAddress = '0x3E64Cd889482443324F91bFA9c84fE72A511f48A'

Expand Down
1 change: 1 addition & 0 deletions core/chains/evm/config/toml/defaults/Ethereum_Sepolia.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
ChainID = '11155111'
LinkContractAddress = '0x779877A7B0D9E8603169DdbD7836e478b4624789'
FinalityDepth = 10
MinContractPayment = '0.1 link'

[GasEstimator]
Expand Down
1 change: 1 addition & 0 deletions core/chains/evm/config/toml/defaults/Fantom_Mainnet.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
ChainID = '250'
LinkContractAddress = '0x6F43FF82CCA38001B6699a8AC47A2d0E66939407'
FinalityDepth = 10
LogPollInterval = '1s'
NoNewHeadsThreshold = '30s'
RPCBlockQueryDelay = 2
Expand Down
1 change: 1 addition & 0 deletions core/chains/evm/config/toml/defaults/Fantom_Testnet.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
ChainID = '4002'
LinkContractAddress = '0xfaFedb041c0DD4fA2Dc0d87a6B0979Ee6FA7af5F'
FinalityDepth = 10
LogPollInterval = '1s'
# Fantom testnet only emits blocks when a new tx is received, so this method of liveness detection is not useful
NoNewHeadsThreshold = '0'
Expand Down
12 changes: 6 additions & 6 deletions docs/CONFIG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1761,7 +1761,7 @@ EVM defaults depend on ChainID:
AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0x514910771AF9Ca656af840dff83E8264EcF986CA'
LogBackfillBatchSize = 1000
Expand Down Expand Up @@ -3344,7 +3344,7 @@ GasLimit = 5400000
AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0x6F43FF82CCA38001B6699a8AC47A2d0E66939407'
LogBackfillBatchSize = 1000
Expand Down Expand Up @@ -4831,7 +4831,7 @@ GasLimit = 5400000
AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0xfaFedb041c0DD4fA2Dc0d87a6B0979Ee6FA7af5F'
LogBackfillBatchSize = 1000
Expand Down Expand Up @@ -5203,7 +5203,7 @@ AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
ChainType = 'arbitrum'
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0xf97f4df75117a78c1A5a0DBb814Af92458539FB4'
LogBackfillBatchSize = 1000
Expand Down Expand Up @@ -6597,7 +6597,7 @@ AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
ChainType = 'arbitrum'
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LogBackfillBatchSize = 1000
LogPollInterval = '1s'
Expand Down Expand Up @@ -6875,7 +6875,7 @@ GasLimit = 5400000
AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0x779877A7B0D9E8603169DdbD7836e478b4624789'
LogBackfillBatchSize = 1000
Expand Down
2 changes: 1 addition & 1 deletion testdata/scripts/node/validate/disk-based-logging-disabled.txtar
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,7 @@ ChainID = '1'
AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0x514910771AF9Ca656af840dff83E8264EcF986CA'
LogBackfillBatchSize = 1000
Expand Down
2 changes: 1 addition & 1 deletion testdata/scripts/node/validate/disk-based-logging-no-dir.txtar
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,7 @@ ChainID = '1'
AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0x514910771AF9Ca656af840dff83E8264EcF986CA'
LogBackfillBatchSize = 1000
Expand Down
2 changes: 1 addition & 1 deletion testdata/scripts/node/validate/disk-based-logging.txtar
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,7 @@ ChainID = '1'
AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0x514910771AF9Ca656af840dff83E8264EcF986CA'
LogBackfillBatchSize = 1000
Expand Down
2 changes: 1 addition & 1 deletion testdata/scripts/node/validate/invalid.txtar
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,7 @@ ChainID = '1'
AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0x514910771AF9Ca656af840dff83E8264EcF986CA'
LogBackfillBatchSize = 1000
Expand Down
2 changes: 1 addition & 1 deletion testdata/scripts/node/validate/valid.txtar
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -312,7 +312,7 @@ ChainID = '1'
AutoCreateKey = true
BlockBackfillDepth = 10
BlockBackfillSkip = false
FinalityDepth = 50
FinalityDepth = 10
FinalityTagEnabled = false
LinkContractAddress = '0x514910771AF9Ca656af840dff83E8264EcF986CA'
LogBackfillBatchSize = 1000
Expand Down
Loading