-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
expiration: shutting down, not attempting further revocation of lease #5000
Comments
What version of Vault? |
@jefferai |
I believe this should be fixed in 0.10.4. |
Can you give me more context about this error? And how is it produced? Thanks. @jefferai |
I can't really as I believe those messages are symptoms, not the cause. We have cleaned up several ways that shutdown could deadlock, however. |
Fill in the details: Has this error been fixed in 0.10.4? Thanks. @jefferai |
That sounds like some other issue, possibly with your underlying storage. Can you provide full logs? |
|
Here is the detailed logs. Trouble to help me see what the problem is. Thanks. @jefferai
|
This comment has been minimized.
This comment has been minimized.
@atomd I'm hiding your comment as it's an entirely different issue not related to this one |
@jefferai Thanks. I raised a new issue. |
@jefferai can you unhide my comment? I think it is helpful to this issue. Because of this problem, vault does not work properly and the secrets are lost. It is very serious for me, please help me see how it can be solved. |
@atomd are you able to build from source? |
@jefferai yes, I can try. you can provide me some more detailed guidelines. |
See https://github.com/hashicorp/vault#developing-vault -- the branch you want to build is This was caused by race conditions that have since been fixed but there isn't currently a way to reverse those changes. We have an idea to prevent some of these problems in the first place (in case there are other bugs that we don't know about) but it's a much bigger change that we won't be able to get to in this dev cycle, so I'm coding a reasonable workaround: if two entities claim the same alias, we merge the entities. Since this situation is only caused by (so far as we know) a race condition in the first place, it is reasonable to merge them since they were always meant to be a single entity with a single alias. |
Hi @jefferai The branch Has this issue been fixed in 0.10.4? Thanks. |
No, the patch hasn't been merged so it will be fixed in the next release. |
What is the order of operations that will lead to the race conditions? @jefferai Thanks! Great help to me. |
It was caused by a bug that was fixed in 0.10.4. |
@jefferai : we're encountering this error in production. since we can't immediately upgrade vault without testing, is there a workaround we could do on v0.10.1? we're running vault HA with a consul and azure backend storage. |
I have encountered this error, I would like to ask what scenario will happen this error:
When this error occurs, the vault will not resume normal operation in any way.
Thanks!
The text was updated successfully, but these errors were encountered: