-
Notifications
You must be signed in to change notification settings - Fork 4
Misiug/jinja docker compose #153
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
| @@ -0,0 +1,50 @@ | |||
| x-presto-native-worker-gpu: &gpu_worker_base | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
services starting with x- are considered "Extensions" and are ignored when bringing up services. This one is uses exclusively as an anchor.
|
Do a clean build. Please delete all dockers images and perform multi-node images build and run once. Fix accordingly based on failures. IIUC, |
We want to be able to dynamically generate docker services for multiple workers that use separately assigned gpus.
This PR does so by replacing the docker-compose.native-gpu.yml file with a jinja template that gets expanded via python script when
start_native_gpu_presto.shis called based on the --num-workers (-w) parameter is called. If the option left unspecified, it continues to generate apresto-native-worker-gpuwith access to all gpus as normal.