Skip to content

Commit f4255fb

Browse files
committed
config: Explicily make consoleSize unspecified if terminal is false or unset
The old language is from a502caf (config: Add consoleSize to process, 2016-09-14, #563), where nobody commented on the "if attached" wording [1]. But reading the old line now, it's not clear to me what consoleSize means when terminal is not true. This commit explicitly declares consoleSize unspecified in that condition, so runtimes are free to do what they want short of erroring out. I considered making the property undefined or requiring it to be unset, but those seemed too strict given our permissive "MUST ignore unknown properties" extensibility requirement. [1]: #563 Signed-off-by: W. Trevor King <[email protected]>
1 parent e6c9689 commit f4255fb

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

config.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,8 @@ For Solaris, the mount entry corresponds to the 'fs' resource in the [zonecfg(1M
136136

137137
* **`terminal`** (bool, OPTIONAL) specifies whether a terminal is attached to that process, defaults to false.
138138
As an example, if set to true on Linux a pseudoterminal pair is allocated for the container process and the pseudoterminal slave is duplicated on the container process's [standard streams][stdin.3].
139-
* **`consoleSize`** (object, OPTIONAL) specifies the console size in characters of the terminal if attached, containing the following properties:
139+
* **`consoleSize`** (object, OPTIONAL) specifies the console size in characters of the terminal.
140+
`consoleSize` is unspecified if `terminal` is `false` or unset.
140141
* **`height`** (uint, REQUIRED)
141142
* **`width`** (uint, REQUIRED)
142143
* **`cwd`** (string, REQUIRED) is the working directory that will be set for the executable.

0 commit comments

Comments
 (0)