Skip to content

Commit bb429f4

Browse files
committed
runtime: Remove status redefinitions from operations
These states are already defined in the "State" section. There's no need to redefine them in the operation sections. Operation-level redefinitions are dicy anyway, because they imply something testable about the immediately-after-this-operation time, and it's not possible to run race-free tests of that time (e.g. the process could die for other reasons between the successful 'create' call and the 'state' call you made to look for a 'created' status). Signed-off-by: W. Trevor King <[email protected]>
1 parent 2d491b0 commit bb429f4

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

runtime.md

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,6 @@ Using the data in [`config.json`](config.md), this operation MUST create a new c
9393
This means that all of the resources associated with the container MUST be created, however, the user-specified program MUST NOT be run at this time.
9494
If the runtime cannot create the container as specified in [`config.json`](config.md), it MUST generate an error and a new container MUST NOT be created.
9595

96-
Upon successful completion of this operation the `status` property of this container MUST be `created`.
97-
9896
The runtime MAY validate `config.json` against this spec, either generically or with respect to the local system capabilities, before creating the container ([step 2](#lifecycle)).
9997
Runtime callers who are interested in pre-create validation can run [bundle-validation tools](implementations.md#testing--tools) before invoking the create operation.
10098

@@ -108,17 +106,13 @@ Attempting to start a container that does not exist MUST generate an error.
108106
Attempting to start an already started container MUST have no effect on the container and MUST generate an error.
109107
This operation MUST run the user-specified program as specified by [`process`](config.md#process).
110108

111-
Upon successful completion of this operation the `status` property of this container MUST be `running`.
112-
113109
### Kill
114110
`kill <container-id> <signal>`
115111

116112
This operation MUST generate an error if it is not provided the container ID.
117113
Attempting to send a signal to a container that is not running MUST have no effect on the container and MUST generate an error.
118114
This operation MUST send the specified signal to the process in the container.
119115

120-
When the process in the container is stopped, irrespective of it being as a result of a `kill` operation or any other reason, the `status` property of this container MUST be `stopped`.
121-
122116
### Delete
123117
`delete <container-id>`
124118

0 commit comments

Comments
 (0)