AmitKumarDas / metac

It is metacontroller and more
Apache License 2.0
57 stars 16 forks source link

Use a distroless image #106

Closed floriankoch closed 4 years ago

floriankoch commented 4 years ago

What do you think of using https://github.com/GoogleContainerTools/distroless as a base image instead of debian slim?

grzesuav commented 4 years ago

Need to be very careful, as it makes debugging (exec into running container) impossible

floriankoch commented 4 years ago

@grzesuav yes and no - you can use the :debug variant, with a busybox included

AmitKumarDas commented 4 years ago

I will change it to distroless image or would love to accept any PRs on the same. In fact I have been using distroless image for my other projects without issues.

Refer - https://github.com/mayadata-io/cstorpoolauto/blob/master/Dockerfile

floriankoch commented 4 years ago

@AmitKumarDas i will send a PR

grzesuav commented 4 years ago

I would say both variant should be present. Also @floriankoch if you wish to have :debug version of metacontroller it should also be distributed (basing on :debug go distroless image)

AmitKumarDas commented 4 years ago

Trying to get my head around debug stuff. I hope log levels do help here. I/we are taking extra steps to unit test everything before pushing a PR. We need to think about exposing operational metrics soon.

However do we feel there might be circumstances where we need to attach a debugger to the running container and hence the need for a debug binary? Or is it something that I am not getting.

grzesuav commented 4 years ago

@AmitKumarDas it is not only about attaching debuger, with distroless there is no shell/package manager so basically you cannot do anything with that process.

I know that in k8s there are plans for ephemeral containers but :

AmitKumarDas commented 4 years ago

Having two images makes sense then.

AmitKumarDas commented 4 years ago

:tada: This issue has been resolved in version 0.2.3 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: