-
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
Correct the post-unseal meaning of the seal status type #23022
Conversation
And at the same time add a RecoverySealType to the response which preserves the old meaning. Updated the CLI to display both when relevant.
Build Results: |
CI Results: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
I'm assuming that the value multiseal
with no additional details within this api end-point is acceptable for seal-ha instances to maintain backwards compatibility.
* Correct the post-unseal meaning of the seal status type And at the same time add a RecoverySealType to the response which preserves the old meaning. Updated the CLI to display both when relevant. * changelog * no longer needed * Don't need this field either, which fixes unit tests * fix unit tests
…nto release/1.15.x (#24170) * Correct the post-unseal meaning of the seal status type (#23022) * Correct the post-unseal meaning of the seal status type And at the same time add a RecoverySealType to the response which preserves the old meaning. Updated the CLI to display both when relevant. * changelog * no longer needed * Don't need this field either, which fixes unit tests * fix unit tests --------- Co-authored-by: Scott Miller <[email protected]> Co-authored-by: Victor Rodriguez <[email protected]>
And at the same time add a RecoverySealType to the response which preserves the old meaning.
Updated the CLI to display both when relevant.