[pve-devel] [PATCH container 0/9] lxc pending changes

[Oguz Bektas]

this series makes it possible to add/revert/delete pending changes in
the backend.

it depends on my previous patch for pve-guest-common, for enabling
inheritance of pending changes related methods into PVE::LXC::Config

some notes or to-be-fixed-soon-after-review points:
    [1]. gui patches are coming soon
    [2]. right now pending changes are applied __only__ at vm_start.
    [3]. --delete $opt --force is not implemented (is in schema, but has no effect atm)
    [4]. clones will __keep__ pending changes in config (for now)
    [5]. there are a couple TODO/FIXME's inside, since they were extra
features or unrelated bugs (which i think should be fixed in separate
commits later since they don't directly affect pending changes functionality)

comments:

[2]: i thought it was best to keep it like this for the first version,
until we decide what's the best way to go about it.

[3]: it seemed quite tricky to implement live force-delete of mounpoints
because of a few reasons, like unmount not being allowed or mp being on
network storage etc.

[4]: will fix this in v2 after the first review

[5]: one bug, where if swap option is deleted while ct is running,
memory.memsw.limit_in_bytes is set to infinite instead of zero. the
other one is the --force implementation.



Oguz Bektas (9):
  add pending section to lxc config parser/writer
  adapt config GET call for taking pending changes
  adapt config PUT to use 'revert' and 'force' parameters
  remove trailing whitespace
  add 'pending' API method to LXC
  add 'pct pending' command
  add vmconfig_hotplug_pending and vmconfig_apply_pending
  rework update_pct_config to write and apply pending changes
  apply pending changes when container is started

 src/PVE/API2/LXC.pm        |  90 +++++-
 src/PVE/API2/LXC/Config.pm |  88 +++---
 src/PVE/CLI/pct.pm         |  27 ++
 src/PVE/LXC.pm             |   7 +
 src/PVE/LXC/Config.pm      | 549 ++++++++++++++++++++++++-------------
 5 files changed, 520 insertions(+), 241 deletions(-)

-- 
2.20.1