Skip to content
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

Changes to Tiltfile Config do not stop previously started services #3499

Closed
andymartin-sch opened this issue Jun 23, 2020 · 4 comments
Closed

Comments

@andymartin-sch
Copy link
Contributor

I'm not sure if this should be considered a Tilt bug or not, but we use set_enabled_resources from https://docs.tilt.dev/tiltfile_config.html to selectively enable services from our helm charts. The problem is that when we try to stop services that were previously enabled, Tilt doesn't actually stop those services (pods stay Running, etc).

This sort of intuitively makes sense, since changes to the Tiltfile Config trigger a Tiltfile re-execution and on that secondary execution Tilt has no way of really knowing that it should stop previously-launched pods/services, but it seems to go against the "user story" of https://docs.tilt.dev/tiltfile_config.html where people can select arbitrary sets of services to run without stopping everything and restarting.

In Slack, someone else mentioned:

I've had similar issues with tilt_options.json . My guess is this really feeds back into the need for lifecycle hooks, but I think removing a service should (by default) destroy related resources. There is something to be said about unexpected side effects of doing such a thing, though

Potentially related: #1699

This is not a critical/major issue, but it does come up as a developer QoL thing and it makes us more likely to come up with our own custom workflow down the road if more developers find this annoying.

@nicks
Copy link
Member

nicks commented Jun 23, 2020

also related to #3137. I've always wanted a garbage collector so that when you remove resources from a tiltfile during a run, Tilt deletes any resources it deployed during that session.

@nicks
Copy link
Member

nicks commented Jun 29, 2021

I'm pretty sure this will get fixed in #4700 in the next release

@andymartin-sch
Copy link
Contributor Author

Yep thanks! This seems to be fixed in 0.21.1 - nice :)

@nicks
Copy link
Member

nicks commented Jul 7, 2021

🎉 🎉 🎉

@nicks nicks closed this as completed Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants