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.13.x #22378

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.13.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/lively-concise-goose branch from e8a7759 to ec60705 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/lively-concise-goose branch from cf651d5 to 29dbce0 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.13.6 milestone Aug 16, 2023
@ncabatoff ncabatoff enabled auto-merge (squash) August 16, 2023 20:06
@github-actions
Copy link

Build Results:
All builds succeeded! ✅

@ncabatoff ncabatoff merged commit bcd2db3 into release/1.13.x Aug 16, 2023
85 of 87 checks passed
@ncabatoff ncabatoff deleted the backport/fix-lease-restore-deadlock/lively-concise-goose branch August 16, 2023 20:20
@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