Author | Commit | Message | Commit date | Issues | |
---|---|---|---|---|---|
Pavel Emelyanov | 02199d87588M | Merge pull request #12 from pvokhmyanin/masterp.haul: Add net_cls,net_prio cgroup controller to criu request | |||
Pavel Vokhmyanin | e91120c59e7 | p.haul: Fix cgroup controller list generationCgroup controller names do not match external mount points, thus we should use separate list to fill the controllers list. Also, ve and beancounter controllers were missing, they're also added by this patch. Signed-off-by: Pavel Vokhmyanin <pvokhmyanin@virtuozzo.com> | |||
Pavel Emelyanov | 526bc5f7494M | Merge pull request #10 from pvokhmyanin/masterp.haul: Fill in cgroup controller list for criu request | |||
Pavel Vokhmyanin | fc8375c22ff | p.haul: Fill in cgroup controller list for criu requestCRIU does not have hardcoded controller list since 2.10.0.17, thus we need to generate it and pass into CRIU RPC server via request opts. | |||
Pavel Emelyanov | 29f30036597M | Merge pull request #9 from pvokhmyanin/masterp.haul: Add pids cgroup external mapping | |||
Pavel Vokhmyanin | a8b190ab5ae | p.haul: Add pids cgroup external mapping | |||
Pavel Emelyanov | 2c81dee6e2dM | Merge pull request #8 from pvokhmyanin/masterp.haul: Use tcp_skip_in_flight CRIU option unconditionally | |||
Pavel Vokhmyanin | cc55e6aaa13 | p.haul: Use tcp_skip_in_flight CRIU option unconditionallyUntil CRIU learns to support sockets from a listen socket queue it's better to drop half-opened sockets - this approach does not pose any noticeable drawback to the users and allows migration to succeed. Signed-off-by: Pavel Vokhmyanin <pvokhmyanin@virtuozzo.com> | |||
Veronika Kabatova | bb44a2e7e19 | runc: add run_action_script functionSigned-off-by: Veronika Kabatova <vkabatov@redhat.com> Acked-by: Adrian Reber <areber@redhat.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Veronika Kabatova | e4c3a30d637 | runc: Migration fail implementation, mkdir permission additionThe migration should fail if container with same ID is running on target host. If same-ID container is created after the migration start, cleanup of bundle directory is performed. /var/run/runc/CONTAINER_ID is also created with the same permissions as it is in runc, not the defaults. Signed-off-by: Veronika Kabatova <vkabatov@redhat.com> Acked-by: Adrian Reber <areber@redhat.com> Signed-off-b... | |||
Veronika Kabatova | 3299a8cb483 | runc: Add support for bind and cgroup mountsAllow migration of containers with binded / cgroup mounts. Using the same system as runc uses for C/R, bind and cgroup mounts are passed to criu via the `--ext-mount-map` options (`req.opts.ext_mnt` for Python criu API). Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Veronika Kabatova | dae1264b002 | Add support for migration of basic runc containerThis patch allows migration of a runc container without --tcp-established option or mounts (a patch with support for mount follows). Example usage includes running `p.haul-wrap service` on target host and `p.haul-wrap client <target-ip> runc <container-id>` on source host. Bundle directory should exist (with the same path) on target host and no container with the same ID should exist there (a p... | |||
Pavel Emelyanov | 9e01ad67e13M | Merge pull request #7 from pvokhmyanin/masterp.haul: Introduce action scripts executed on different stages | |||
Pavel Vokhmyanin | c912f9d9840 | p.haul: Introduce action scripts executed on different stagesCRIU has actions scripts which are executed on different stages, however when it is executed in RPC mode - scripts are not invoked. This patch introduces action scripts routine in p.haul itself. Action scripts are required to enable migration of a container with active NFS client running inside. Signed-off-by: Pavel Vokhmyanin <pvokhmyanin@virtuozzo.com> | |||
Pavel Emelyanov | cc1a00526cfM | Merge pull request #5 from pvokhmyanin/masterDrop ifdown from net_lock | |||
Pavel Vokhmyanin | 71d0c8b2a1d | Drop ifdown from net_lockBlocking network interfaces is non-essential for a dump, however it blocks traffic which is crucial for a successful checkpoint of a NFS client inside of the container. | |||
Pavel Vokhmyanin | 78d2b04dd5d | p.haul: Generate cgroup external mount map dynamicallylibvzctl generates map dynamically by iterating through the known list and looking up actual mount points, however p.haul just uses hardcoded map. Due to difference in algorithms they might generate different maps which leads to migration failure. Signed-off-by: Pavel Vokhmyanin <pvokhmyanin@virtuozzo.com> Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com> Signed-off-by: Pavel Emelyanov <xemu... | |||
Pavel Tikhomirov | e05a1424070 | add --auto_dedup optionadd option on local dump/predump, on page-server and on restore Signed-off-by: Pavel Tikhomirov <ptikhomirov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Pavel Emelyanov | fe02f1256ccM | Merge pull request #3 from dmitry-guryanov/fix2Fix syntax to partially comply PEP8 | |||
Dmitry Guryanov | 850830d7c3b | Fix syntax slightly to make flake8 happy (except tabs) | |||
Dmitry Guryanov | acdc55e228c | Add ability to check PEP8 compliance with flake8Let's do it via tox with extra checks from hacking module. To check syntax run: tox -e pep8 | |||
Pavel Emelyanov | ecdb1bb03efM | Merge pull request #1 from dmitry-guryanov/fix2Please, take patches from downstream | |||
Cyrill Gorcunov | 4c730d81305 | Fix rpc module name changeIn criu commit f3e412f9004486cbfab285bedf8d47d421551f8a the RPC module has been renamed to rpc_pb2.py thus we have to reflect this change here too. Note the renaming happened in CRIU-2.8 series. Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com> | CRIU-2 | ||
Dmitry Guryanov | 5b066fa9036 | add --nostart optionAdd ability to skip start on destination node if migration is run in restart mode. It's needed to support --nostart option of vzmigrate. | |||
Dmitry Guryanov | db375002cf2 | Fix restart migration on vzstorageUmount container before start on target host, because we can't mount ploop image, located on vzstorage from two hosts simultaneously. | |||
Alexander Burluka | bfa9601feb8 | Fix rollback for Virtuozzo container migration in restart modefail migration if can't start container on target host. Without current fix container migrate successfully but stay in stopped state. Author: nspiridonov@ Signed-off-by: Alexander Burluka <aburluka@virtuozzo.com> | |||
Nikita Spiridonov | 09434bcc519 | p.haul: ignore existing old/obsoleted active delta files for ploop disksEase ploop disks check on destination side - don't fail if old/obsoleted active delta files exist during migration of Virtuozzo containers. ploop library ignore such files and overwrite them. This is needed for migration over existing directory. In this mode vzmigrate reuse existing on destination static data of container left there by previously failed migration runs. Signed-off-by: Nikita S... | |||
Nikita Spiridonov | e7fd2bf3890 | p.haul: fix path to freezer cgroup for Virtuozzo containersFix path to freezer cgroup which phaul passes to criu on predumps and final dump. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | d4bf30add93 | p.haul: increase dump and predump timeouts for Virtuozzo containersBy 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. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 2aa71f41470 | p.haul: specify secondary ploop disks as external in dump request optionsSpecify secondary ploop disks (second, third and so on) as external in dump request options for Virtuzzo containers. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | d787b7fa06d | p.haul: minor functions reordering in Virtuozzo haulerMove __parse_fdfs_arg function of Virtuozzo hauler to the bottom of the class to improve readability a bit. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 771cebc202f | p.haul: add Virtuozzo specific argument to pass list of secondary ploop diskAdd Virtuozzo specific --vz-secondary-disks argument needed to pass list of secondary ploop disks. Consider all additional disks of container (second, third and so on) are secondary. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 00c8920c16f | p.haul: make it possible to adjust predump requestPass htype to criu_predump function to make it possible to adjust predump requests in specific hauler modules. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 227b1c3baa9 | p.haul: specify freezer cgroup for predump and dump requests in vz moduleSpecify freezer cgroup for predump and dump rpc requests for Virtuozzo containers. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Arthur Lockman | fa0869e72ad | p.haul: Repairing WebGUIPreviously the WebGUI had a redundant --fdmem parameter, causing process migration to fail. This is now removed. Signed-off-by: Arthur Lockman <alockman@redhat.com> Acked-by: Adrian Reber <areber@redhat.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 9c055fa3bdf | p.haul: add note about patches subject prefix to readmeAdd note about p.haul patches subject prefix and example of the command to set subject prefix for local repository to readme file. Subject prefix needed since we are going to use patchwork for criu and p.haul and it is needed to separate patches for each project. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | cc4c2c1abf9 | p.haul: fix error handling after successfull restore at target hostSince we can't fail migration after htype was restored (or started for restart migration mode) at target host need to catch all exceptions in source part of phaul during final cleanup. Phaul can ruin running container without such exceptions handling. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 13c2afe4116 | p.haul: create Virtuozzo specific dump images during final dumpFor Virtuozzo containers vzctl create addtional images during local dump of the container - vz_clock_bootbased.img, vz_clock_monotonic.img, vz_aio_max_nr.img and vz_core_pattern.img. Need to duplicate this logic in vz module of phaul since we use "vzctl restore" for final restore and it rely on this additional images. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: ... | |||
Nikita Spiridonov | 86668541e8b | p.haul: minor style improvements in vz moduleMove __setup_restore_extra_args and __remove_restore_extra_args functions below final_restore function to improve readability. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | b623847d045 | p.haul: forward correct verbosity level to criu restoreForward correct verbosity level to criu restore during Virtuozzo containers live migration. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 28a00e9cf32 | p.haul: fix root_task_pid function for Virtuozzo containersIt is incorrect to grab root pid of container from first line of cgroups tasks file; get root pid from pidfile instead. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 978ab87902e | p.haul: improve subprocess output loggingAdd log_subprocess_output function which add offset and prefix to subprocess output. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 3900fcd4725 | p.haul: move log_uncaught_exception function to util.pymove log_uncaught_exception function from p.haul and p.haul-service to util.py file to avoid code duplication. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 77f88f670b2 | p.haul: log few blank lines before each runSince phaul and phaul-service don't overwrite logs anymore, log few blank lines before each run to make logs more readable. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 08000dd8f25 | p.haul: tweak loggers configurationDon't overwrite phaul and phaul-service log files - append to the end instead. Log pid after timestamp for each line. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | e5abb7e561b | p.haul: fix lint warningadd blank line. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 4c88f3be5b6 | p.haul: handle migration with id changeHandle migration with id change. For example, now we can migrate Virtuozzo container 123 to new id 124 using "--dst-id" option. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 571c47348eb | p.haul: add --dst-id optionAdd --dst-id option needed to handle migration with id change. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Alexander Burluka | 3243399638b | p.haul: prevent VZ CT ARP ping on destination restorevzctl mades address detection before ARP announce --skip_arpdetect option is undocumented that prevents detection. Adding this option will decrease container downtime. Signed-off-by: Alexander Burluka <aburluka@virtuozzo.com> Acked-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> | |||
Nikita Spiridonov | 056e13f37c5 | p.haul: add --skip-cpu-check and --skip-criu-check optionsAdd --skip-cpu-check and --skip-criu-check options needed to skip specified sanity check. In addition --force option still skip all sanity checks. Signed-off-by: Nikita Spiridonov <nspiridonov@virtuozzo.com> Signed-off-by: Pavel Emelyanov <xemul@virtuozzo.com> |
Commits
Author | Commit | Commit date | Issues | |
---|---|---|---|---|
Pavel Emelyanov | 02199d87588M | |||
Pavel Vokhmyanin | e91120c59e7 | |||
Pavel Emelyanov | 526bc5f7494M | |||
Pavel Vokhmyanin | fc8375c22ff | |||
Pavel Emelyanov | 29f30036597M | |||
Pavel Vokhmyanin | a8b190ab5ae | |||
Pavel Emelyanov | 2c81dee6e2dM | |||
Pavel Vokhmyanin | cc55e6aaa13 | |||
Veronika Kabatova | bb44a2e7e19 | |||
Veronika Kabatova | e4c3a30d637 | |||
Veronika Kabatova | 3299a8cb483 | |||
Veronika Kabatova | dae1264b002 | |||
Pavel Emelyanov | 9e01ad67e13M | |||
Pavel Vokhmyanin | c912f9d9840 | |||
Pavel Emelyanov | cc1a00526cfM | |||
Pavel Vokhmyanin | 71d0c8b2a1d | |||
Pavel Vokhmyanin | 78d2b04dd5d | |||
Pavel Tikhomirov | e05a1424070 | |||
Pavel Emelyanov | fe02f1256ccM | |||
Dmitry Guryanov | 850830d7c3b | |||
Dmitry Guryanov | acdc55e228c | |||
Pavel Emelyanov | ecdb1bb03efM | |||
Cyrill Gorcunov | 4c730d81305 | CRIU-2 | ||
Dmitry Guryanov | 5b066fa9036 | |||
Dmitry Guryanov | db375002cf2 | |||
Alexander Burluka | bfa9601feb8 | |||
Nikita Spiridonov | 09434bcc519 | |||
Nikita Spiridonov | e7fd2bf3890 | |||
Nikita Spiridonov | d4bf30add93 | |||
Nikita Spiridonov | 2aa71f41470 | |||
Nikita Spiridonov | d787b7fa06d | |||
Nikita Spiridonov | 771cebc202f | |||
Nikita Spiridonov | 00c8920c16f | |||
Nikita Spiridonov | 227b1c3baa9 | |||
Arthur Lockman | fa0869e72ad | |||
Nikita Spiridonov | 9c055fa3bdf | |||
Nikita Spiridonov | cc4c2c1abf9 | |||
Nikita Spiridonov | 13c2afe4116 | |||
Nikita Spiridonov | 86668541e8b | |||
Nikita Spiridonov | b623847d045 | |||
Nikita Spiridonov | 28a00e9cf32 | |||
Nikita Spiridonov | 978ab87902e | |||
Nikita Spiridonov | 3900fcd4725 | |||
Nikita Spiridonov | 77f88f670b2 | |||
Nikita Spiridonov | 08000dd8f25 | |||
Nikita Spiridonov | e5abb7e561b | |||
Nikita Spiridonov | 4c88f3be5b6 | |||
Nikita Spiridonov | 571c47348eb | |||
Alexander Burluka | 3243399638b | |||
Nikita Spiridonov | 056e13f37c5 |