Added option to allow the user the option of not using crate2nix but the default Docker build process instead.#560
Added option to allow the user the option of not using crate2nix but the default Docker build process instead.#560soenkeliebau wants to merge 3 commits intomainfrom
Conversation
…the default Docker build process instead. Not sure if this is feasible to use with Tilt due to fairly long build times, might be necessary to pause deployment of the resources in Tilt and manually triggering as and when needed.
|
full disclosure, not tested it yet, just rescued from https://github.com/stackabletech/operator-templating/tree/feat/multiple_builders |
| outputs_image_ref_to='result/ref', | ||
| ) | ||
| else if builder == 'docker': | ||
| docker_build(registry + '/' + operator_name, '.', dockerfile='docker/Dockerfile') |
There was a problem hiding this comment.
Does this also need deps to be set, like the custom_build?
There was a problem hiding this comment.
Doesn't look like docker_build supports that: https://docs.tilt.dev/api.html#api.docker_build
Not sure how it would know when to rebuild though ...
There was a problem hiding this comment.
The '.' in the docker_build call is the context, that is watched for changes, a rebuild is triggered when anything changes in the current working directory.
We can probably do better than that, but auto-rebuild should happen with this..
docker_build supports live update functionality, which I think would allow us to copy the compiled binary into the running container .. not sure how the restart would then happen though ...
https://github.com/tilt-dev/live_update/blob/master/go/Tiltfile
NickLarsenNZ
left a comment
NickLarsenNZ
left a comment
There was a problem hiding this comment.
I suggest we change the builder name to just nix
2 participants
Not sure if this is feasible to use with Tilt due to fairly long build times, might be necessary to pause deployment of the resources in Tilt and manually triggering as and when needed.