Conversation
@@ -243,6 +243,7 @@ type Info struct { | |||
ServerVersion string | |||
ClusterStore string | |||
ClusterAdvertise string | |||
SecurityOptions string |
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.
should we make this a []string
? I think that'd be better for the API (and similar to, e.g. how we have a list of installed plugins;
Lines 248 to 257 in 0ec73ff
// PluginsInfo is a temp struct holding Plugins name | |
// registered with docker daemon. It is used by Info struct | |
type PluginsInfo struct { | |
// List of Volume plugins registered | |
Volume []string | |
// List of Network plugins registered | |
Network []string | |
// List of Authorization plugins registered | |
Authorization []string | |
} |
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.
@thaJeztah Yes []string
is definitely better than a space separated string and easy to process. Let me update the pull request. Thanks!
This PR tries to add SecurityOptions field in Info so that security options such as apparmor, seccomp, or selinux could be displayed by `docker info`. Detailed discussion could be found in: moby/moby#20909 moby/moby#21172 Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@thaJeztah Just updated the pull request to change to |
LGTM |
1 similar comment
LGTM |
Please, do not merge this PR yet. We're waiting for a release cut in docker/docker before moving this forward. |
We already have a tag a frozen branches for both projects. Merging this in 🎉 |
This PR tries to add SecurityOptions field in Info so that security options such as apparmor, seccomp, or selinux could be displayed by
docker info
.Detailed discussion could be found in:
moby/moby#20909
moby/moby#21172
cc @justincormack @calavera @thaJeztah
Signed-off-by: Yong Tang yong.tang.github@outlook.com