-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Feature/release/2.26.0 aptos workflow fix #18764
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
Feature/release/2.26.0 aptos workflow fix #18764
Conversation
* derive name based on filter params * rm unused * derive filter name from filter params * add comments * fix linter * fix register filter
* add rm duplicated filters solana migration * add rm duplicated filters solana migration * fix query * add subquery alias * tidy
* add filter deduplication migration for evm * fix annotation * fix partition by * add default for is_legacy_name
…snoozing to allow for attestation check in the next round (#18552)
* Disabling headtracker's persistence layer for some chains * bump * Regenerate configs * Regenerate configs * Regenerate configs * Regenerate configs
* [CRE-533] Real signatures in fake consensus capability * Fixes chainlink version in CI --------- Co-authored-by: Adam Hamrick <adam.hamrick@smartcontract.com>
* Add keystore service to standardcapabilities delegate * Update chainlink-common version, fix trigger initialise func * fix nil pointer * fix lint * update gomodtidies * add p2p key use * undo removal of function * Update core/capabilities/integration_tests/keystone/contracts_setup.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * lowercase getSignerStringFromOCRKeyBundle * add error for no p2p key * fix merge conflict * lint * add changeset * add error check * lint * lint * lint * lint * Update chainlink-common, use single account signer. * resolve merge conflict * resolve merge conflict 2 * resolve merge conflict 2 * Fixes chainlink version in CI --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Adam Hamrick <adam.hamrick@smartcontract.com>
* update target strategy with biling related calls * fix test * fix decimal * fix decimal * bump framework * handle overflow
* Bump common to 9c693c79b285f27ede6473188fc5933a40cd988f * Add changeset * More fixes * More test fixes * Use newer common version * Regen mocks * Fix lint issues * Fix more tests * Bump to main branch version * Fix 2 more lint issues * Undo erroneous change * Remove invalid test * Additional fix
* Update image build action * Update tool version * Update CTF versions * Remove duplicate build step * Fix load tests
) * Update to use the new SDK and the new protos generated in common * Regenerate to fix md and fix evm_chain * Passes v2 test * Fix md with generate...? * Fix lint * attempt to fix lint * Fix ref to test capabilities * Fix consensus no dag * Fix lint for real * imports again after removing all lines... * remove duplicate import
…enable Stream Multiplication (#18463) * Add Channel Definition Opts to LLO capability_trigger report for Multiplication * Fix lint errors + add changeset * lint * Make Opts and Opts.Multipliers.Multiplier required * Use errors.New instead * Make Opts non required * fix import order error * fix import order error * Remove omitempty from Opts.Multipliers * Make Capability LLO Integration test use capability_trigger channel report type * fixup comment * Make multiplier always 1e18 in Test_runLLOWorkflow test * only multiply by 1 * Add core/capabilities/integration_tests/keystone/llo_feed_test.go comment wtr to 1 using Multiplier in test * typo
* Bump chainlink-common * Scale by 1e18 in channel definition * Add changeset * Edit changeset
* Allow no opts in capability_trigger channel definition * fixup test * Make Opts optional for capability_trigger * Add changeset * Add changeset
|
I see you added a changeset file but it does not contain a tag. Please edit the text include at least one of the following tags:
|
| with: | ||
| channel-id: "#team-test-tooling-internal" | ||
| slack-message: ":x: :mild-panic-intensifies: Publish Integration Test Image failed: \n${{ format('https://github.com/{0}/actions/runs/{1}', github.repository, github.run_id) }}\nRepository: Chainlink\n${{ format('Notifying <!subteam^{0}|{0}>', secrets.GUARDIAN_SLACK_NOTIFICATION_HANDLE)}}" | ||
| build-chainlink-image: |
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.
Is this change meant to be included?
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.
This change isn't part of my primary goal. I am happy to try and remove it.
|
|
Could this PR be made to only have what is necessary for the hot fix? |




This PR adds these 3 patches:
85f86fa
31711ce
#18512
To release/2.26.0, for LLO precision scaling
It also adds this logging patch:
f89c770
As well as any other patches required to build as a result of dependency changes.
There is also this chainlink-common patch to fix a bug in the LLO Workflow Aggregator:
smartcontractkit/chainlink-common#1411
Which we have added to this PR via a custom
chainlink-commontagv0.7.1.