[Discussion] Whether to remove init_image parameter in Python client module

[jasoonn]

Search before asking

• I had searched in the issues and found no similar feature requirement.

Description

The parameter init_image in the function build_worker() is originally used to set the image in init container in worker group spec. However, after #973, the init container is no longer needed for worker group spec, and the code snippet relevant to init container in Python-client are removed in #1006. Hence, the init_image became a redundant parameter.

On the other hand, because build_worker() is a user-facing, removing init_image will break backward compatibility.
Hence, I am wondering whether it is a good idea to remove init_image from build_worker().

Use case

No response

Related issues

No response

Are you willing to submit a PR?

• Yes I am willing to submit a PR!

[kevin85421]

We learned several lessons in these two months, which underscore the importance of keeping the init image configurable.