Skip to content
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

Backport of Prevent a deadlock in expiration into release/1.12.x #22377

Conversation

hc-github-team-secure-vault-core
Copy link
Collaborator

Backport

This PR is auto-generated from #22374 to be assessed for backporting due to the inclusion of the label backport/1.12.x.

The below text is copied from the body of the original PR.


Deadlock occurs when Restore grabs lease lock then pending lock, while a revocationJob's OnFailure grabs those lock in the reverse order.


Overview of commits

@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/fix-lease-restore-deadlock/urgently-allowing-kiwi branch from 9cc2dfc to 4a57de2 Compare August 16, 2023 20:04
@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/fix-lease-restore-deadlock/urgently-allowing-kiwi branch from 81f2350 to 9fc7f16 Compare August 16, 2023 20:04
@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Aug 16, 2023
@ncabatoff ncabatoff added this to the 1.12.10 milestone Aug 16, 2023
@ncabatoff ncabatoff enabled auto-merge (squash) August 16, 2023 20:05
@github-actions
Copy link

Build Results:
All builds succeeded! ✅

@ncabatoff ncabatoff merged commit f4d0bbd into release/1.12.x Aug 16, 2023
85 of 87 checks passed
@ncabatoff ncabatoff deleted the backport/fix-lease-restore-deadlock/urgently-allowing-kiwi branch August 16, 2023 20:18
@github-actions
Copy link

CI Results:
All Go tests succeeded! ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants