Enabling PodSecurityContext to be configured for JM and TM pod templates

lyft / flinkk8soperator

Kubernetes operator that provides control plane for managing Apache Flink applications

Apache License 2.0

569 stars 159 forks source link

Closed kelly-sm closed 4 years ago

kelly-sm commented 4 years ago

Updating the CRD to enable pod security context to be specified and applied to JM and TM pods.
Adding security context fields to crd validation.
Changing security context injection to maintain hash-based backwards compatibility.
Adding tests to ensure that security context is being injected properly.

anandswaminathan commented 4 years ago

+1 LGTM

kelly-sm commented 4 years ago

Is there a way to pull an image of this once it is merged in? Does each commit get built and pushed somewhere?

anandswaminathan commented 4 years ago

@kelly-sm Once you merge. We have Travis that will automatically build and push the image.

You can then pull the image from docker.io/lyft/flinkk8soperator:{sha}

kelly-sm commented 4 years ago

@anandswaminathan sounds good. Sounds like we just need another person to review/approve?

Thanks!