Skip to content

Commit 05415d4

Browse files
committed
specs-go/config: Make User.UID and User.GID pointers
Avoiding: $ go test ./specs-go --- FAIL: TestConfigRoundTrip (0.00s) --- FAIL: TestConfigRoundTrip/config_2 (0.00s) round_trip_test.go:35: failed to round-trip: {"ociVersion":"1.0.0","platform":{"os":"windows","arch":"amd64"},"process":{"user":{"uid":0,"gid":0,"username":"containeradministrator"},"args":["sh"],"cwd":"C:\\"},"root":{"path":"rootfs"}} FAIL FAIL github.com/opencontainers/runtime-spec/specs-go 0.003s We can't just add omitempty (without also making them pointers), because 0 is a meaningful value for both properties. Signed-off-by: W. Trevor King <[email protected]>
1 parent bdae383 commit 05415d4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

specs-go/config.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,9 @@ type Box struct {
8484
// User specifies specific user (and group) information for the container process.
8585
type User struct {
8686
// UID is the user id.
87-
UID uint32 `json:"uid" platform:"linux,solaris"`
87+
UID *uint32 `json:"uid,omitempty" platform:"linux,solaris"`
8888
// GID is the group id.
89-
GID uint32 `json:"gid" platform:"linux,solaris"`
89+
GID *uint32 `json:"gid,omitempty" platform:"linux,solaris"`
9090
// AdditionalGids are additional group ids set for the container's process.
9191
AdditionalGids []uint32 `json:"additionalGids,omitempty" platform:"linux,solaris"`
9292
// Username is the user name.

0 commit comments

Comments
 (0)