2.15 merge #1401
Merged
2.15 merge #1401
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
* 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 <[email protected]> * Bump libocr to fix nil msg error (#13919) * Finalize date on changelog for 2.14.0 (#13943) --------- Signed-off-by: Sneha Agnihotri <[email protected]> Co-authored-by: chainchad <[email protected]>
Signed-off-by: chainchad <[email protected]>
(cherry picked from commit 7147653)
(cherry picked from commit 7147653)
…changelog-fix Use changeset github changelog generator
Signed-off-by: chainchad <[email protected]>
mateusz-sekara
had a problem deploying
to
sdlc
GitHub Actions
Failure
mateusz-sekara
force-pushed
the
2.15-merge
branch
from
af42df8 to
b134b80
Compare
mateusz-sekara
force-pushed
the
2.15-merge
branch
from
b134b80 to
061d0c4
Compare
mateusz-sekara
force-pushed
the
2.15-merge
branch
from
061d0c4 to
10ecf42
Compare
makramkd
reviewed
Sep 2, 2024
asoliman92
reviewed
Sep 3, 2024
| } | ||
|
|
||
| srvcs = append(srvcs, externalPeerWrapper) | ||
| var capabilityRegistrySyncer registrysyncer.Syncer |
There was a problem hiding this comment.
Code here is a bit outdated. even if 2.15 is 2 weeks ago I think it was different as most of the part related to the capabilities was last edited first week of August. Can you double check nothing was auto resolved wrongly here?
There was a problem hiding this comment.
2.15 was cut on 30th July from develop, which means that code is at least a month behind the current develop
asoliman92
reviewed
Sep 3, 2024
| @@ -258,14 +258,10 @@ packages: | |||
| ORM: | |||
| Runner: | |||
| PipelineParamUnmarshaler: | |||
| github.com/smartcontractkit/chainlink/v2/core/services/headreporter: | |||
| github.com/smartcontractkit/chainlink/v2/core/services/promreporter: | |||
There was a problem hiding this comment.
I'm not familiar with prom or headreporter. What's the reason for removing headreporter completely like this?
There was a problem hiding this comment.
That's the state of the repo on v2.15.0. It seems that promreporter replaced headreporter
Merged
asoliman92
approved these changes
Sep 3, 2024
|
Protobuf checks fails because of this breaking change smartcontractkit/chainlink#13750 |
asoliman92
approved these changes
Sep 3, 2024
makramkd
requested changes
Sep 3, 2024
makramkd
approved these changes
Sep 3, 2024
|
RensR
approved these changes
Sep 3, 2024
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.
Merging chainlink repository from
v2.15.0