Skip to content

Survive a cluster peer that has not learned our public key yet - #70566

Merged
dwoz merged 2 commits into
saltstack:3008.xfrom
nsychev:fix/cluster-aes-keyerr
Oct 30, 2026
Merged

dwoz merged 2 commits into
saltstack:3008.xfrom
nsychev:fix/cluster-aes-keyerr

Conversation

@nsychev

@nsychev nsychev commented Oct 26, 2026

Copy link
Copy Markdown

What does this PR do?

Fixes KeyError on events from peers that have not announced their public key to us yet.

What issues does this PR fix or reference?

Fixes #70565

Previous Behavior

  • Exception raised on an event from a peer until we received its public key.

New Behavior

  • No exception raised, info log event is emitted.

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

@nsychev
nsychev requested a review from a team as a code owner October 26, 2026 16:50
@dwoz dwoz added the test:full Run the full test suite label Oct 26, 2026
@dwoz
dwoz changed the base branch from 3007.x to 3008.x October 27, 2026 07:21
@dwoz
dwoz force-pushed the fix/cluster-aes-keyerr branch from 9d428e0 to 5141637 Compare October 27, 2026 07:25
twangboy
twangboy previously approved these changes Oct 29, 2026
@twangboy twangboy added this to the Argon v3008.7 milestone Oct 29, 2026
@twangboy twangboy linked an issue Oct 29, 2026 that may be closed by this pull request
@dwoz
dwoz merged commit a80bdcf into saltstack:3008.x Oct 30, 2026
886 of 892 checks passed

This branch was previously deployed

1 inactive deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Cluster master gets KeyError: 'aes' during the key exchange

3 participants