Skip to content

container-spec: only use required CAP_*#38

Open
parthpower wants to merge 1 commit intomainfrom
dev-reduced-security-ctx
Open

container-spec: only use required CAP_*#38
parthpower wants to merge 1 commit intomainfrom
dev-reduced-security-ctx

Conversation

@parthpower
Copy link

securityContext.privileged = true gives unnecessary permissions to containers.

Only use required capabilities with securityContext.Capabilities option

use different security contexts for them otg-gnmi-server and keng-controller since they don't need as many capabilities as TE and PE

securityContext.privileged = true gives unnecessary permissions to containers.

Only use required capabilities with securityContext.Capabilities option

use different security contexts for them otg-gnmi-server and keng-controller
since they don't need as many capabilities as TE and PE
@anjan-keysight
Copy link
Collaborator

Without explicitly assigning some security cap to controller & gnmi, the node mapped port is not accepting requests. It will need some capability to be assigned. I will investigate further, update this PR and then merge.

@parthpower
Copy link
Author

Without explicitly assigning some security cap to controller & gnmi, the node mapped port is not accepting requests. It will need some capability to be assigned. I will investigate further, update this PR and then merge.

would need CAP_NET_BIND_SERVICE to bind to port number < 1024

@anjan-keysight
Copy link
Collaborator

The node ports are > 30000. I think we may have to add some explicit, minimal capability. I will investigate later this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants