dzueck commented on PR #8572: URL: https://github.com/apache/texera/pull/8572#issuecomment-5798969673
> Thanks for this, I think it's a great and relatively lightweight addition. I would like to see the ability to separately specify the runtime class for the CU. There is a clear difference in security domain between what runs on the CU and what runs in other services. The CU itself contains untrusted user code, whereas the other services may not. You still might want to use a different runtime for these services, but it's not the same potential threat. I think making the flag use one runtime class for everything is unnecessarily conflating the two things. This is already how it works. If you just change the environment variables "KUBERNETES_COMPUTING_UNIT_RUNTIME_CLASS_CPU" and "KUBERNETES_COMPUTING_UNIT_RUNTIME_CLASS_GPU" then it will create CUs with those runtimes. This does not change any micro services. Updating texeraMicroServiceRuntimeClass in global is what configures all texera specific micro services (but not CUs). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
