Re: [PATCH v4 00/15] parallels: rewrite driver with parallels SDK

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 02 December 2014 14:15:26 Peter Krempa wrote:
> On 12/01/14 19:49, Maxim Nestratov wrote:
> > 01.12.2014 18:38, Dmitry Guryanov пишет:
> >> This patch series replaces all code, which used prlctl command
> >> to interact with parallels cloud server with calls to
> >> parallels sdk functions.
> >> 
> >> The model of this driver remain almost the same - in creates a
> >> list of virDomainObj objects on connect and then functions, which
> >> returns different information get info from this list.
> >> 
> >> Changes in v2:
> >>   * Rebase to latest libvirt sources
> >>   * Use only "parallels" prefix for functions in parallelsDriver,
> >>   
> >>     so that make check will pass
> >>   
> >>   * Update privconn->domains in case we change something from current
> >>   
> >>     connection.
> >> 
> >> Changes in v3:
> >> * in parallels: get domain info with SDK:
> >> replace
> >> +    case VIR_ARCH_X86_64:
> >> with
> >> +    case PCM_CPU_MODE_64:
> >> 
> >> Changes in v4:
> >> * handle onReboot, onPoweroff and onCrash properly
> >> * handle disks cache mode
> >> * don't set net interface name if virDomainNetDef.ifname is NULL
> >> * improve error handling
> >> 
> >> Alexander Burluka (4):
> >>    parallels: get domain info with SDK
> >>    parallels: handle events from parallels server
> >>    parallels: added function virDomainIsActive()
> >>    parallels: Add domainCreateWithFlags() function.
> >> 
> >> Dmitry Guryanov (11):
> >>    parallels: move IS_CT macro to parallels_utils.h
> >>    parallels: move parallelsDomNotFoundError to parallels_utils.h
> >>    parallels: reimplement functions, which change domain state
> >>    parallels: rewrite parallelsApplyConfig with SDK
> >>    parallels: create VMs and containers with sdk
> >>    parallels: refactor parallelsDomainDefineXML
> >>    parallels: add cdroms support
> >>    parallels: implement domainUndefine and domainUndefineFlags
> >>    parallels: return PRL_RESULT from waitJob and getJobResult
> >>    parallels: fix getJobResultHelper
> >>    parallels: report proper error in Create/Destroy/Suspend e.t.c.
> >>   
> >>   src/parallels/parallels_driver.c | 2456
> >> 
> >> +++++++-----------------------------
> >> 
> >>   src/parallels/parallels_sdk.c    | 2544
> >> 
> >> +++++++++++++++++++++++++++++++++++++-
> >> 
> >>   src/parallels/parallels_sdk.h    |   25 +
> >>   src/parallels/parallels_utils.h  |   11 +
> >>   4 files changed, 3033 insertions(+), 2003 deletions(-)
> > 
> > Series looks good to me as a first step to switch to SDK.
> 
> I'll take over from here. I need to install the SDK first so I can make
> sure that everything compiles before pushing. I'll hope I get to push
> this this week.

Thanks!

BTW, it fails to compile now, the patch
"parallels: fix compilation of parallels_storage.c" fixes it.


> 
> Peter

-- 
Dmitry Guryanov

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]