[Libvirt-cim] [PATCHv2 0/3] Make Shutdown and Reboot state changes jobs

Sharad Mishra snmishra at linux.vnet.ibm.com
Wed Mar 7 17:12:02 UTC 2012


On Tue, 2012-03-06 at 14:55 -0300, Eduardo Lima (Etrunko) wrote:
> From: "Eduardo Lima (Etrunko)" <eblima at br.ibm.com>
> 
> This code is based on the VSMigrationService for the job creation and on
> ComputerSystemIndication for the job thread, which will listen for the
> respective domain event to mark the job as finished.
> 
> I am still not able to test this feature in my environment, every call to
> CBGetInstance fails with code error 100. This is happening in other places of
> the codebase, not only the ComputerSystem provider, but also ElementConfors,
> for instance. Please use the test case send in previous email for reference.

Eduardo, It is not clear if you started seeing this error code after
applying this patch or were you getting it before.

-Sharad

> 
> Differences from v1:
> 
>  - Fix conditional causing Reqstate not being set
>  - Fix domain event id for shutdown
>  - Now instead of using the domain events for both shutdown and reboot, there is
>    an intermediate patch, that will deal with the shutdown case by polling for
>    the domain state until it reflects the desired state.
> 
>    This workaround won't work for the reboot case, because is is not possible to
>    monitor the reboot event if not using the domain event callbacks. The third
>    patch of the series contains the complete solution for both cases.
> 
> Eduardo Lima (Etrunko) (3):
>   VSMigrationService: Move job state definitions to svpc_types.h
>   ComputerSystem: Make Shutdown state change a job
>   ComputerSystem: Reboot state change also a job
> 
>  schema/ComputerSystem.mof     |    9 ++
>  src/Virt_ComputerSystem.c     |  304 +++++++++++++++++++++++++++++++++++++++--
>  src/Virt_VSMigrationService.c |   14 +--
>  src/svpc_types.h              |   13 ++
>  4 files changed, 321 insertions(+), 19 deletions(-)
> 





More information about the Libvirt-cim mailing list