Skip to content

Fix DKG key import error to use 'ErrKeyExists' error - #19862

Merged
Tofel merged 2 commits into
developfrom
dkg-key-fix
Oct 14, 2025
Merged

Tofel merged 2 commits into
developfrom
dkg-key-fix

Conversation

@vreff

@vreff vreff commented Oct 13, 2025 •

Copy link
Copy Markdown
Contributor

Fixes a bug in shell_cmd: https://github.com/smartcontractkit/chainlink/blob/develop/core/cmd/shell_local.go#L534C32-L534C44, where the error is not the expected type.

@vreff
vreff requested a review from a team as a code owner October 13, 2025 12:42
@github-actions

Copy link
Copy Markdown
Contributor

👋 vreff, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@Tofel
Tofel self-requested a review October 13, 2025 12:48
Tofel
Tofel previously approved these changes Oct 13, 2025
@vreff
vreff requested a review from a team as a code owner October 13, 2025 12:52
@github-actions

Copy link
Copy Markdown
Contributor

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:

  • #added For any new functionality added.
  • #breaking_change For any functionality that requires manual action for the node to boot.
  • #bugfix For bug fixes.
  • #changed For any change to the existing functionality.
  • #db_update For any feature that introduces updates to database schema.
  • #deprecation_notice For any upcoming deprecation functionality.
  • #internal For changesets that need to be excluded from the final changelog.
  • #nops For any feature that is NOP facing and needs to be in the official Release Notes for the release.
  • #removed For any functionality/config that is removed.
  • #updated For any functionality that is updated.
  • #wip For any change that is not ready yet and external communication about it should be held off till it is feature complete.

@vreff
vreff requested a review from Tofel October 13, 2025 13:00
@cl-sonarqube-production

Copy link
Copy Markdown

@Tofel
Tofel enabled auto-merge October 14, 2025 08:43
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