Skip to content

Conversation

@vreff
Copy link
Contributor

@vreff vreff commented Sep 1, 2025

This PR:

  • Uses the P256 DKG Recipient key & DKG result inside the vault reporting plugin to populate private key share and master public key.
  • Bumps smdkg, chainlink-common, and libocr
  • Adds an EnableDKGRecipient configuration to the chainlink node to optionally enable the use of the DKG recipient key.
  • Uses the EnableDKGRecipient configuration & the DKGRecipientKey ensured by the node, such that the E2E CRE suite is able to start the vault capability job that now requires the DKGRecipientKey.

Related:
#19162
#19161
smartcontractkit/chainlink-common#1508

@cl-sonarqube-production
Copy link

@vreff vreff marked this pull request as ready for review September 6, 2025 06:13
@vreff vreff requested review from a team as code owners September 6, 2025 06:13
@cedric-cordenier cedric-cordenier added this pull request to the merge queue Sep 8, 2025
Merged via the queue into develop with commit 71bcea4 Sep 8, 2025
218 of 221 checks passed
@cedric-cordenier cedric-cordenier deleted the PRIV-151-use-dkg-package-p256key branch September 8, 2025 09:47
cedric-cordenier pushed a commit that referenced this pull request Oct 16, 2025
* Add dkgresult database

* add p256 key to keystore

* tests

* Use Use DKG package in vault reporting plugin

* Add dkgresult database

* make gomodtidy

* fix merge conflicts

* make generate

* fix lint / possible overflow error

* fix lint

* update test package

* Update to use new keyring

* make gomodtidy

* make gomodtidy

* make generate

* remove local replace, remove redundant orm

* Only ensure DKG recipient key if it is configured

* remove ensurekey

* add more tests

* fix lint

* changeset

* gomodtidy

* bump smdkg, libocr

* Use smdkg updates in plugin & test

* bump chainlink-common, make generate

* Add more config updates

* fix test expected error log.

* changeset

* fix lint

* fix config docs

* fix tests

* fix chainlink-common import

* Enable dkg recipient key for CRE E2E tests

* fix panic

* fix lint, add more tests

* ensure key for test env

* temp: always ensure key

* fix keystore test panic

* fix config default behavior

* make gomodtidy

* bump cdf

* add override to other configs

* update go mod
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.

3 participants