Matt Landis

salat 1.11.1 has been published for 2.12: http://search.maven.org/#search%7Cga%7C1%7Csalat

For Chesterton's Fence purposes, Milas confirmed that this was simply an unconsidered use case.

Thanks for filing! Yeah, that doesn't behave very well at the moment. We need to change the git_resource extension to disable tty. You should be able to work around this...

Thanks for reporting! I've taken the liberty of renaming the bug. You're correct that the lack of root is an issue, but the extension also relies on the existence of...

It turns out getting rid of the extension's dependencies won't help much for the moment. I removed them all in a branch only to learn live_update doesn't work on scratch...

I've reproduced this [in servantes](https://github.com/tilt-dev/servantes/blob/matt/391/Tiltfile), with just `k8s_custom_deploy` and `docker_build_with_restart`. Nick's hunch seems pretty accurate: * `docker_build_with_restart` works by setting `ImageMap.Spec.OverrideCommand`. * info only flows from `ImageMap` to `k8s_custom_deploy` via...

I got live update to work in this situation by overriding the image's entrypoint with `docker_build_sub` like this: ``` load('ext://restart_process', 'docker_build_with_restart') load('ext://docker_build_sub', 'docker_build_sub') k8s_custom_deploy( 'fortune-depl', apply_cmd='m4 -Dvarowner="$(whoami)" deploy/fortune.yaml | sed...

> fwiw - i'm not too worried about problem (2) at this point (the original reporter confirmed this wasn't a problem for them) I wasn't super worried about it, either,...

> slight modification on (d) - rather than applying the yaml, maybe add a warning if the KubernetesDeploy reconciler sees a Pod overriding the command on a k8s_custom_deploy? yeah, that...

I took a stab at using the builtin, but it turns out the Tiltfile builtins for Tilt API objects do not support metav1.Time, so the `start_after` field is just missing....