specs-go/config: Add omitempty to LinuxSyscall.Args

wking · wking · commit bb4cca78cdb0 · 2017-04-06T23:13:22.000-07:00
It used to have this, but the omitempty was dropped in 652323c (improve seccomp format to be more expressive, 2017-01-13, opencontainers#657). However, the docs that landed in 3ca5c6c (config-linux.md: fix seccomp, 2017-03-02, opencontainers#706) list the property as optional, and if it is optional, we can leave it unset instead of serializing an empty array. Signed-off-by: W. Trevor King <wking@tremily.us>
diff --git a/specs-go/config.go b/specs-go/config.go
@@ -551,7 +551,7 @@ type LinuxSeccompArg struct {
 type LinuxSyscall struct {
 	Names  []string           `json:"names"`
 	Action LinuxSeccompAction `json:"action"`
-	Args   []LinuxSeccompArg  `json:"args"`
+	Args   []LinuxSeccompArg  `json:"args,omitempty"`
 }
 
 // LinuxIntelRdt has container runtime resource constraints

Original file line number	Diff line number	Diff line change
`@@ -551,7 +551,7 @@ type LinuxSeccompArg struct {`
`551`	`551`	`type LinuxSyscall struct {`
`552`	`552`	Names []string `json:"names"`
`553`	`553`	Action LinuxSeccompAction `json:"action"`
`554`		- Args []LinuxSeccompArg `json:"args"`
	`554`	+ Args []LinuxSeccompArg `json:"args,omitempty"`
`555`	`555`	`}`
`556`	`556`
`557`	`557`	`// LinuxIntelRdt has container runtime resource constraints`