release/2.5.0 -> master #10622
Merged
release/2.5.0 -> master #10622
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ber is added to the left side of the filter condition (#9968)
* update VL contracts * update
Introduce a chain-agnostic LogPollerWrapper that will be provided by the Relayer. LogPollerWrapper needs to support Listener, ConfigProvider and ContractTransmitter (+ potentially more components in the future).
* (test): Add integration test for contracts v1 * (fix): V1 listener handle correctly stops awaiting * Requested changes * Change v1 integration test port * Increase MaxReportLengthBytes from 10k -> 15k
(cherry picked from commit d0c54ea)
1. Subscribe to route updates in poller, transmitter and digester. 2. LogPollerWrapper support for v0 contracts (single, synchronous update). 3. LogPollerWrapper subscription management. 4. Stub of public API for the Listener (latest request/response).
* golf acceptTermsOfService * golf createSubscription * packing them structs * Golf Commitment a bit more * golf deployment cost (doesn't count :( ) * golf storage layout FunctionsSubscriptions * Storage layout of FunctionsBilling * add foundry setup test * Mostly comments and code improvements + micro golf * I'm so sorry for stealing the mapping Deividas * This doesn't give points but has annoyed me for too long OCR2 doesn't pack nicely, even though it could combine s_owner, s_configCount and s_latestConfigBlockNumber into the same slot. My issue is mostly with ConfirmedOwnerWithProposal somehow making s_owner completely unpackable by following it with the not remotely as useful s_pendingOwner. Again, no gas savings for this competition but let's all agree that this is simply wrong * removing unused imports * unchecked ++i, please don't use this in prod * fix rebase issues * pack Commitment struct * pack Commitment * no gas in measurement but saves a storage slot * fix test, add type safety to config * undo ConfirmedOwnerWithProposal which has 0 gas impact 0 in the gas measurement test, but non-zero in setting OCR configs but since this is a shared file and CI is very angry if old contracts change Ill just undo this part * Revert v0_0_0 changes * Remove unchecked loop increments, replace with comments documenting bounds * Regenerate geth wrappers * (test): Amend go integration test client contract method --------- Co-authored-by: Rens Rooimans <[email protected]>
* Removed unnecessary _nonReentrant call from oracleWithdraw * Changed access from internal to external for ownerWithdraw * More reentrancy guard removal * Set FunctionsClient and Routable routers to immutable * Keep subscription methods protected against reentrancy * (test): Add reentrancy unit tests * Remove reentrancy lock because of CEI pattern's protection (-8500 gas) * Amend line from merging develop * Regenerate geth wrappers * (test): Lower test's sub funding to prevent flakes from gasprice fluctuations * (test): Pin tx.gasprice for reentrancy test --------- Co-authored-by: Justin Kaseman <[email protected]>
* Use ReadAccessController interface within ToS Allowlist & add method for get all * (test): Amend ToS test for new getter name * Remove ToS Allow List's overloaded .hasAccess()
…9992) Partial credit goes to @justinkaseman - this PR re-uses parts of PR #9979.
* Adding reqCount to VRFV2+ Subscription * Prettify sol files * Prettify sol interface files * Updated comment --------- Co-authored-by: Sri Kidambi <[email protected]>
* move test/vendor -> shared/vendor * move OZ folder shared/vendor/openzeppelin-solidity/v.4.8.0 -> vendor/openzeppelin-solidity/v4.8.0 * bump mock helper & dev code version dependency removes the need to include a v4.3.1, 4.7.3 should behave the same way * rm moved OZ folder * move OZ and CBORUtils * move last shared vendor to top level vendor * move arb and optimism dev vendor * move entrypoint and arb-bridge-eth * move address to correct vendor
* Decouple chainlink client used in tests Chainlink node client should be decoupled from K8s. This allows to use it in non-k8s environments like local docker containers. * Fix k8 client * Fix
* getter function * rename * config changes * removed flag from ocr1 configs * test update + config-docs * 1 CONFIG.md 1 line * tests and toml defaults added * default values/variables for testdata/.../validate * rename + embed/testdata flag * toml fixed
* Hardcoded exact gas used causing issues in test, testing against range * Compare specific event log fields while skipping the gas payment field --------- Co-authored-by: Sri Kidambi <[email protected]>
* rename FallbackGasPriceUAtom to FallbackGasPrice * revert config.md formatting * one test passes * update TestTxm_Integration to use uatom * update cosmos packages * update tests * Add Base soak test config (#9860) * Base(Mainnet) + Optimism hotfix * Add soak config * revert ocr config changes * adjust confirmations * bump cl env and ctf * Add patch * Add patch * Update base test config --------- Co-authored-by: simsonraj <[email protected]> * bump libocr to c5e49b4195ac49bbf9ea3b122f24b3e19dbb8c4a (#9891) * fix arb min gas bug (#9872) * add admin privilege config (#9902) * update cosmos packages * add await logic * add back go tags * update imports * tools/ci: add install_wasmd script * .github/actions: add initial setup-wasmd action * .github/workflows: add setup-wasmd step to core workflow * update wasmd version * remove comment * remove debugging code * generate new wasmbech32 address * tidy * config docs * tidy * postgres version --------- Co-authored-by: Calvin Wang <[email protected]> Co-authored-by: Calvin <[email protected]> Co-authored-by: David Cauchi <[email protected]> Co-authored-by: simsonraj <[email protected]> Co-authored-by: Jordan Krage <[email protected]> Co-authored-by: Ryan Hall <[email protected]> Co-authored-by: cfal <[email protected]>
* wip * wip - cron * wip - 5+ tests, ragep2p issues * rollback ocr/ocr2 * comments, cleanup * increase timeouts * rollback go.mod * re-use mercury env as a boilerplate * re-use mercury env as a boilerplate * fix action's Go version * update go.mod from develop * explicitly login into aws registry * update deps * set explicit go version * re-trigger * github-action v2.2.2 * fix the client * reusable envs * review fixes * review fixes
* Bump version and update CHANGELOG for core v2.3.0 * Add base config (#9865) * Add base config * Updated Docs * Base goerli config to match Optimism goerli --------- Co-authored-by: simsonraj <[email protected]> (cherry picked from commit ee724fc) * Base(Mainnet) + Optimism hotfix (#9862) * Base(Mainnet) + Optimism hotfix * Revert "Base(Mainnet) + Optimism hotfix" This reverts commit a13e848. * Base+ optimism hotfix for non-successfull ocr rounds * fix tests (cherry picked from commit 38e9e4a) * Added automation gaslimit for Base (#9878) (cherry picked from commit 22a60fb) * re run config docs generate * Finalize date on changelog for 2.3.0 --------- Co-authored-by: David Cauchi <[email protected]> Co-authored-by: Simson <[email protected]>
* Move FulfillResultCodes to interfaces folder * Rename Function.sol setArgs -> addArgs * Rename Routable s_config_hash -> s_configHash * Make Routable updateConfig public for re-use in constructor * Move comment out of IConfigurable * Make Routable s_router private with an internal getter * Rename constant to ROUTER_ID * Rename RouterBase s_config_hash -> s_configHash * Rename Routable.sol -> HasRouter.sol * Add IOwnableRouter interface * FunctionsSubscriptions MustBeSubOwner -> MustBeSubscriptionOwner * Make RouterBase variable private * RouterBase togglePause -> pause/unpause * Use full semantic version in typeAndVersion * ToS Allow List rename proof -> signature * Change acceptTermsOfService arguments from signature to raw r,s,v * Prettier format 💅 * Remove RouterBase versioning * Move RequestTimedOut event to Router * Rename requestSubscriptionOwnerTransfer -> proposeSubscriptionOwnerTransfer * (test): Update tests for changes * (test): Add v0 gas test * Regenerate geth wrappers * use ConfirmedOwner and uint256 * Revert _handleConfigUpdate * Add RequestDeleted event to FunctionsBilling * Use uint256 in loops * Use separate import for ITypeAndVersion * Use public constants for typeAndVersion getters * (test): Use v0.8 mocks * Regen geth wrappers --------- Co-authored-by: Rens Rooimans <[email protected]>
chainchad
requested review from
a team,
javuto,
jkongie,
jmank88,
samsondav,
bolekk,
justinkaseman,
KuphJr,
pinebit,
se3000,
connorwstein,
sdrug,
martin-cll,
prashantkumar1982,
krehermann,
PiotrTrzpil,
archseer,
cfal and
reductionista
as code owners
chainchad
temporarily deployed
to
build-publish
GitHub Actions
Inactive
|
SonarQube Quality Gate
See analysis details on SonarQube Fix issues before they fail your Quality Gate with |
patrickhuie19
approved these changes
Sep 13, 2023
There was a problem hiding this comment.
Digestable diff is https://github.com/smartcontractkit/chainlink/pull/10628/files, which adds default configs for the Base chain.
LGTM!
|
CI is choking on DB errors: This is fixed on develop. Not sure how to handle it here. |
jmank88
approved these changes
Sep 13, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Digestible diff