[FEATURE] - Executor Resource Requests & Limits

[gambol99]

Currently consumers are not permitted to set the executor resource requests/limits - this has created an error in the past, due to higher than expected requirements, hence it was bumped to 1Gi. For certain modules though this may not be enough, hence the feature to support controling the fields from the controller arguments

[jest@razor terranetes-controller]$ bin/controller --help | egrep '(request|limit)' -i
      --executor-cpu-limit string            The default CPU limit for the executor container (default "1")
      --executor-cpu-request string          The default CPU request for the executor container (default "5m")
      --executor-memory-limit string         The default memory limit for the executor container (default "1Gi")
      --executor-memory-request string       The default memory request for the executor container (default "32Mi")

Note: the user can technically overload the template using the method described here https://terranetes.appvia.io/terranetes-controller/admin/template/ … but that is largely for last ditch effort, as it comes was compatibility issues going forward.

[KashifSaadat]

(nitpick, not important): Perhaps leave off the CPU default limit (make it conditional, so if set to a custom value only then will it be specified in the template).

[gambol99]

Do we prefer fec8d22 .. and place the decision solely on the administrator? cc @KashifSaadat

[gambol99]

sorry rebased - 9ddbaaf

[KashifSaadat]

LGTM