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

CCIP-3914 Cherry-pick OTI head report #1507

Closed
wants to merge 140 commits into from

Conversation

bukata-sa
Copy link
Contributor

anirudhwarrier and others added 30 commits July 19, 2024 15:46
* fix automation chaos test

* fix ondemand workflow for chaos tests

* use standard automation test functions

* update automation testconfig

* lint

* lint
* Remove the old log buffer

* Add a changeset

* Update integration test

* Lint

* Clean up from config

* Update tests

* Integration test lint

* Clean up sync filters
Wait for the handler to be called before proceeding.
* Notify slack on new release tags and release branches

* Add changelog url

* Use released version of action

* Only notify slack on tags
* [KS-370] Add ConfigForCapability to registry

- This allows us to set some config inside the registry without needing
  to specify it on the capability registry

* Update common

* Update PR

* Linting

* Rename ExecuteConfig -> DefaultConfig

* Add IsLocal to CapabilityInfo

* Minor review feedback; rename GetLocalNode -> LocalNode

* More comments

* Remote CapabilityID type

* Stop coercing capability.ID since it's already a string

* Update common

* Linting / Go generate

* Update common

* Update common
* Update CHANGELOG and package version for contracts release

* Add today's date to CHANGELOG 1.2.0
* Add updateFromPrevious method to Functions ToS contract

* Update test

* Update gethwrappers

* Added test

* prettier

* Address feedback

* Added tag to changeset

* Addressed feedback on tests

* Updated gas snapshot

* Add missing deployment arg to ToS in core tests

* New approach which enables batching

* Add require

* prettier, gas snapshot & wrappers

* update snapshot

* addressed feedback

* prettier

---------

Co-authored-by: app-token-issuer-infra-releng[bot] <120227048+app-token-issuer-infra-releng[bot]@users.noreply.github.com>
* prevent malicious a node operator from taking over another node belonging to another node operator

* prevent malicious node operator from becoming the admin for another node operator
* Bump version and update CHANGELOG fore core v2.14.0

Signed-off-by: Sneha Agnihotri <sneha.agnihotri@smartcontract.com>

* Bump libocr to fix nil msg error (#13919)

* Finalize date on changelog for 2.14.0 (#13943)

---------

Signed-off-by: Sneha Agnihotri <sneha.agnihotri@smartcontract.com>
Co-authored-by: chainchad <96362174+chainchad@users.noreply.github.com>
Signed-off-by: chainchad <96362174+chainchad@users.noreply.github.com>
(cherry picked from commit 7147653)
(cherry picked from commit 7147653)
…changelog-fix

Use changeset github changelog generator
Signed-off-by: chainchad <96362174+chainchad@users.noreply.github.com>
## Motivation
Use new ccip tests designed with deployment in CI 

## Solution

---------

Co-authored-by: Makram <makramkd@users.noreply.github.com>
Fetch all the token prices in the jobspec for dest tokens
kalverra and others added 18 commits September 17, 2024 15:19
ZK chains introduce the need to send CCIP messages out of order. This
change enables us to do so at will in our E2E tests.

---------

Co-authored-by: Balamurali Gopalswami <balamurali.gopalswami@smartcontract.com>
## Background
[Original PR](#1308) -
there is some discussion here that is possibly still relevant)
[Open PR to add
cursing](#1400) - I copied
the relevant parts for the `RMNRemote`.

Note: we are intentionally separating `RMNRemote` and `RMNHome` since
they have different timelines.

## Motivation
Productionize the RMNRemote and include it in integration tests

---------

Co-authored-by: Makram <makramkd@users.noreply.github.com>
## Motivation

#1431 (comment)

## Solution
Expose the constant as public
## Motivation
RMN contract is updated to 1.5 version. Need to update in config.

## Solution
Update config with latest 1.5 RMN contract address and remove unwanted
secondary config.
## Motivation
The prior approach to find if there is traffic in a lane was not
detecting the transactions properly. It could be due to the
`CCIPSendRequestedWatcher` captures the event after it is initiated not
any transaction before that.

## Solution
The revised approach is to use the FilterCCIPSendRequested by a past
derived block number and see if there is any traffic. The block number
is derived by this formula:
`filterFromBlock = latestBlockNumber -
(SkipRequestIfAnotherRequestTriggeredWithin/avgBlockTime)`
By this approach, we will be able to find traffic better.

https://smartcontract-it.atlassian.net/browse/CCIP-3420
* Added gas limit estimation feature to EVM gas estimators

* Added changeset

* Fixed linting

* Added new failure tests

* Fixed test

* Fixed mock configs in ccip capabilities

* Fixed configs

* Fixed test

* Refactored GetFee method

* Added EstimatedGasBuffer and addressed feedback

* Fixed linting

* Fixed config doc and tests

* Addressed feedback

* Fixed typo

* Repurposed LimitMultiplier to be used as a buffer for estimated gas

* Updated mocks

* Removed LimitMultiplier if gas estimation fails and updated config docs

* Fixed Broadcaster test

* Enabled uncapped gas limit estimation if provided fee limit is 0

* Updated estimate gas buffer to be a fixed value instead of using LimitMultiplier

* Updated log message

* Updated logs

---------

Co-authored-by: Prashant Yadav <34992934+prashantkumar1982@users.noreply.github.com>
* Set from address for gas limit estimation

* Updated tests

* Added changeset
* Updated EstimateGasLimit config name to EstimateLimit

* Updated mocks

* Fixed linting

* Updated changeset
* Add Mantle errors

* Add tests for Mantle errors

* changeset

* Update seven-kiwis-run.md
* edited op stack oracle to include tokenRatio for Mantle

* typo fix

* fixed syntax errors

* fixed compilation errors in test

* added unit tests for Mantle oracle changes

* typo fix

* added changeset file

* added hashtag to changeset

* changed unit test to include new chaintype

* changed test to include new chaintype

* typo fix

* addressed alphabetical order and error issues

* used batch call instead of separate calls for L1 Base Fee

* added test cases for RPC errors to Mantle
* make Mantle use vanilla OP gas price oracle

* changeset

* bring back l1 oracle

* update mock

* [BCI-4072] changeset update

---------

Co-authored-by: valerii.kabisov <valerii.kabisov@smartcontract.com>
Co-authored-by: Valerii Kabisov <172247313+valerii-kabisov-cll@users.noreply.github.com>
* Introduce universal estimator

* Fixes

* Use WeiMin to cap bump price

* Update connectivity logic

* Fix error

* Fixes

* Cover an edge case when enforcing limits

* Add changeset

* Update mempool check logic

* Update config names

* Convert Universal Estimator to service

* Client changes to support UE

* Introduce configs

* Update mocks

* Fix lint

* Fix test cases

* Fix mockery

* Fix test cases

* Update comment

* Fix Start/Close sync issue

* Address feedback

* Fix lint

* Fix lint

* More changes

* Add more comments

* Fix merge conflicts

* Update CONFIG

* Rename to FeeHistory estimator

* Rename

* Exclude zero priced priority fees

* Remove HasMempool

* Remove testing commit

* Fixes

* Add DefaultJitter

* Add optimizations

* Fix testscripts

* Fix name

* Update error messages
* Add node level OOC error

* Add changeset
* feat(oti-head-report): Report new heads to atlas' OTI

* tests

tests

review fixes

ci fixes

ci fixes

telemetry reporter test

fix chain in proto

fix chain in proto

fix ci

changeset

fix config test

review fix

fix config test

fix docs test

fix config testscript

fix config testscript

fix test

* move to relayer

move to relayer

move service to evm

* go generate

* move config

* simplify

* review

* Revert moving to relayer

* review

* review

* head telemetry enabled by default

* review

* rebase

* review

* drop config

* rebase

* review

* regenerate

* cl node timeout

* review fix

* rebase

---------

Co-authored-by: Jordan Krage <jmank88@gmail.com>
* [CCIP-3072] Info log on broken chain

* sort imports

---------

Co-authored-by: Lukasz <120112546+lukaszcl@users.noreply.github.com>
* fix: non-zero default period

* Update eight-bees-speak.md
* fix: head report chain_id

* test
@bukata-sa bukata-sa requested a review from a team as a code owner October 22, 2024 11:56
@bukata-sa bukata-sa changed the base branch from ccip-develop to release/2.14.0-ccip1.5 October 22, 2024 11:57
@bukata-sa bukata-sa closed this Oct 22, 2024
@bukata-sa bukata-sa deleted the head-report-cherry-pick branch October 22, 2024 11:58
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.