[CRIU] p.haul web-gui

[Nikita Spiridonov]

As for me, phaul package (phaul directory inside p.haul repository) is not a proper place for procs.py, p_haul_web_gui.py and such logic. If web gui will be added it would be great to separate it completely from p.haul primary logic.
________________________________________
От: criu-bounces at openvz.org <criu-bounces at openvz.org> от имени Adrian Reber <adrian at lisas.de>
Отправлено: 14 октября 2015 г. 22:23
Кому: Pavel Emelianov
Копия: criu at openvz.org
Тема: Re: [CRIU] p.haul web-gui

On Wed, Oct 14, 2015 at 03:02:03PM +0300, Pavel Emelyanov wrote:
> > I have integrated an existing criu web-gui
> >
> >  https://github.com/ThomasJClark/criu-gui-demo
>
> Interesting :)
>
> > into p.haul. The above mentioned criu web-gui was re-implementing many
> > things p.haul already does and therefore I thought it might be
> > interesting. The web-gui shows the process trees from two systems and
> > the user can drag-and-drop a process from one system to another and
> > p.haul will migrate the process in the background. This was of course
> > before the stand-alone mode was removed ;-)
> >
> > The current code is here:
> >
> >  https://github.com/adrianreber/p.haul
> >
> > Before integrating this further into p.haul-wrap I wanted to ask if this
> > a feature which would be interesting to integrate into p.haul.
>
> Yes I think :) If it sits completely aside the rest of the code,
> I'd consider this as a nice web-giu extension :)

My current integration has the code in separate files (and directories)
and only a command-line option integrated into p.haul to use the code if
requested from the user. I will then submit this for review.

One more question I cherry-picked all the commits (33) from the original
repository into p.haul to keep the history. Would you prefer a single
commit for the p.haul import or all the patches I cherry-picked?

                Adrian
_______________________________________________
CRIU mailing list
CRIU at openvz.org
https://lists.openvz.org/mailman/listinfo/criu