Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-10-03 Thread Daniel Hunsaker
Actually, that part wasn't me, but since the answer is yes, I'll look into getting QEMU to save state to disk so we can do the rest. :-) And now to sleep, finally... On Oct 3, 2014 2:53 AM, "Dietmar Maurer" wrote: > > > 1.) Implement suspend/resume API > > > 2.) add it to pvectl > > > 3.) Imple

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-10-03 Thread Dietmar Maurer
> > 1.) Implement suspend/resume API > > 2.) add it to pvectl > > 3.) Implement suspend/resume GUI (extjs) > > 4.) Implement suspend/resume GUI (mobile) > Alright, I'll make that happen tomorrow.  Currently just after 02:00 here.  > :-) Thanks! > > I also have some further ideas. Currently qemu

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-10-03 Thread Daniel Hunsaker
> > How would you recommend I split the changes? They're all related directly to > > providing suspend/resume support. > > 1.) Implement suspend/resume API > 2.) add it to pvectl > 3.) Implement suspend/resume GUI (extjs) > 4.) Implement suspend/resume GUI (mobile) Alright, I'll make that happen

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-10-03 Thread Dietmar Maurer
> I noticed the whitespace changes after I sent this one, and resent it without > them shortly after.  Sent a reply that I was going to resend, but it seems > not all > my emails get through to the list? I also received the second mail. But I already answered the first mail by that time. > How

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-10-03 Thread Daniel Hunsaker
I noticed the whitespace changes after I sent this one, and resent it without them shortly after. Sent a reply that I was going to resend, but it seems not all my emails get through to the list? How would you recommend I split the changes? They're all related directly to providing suspend/resume

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-10-02 Thread Dietmar Maurer
First, thanks for the patch. But please can you split the patch into smaller ones? > Signed-off-by: Dan Hunsaker > --- > PVE/API2/OpenVZ.pm| 308 > +++--- > PVE/OpenVZ.pm | 92 - > bin/pvectl| 16

[pve-devel] [PATCH] Add CT suspend/resume to PVE API (resubmit without whitespace changes)

2014-10-02 Thread Daniel Hunsaker
From: Dan Hunsaker As discussed in a previous thread, following is a patch to support container suspend (via vzctl chkpnt) and resume (via vzctl restore). - Added /nodes/{node}/openvz/{vmid}/status/suspend to API - Added /nodes/{node}/openvz/{vmid}/status/resume to API - Adapted vm_suspend/vm_re

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-10-02 Thread Daniel Hunsaker
Oops. That's a lot of whitespace changes. Let me resubmit again without them... ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-10-02 Thread Daniel Hunsaker
From: Dan Hunsaker As discussed in a previous thread, following is a patch to support container suspend (via vzctl chkpnt) and resume (via vzctl restore). - Added /nodes/{node}/openvz/{vmid}/status/suspend to API - Added /nodes/{node}/openvz/{vmid}/status/resume to API - Adapted vm_suspend/vm_re

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-03-04 Thread Daniel Hunsaker
Alright. If I need to rebase again by then, let me know. On Mar 4, 2014 1:02 AM, "Dietmar Maurer" wrote: > Thanks. > > I will try to add that after the 3.2 release, which is planned next week. > > > As discussed in a previous thread, following is a patch to support > container > > suspend (via v

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-03-04 Thread Dietmar Maurer
Thanks. I will try to add that after the 3.2 release, which is planned next week. > As discussed in a previous thread, following is a patch to support container > suspend (via vzctl chkpnt) and resume (via vzctl restore). ___ pve-devel mailing list pve

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-03-03 Thread Daniel Hunsaker
Rebased on latest public master. - Daniel Hunsaker Owner / Developer Lei's Genesis Experiment: Code For The Future! On Mon, Mar 3, 2014 at 2:16 PM, Daniel Hunsaker wrote: > As discussed in a previous thread, following is a patch to support > container > suspend (via vzctl chkpnt) and resume (vi

[pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-03-03 Thread Daniel Hunsaker
As discussed in a previous thread, following is a patch to support container suspend (via vzctl chkpnt) and resume (via vzctl restore). - Added /nodes/{node}/openvz/{vmid}/status/suspend to API - Added /nodes/{node}/openvz/{vmid}/status/resume to API - Adapted vm_suspend/vm_resume from PVE/QemuSer

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-03-03 Thread Daniel Hunsaker
> > To: pve-devel@pve.proxmox.com > > Subject: [pve-devel] [PATCH] Add CT suspend/resume to PVE API > > > > As discussed in a previous thread, following is a patch to support > container > > suspend (via vzctl chkpnt) and resume (via vzctl restore). > >

Re: [pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-03-02 Thread Dietmar Maurer
l-boun...@pve.proxmox.com] On Behalf > Of Daniel Hunsaker > Sent: Samstag, 01. März 2014 17:37 > To: pve-devel@pve.proxmox.com > Subject: [pve-devel] [PATCH] Add CT suspend/resume to PVE API > > As discussed in a previous thread, following is a patch to support container > suspend (via vz

[pve-devel] [PATCH] Add CT suspend/resume to PVE API

2014-03-01 Thread Daniel Hunsaker
As discussed in a previous thread, following is a patch to support container suspend (via vzctl chkpnt) and resume (via vzctl restore). - Added /nodes/{node}/openvz/{vmid}/status/suspend to API - Added /nodes/{node}/openvz/{vmid}/status/resume to API - Adapted vm_suspend/vm_resume from PVE/QemuSer