lxc / go-lxc

Go bindings for liblxc
https://linuxcontainers.org/lxc
Other
431 stars 76 forks source link

c/r: make sure container is running before doing a dump #70

Closed tych0 closed 8 years ago

tych0 commented 8 years ago

Signed-off-by: Tycho Andersen tycho.andersen@canonical.com

stgraber commented 8 years ago

So nothing wrong with doing the check here, but does that mean that liblxc itself doesn't check? and if so, shouldn't it?

tych0 commented 8 years ago

It fails, but not in the way you'd expect. I'll send a patch for that too.

caglar10ur commented 8 years ago

LGTM, please merge