-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add new constraint for retiring-to-retired transition #143
Conversation
bd3fd56
to
d3accd1
Compare
docs/api.md
Outdated
@@ -299,6 +300,10 @@ The new state is given by contents of request body and should be one of: | |||
|
|||
HTTP status code: 400 Bad Request | |||
|
|||
- Disk encryption keys have not been deleted. |
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.
ちょっと伝わりにくいです。例えば
- Disk encryption keys have not been deleted. | |
- Transitioning a retiring server that still have disk encryption keys to be retired. |
docs/lifecycle.md
Outdated
* Disk encryption keys of a machine can be deleted if the machine is in **Retiring** state. | ||
* A machine transitions to **Retired** when its disk encryption keys are deleted. | ||
* A **Retiring** machine can transition to **Retired** only when its disk encryption keys have been deleted. |
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.
* A **Retiring** machine can transition to **Retired** only when its disk encryption keys have been deleted. | |
* A **Retiring** machine can transition to **Retired** only when it has no disk encryption keys. |
docs/sabactl.md
Outdated
Deletes all keys of a machine, and make its status `retired`. | ||
The command fails when the target machine's status is not `retiring`. | ||
Deletes all keys of a machine. | ||
The command fails when the target machine's status is not `retiring`, or when the target machine's disk encryption keys have not been deleted. |
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.
追加部分の意味がわかりません。
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.
LGTM
No description provided.