[CRIU] [PATCH p.haul] p.haul: increase dump and predump timeouts for Virtuozzo containers
Nikita Spiridonov
nspiridonov at virtuozzo.com
Tue Jun 21 02:41:36 PDT 2016
On Mon, 2016-06-20 at 20:31 -0700, Andrew Vagin wrote:
> On Wed, Jun 15, 2016 at 11:04:48AM +0400, Nikita Spiridonov wrote:
> > By default timeout to freeze all processes in container is 10
> > seconds. For containers with large number of processes freeze can
> > take more time, so increase timeout up to 180 seconds.
>
> Do we really want to migrate a container with so big downtime?
>
Yep as for me; at the present moment this fix needed for our internal
stress tests (which fails due to freeze timeout). Pick 180 seconds on
the advice of Cyrill.
>
> >
> > Signed-off-by: Nikita Spiridonov <nspiridonov at virtuozzo.com>
> > ---
> > phaul/p_haul_vz.py | 7 ++++++-
> > 1 files changed, 6 insertions(+), 1 deletions(-)
> >
> > diff --git a/phaul/p_haul_vz.py b/phaul/p_haul_vz.py
> > index 97c2bec..a0ba306 100644
> > --- a/phaul/p_haul_vz.py
> > +++ b/phaul/p_haul_vz.py
> > @@ -140,11 +140,16 @@ class p_haul_type:
> > # Increase ghost-limit up to 50Mb
> > req.opts.ghost_limit = 50 << 20
> >
> > - # Specify freezer cgroup for both predump and dump requests
> > + # Specify both predump and dump specific options
> > if req.type == pycriu.rpc.PRE_DUMP or req.type == pycriu.rpc.DUMP:
> > +
> > + # Specify freezer cgroup
> > req.opts.freeze_cgroup = \
> > "/sys/fs/cgroup/freezer/{0}/".format(self._ctid)
> >
> > + # Increase timeout up to 180 seconds
> > + req.opts.timeout = 180
> > +
> > def root_task_pid(self):
> > path = "/var/run/ve/{0}.init.pid".format(self._ctid)
> > with open(path) as pidfile:
> > --
> > 1.7.1
> >
> > _______________________________________________
> > CRIU mailing list
> > CRIU at openvz.org
> > https://lists.openvz.org/mailman/listinfo/criu
More information about the CRIU
mailing list