Skip to content

Conversation

hashiblaum
Copy link
Member

Description

What does this PR do?

  • evaluates the irrevocableLeaseRemoval feature at Vault startup
  • removes the deleteLeaseCommon function
  • if irrevocableLeaseRemoval is enabled, a force revoke is performed instead of a deferred delete

@hashiblaum hashiblaum requested a review from digivava May 21, 2025 18:38
@hashiblaum hashiblaum requested a review from a team as a code owner May 21, 2025 18:38
@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label May 21, 2025
Copy link

github-actions bot commented May 21, 2025

CI Results:
All Go tests succeeded! ✅

Copy link

github-actions bot commented May 21, 2025

Build Results:
All builds succeeded! ✅

@hashiblaum hashiblaum added this to the 1.20.0-rc milestone May 21, 2025
@hashiblaum hashiblaum force-pushed the VAULT-36164/irr-lease-removal-2 branch from 973abe2 to 7626113 Compare May 21, 2025 19:07
@hashiblaum hashiblaum force-pushed the VAULT-36164/irr-lease-removal-2 branch from 7626113 to 321c361 Compare May 21, 2025 19:55
@hashiblaum hashiblaum requested a review from digivava May 22, 2025 16:21
@hashiblaum hashiblaum merged commit 5688d24 into main May 22, 2025
92 checks passed
@hashiblaum hashiblaum deleted the VAULT-36164/irr-lease-removal-2 branch May 22, 2025 20:19
miagilepner pushed a commit that referenced this pull request May 23, 2025
* refactor changes for irrevocable lease removal

* changelog addition

* add logs

* update changelog; always sleep during irrevocable lease revoke party

* comment

* typo

* pass Core pntr to removeIrrevocableLeasesEnabled

* remove log
erentantekin pushed a commit that referenced this pull request May 23, 2025
* refactor changes for irrevocable lease removal

* changelog addition

* add logs

* update changelog; always sleep during irrevocable lease revoke party

* comment

* typo

* pass Core pntr to removeIrrevocableLeasesEnabled

* remove log
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.

2 participants