lxd can only be ^C'd until a container has started about lxd

[stgraber]

ctrl+c shouldn't kill the containers. There needs to be a way (probably a signal) to tell to lxd to do a clean shutdown and that'll be used by the init script on host shutdown, but for the common case, lxd exitting during normal operation must keep the containers running so that on crash/failure, it gets respawned and can keep things running.

Remember that the main usage scenario for lxd is a compute node, on those, you don't want your containers to die unless they absolutely have to. For someone experimenting or developing lxd, they will have to stop their containers themselves or if using a distro package, removing the package will also cause a clean shutdown.

So as a conclusion, ctrl+c must be allowed and must have lxd exit immediately without doing anything to the running containers. When restarted, lxd should just pick things were they were left off.

[tych0]

For the record: I disagree that ^C shouldn't kill the containers.

[kapilt]

fwiw, typically behavior in app container tools is to associate a restart
policy to the container which determines behavior also for exit of app
container and daemon restarts.

-k

On Wed, Dec 10, 2014 at 10:46 AM, Tycho Andersen [email protected]
wrote:

For the record: I disagree that ^C shouldn't kill the containers.

—
Reply to this email directly or view it on GitHub
lxc/incus#122 (comment).

[tych0]

That policy is per-container or per-daemon? We'd talked about having a daemon-wide option, at least.

[hallyn]

Hi, just to be clear, are you running with ppa:ubuntu-lxc/daily? Because the ctrl-c bug should have been fixed not too long ago, when I originally found it while working on the flex attach method.

[tych0]

On Tue, Jan 06, 2015 at 08:12:02AM -0800, hallyn wrote:

Hi, just to be clear, are you running with ppa:ubuntu-lxc/daily? Because the ctrl-c bug should have been fixed not too long ago, when I originally found it while working on the flex attach method.

I've seen this with the trunk at least as of last month when I paused
lxd development. ^C works in some cases, but not others (but whether
or not it worked didn't have anything to do with attach, I didn't
think). Unfortunately I just fiddled around with it and I can't seem
to reproduce it. We can close this for now and I can re-open when I
see the behavior again with steps to reproduce.

---

Reply to this email directly or view it on GitHub:
lxc/incus#122 (comment)