Review Request 683: openlmi-software [1/1] software: added classes for asynchronous methods

[Stephen Gallagher]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/683/
-----------------------------------------------------------

Review request for OpenLMI Developers.


Repository: openlmi-providers


Description
-------

software: added classes for asynchronous methods

Pegasus does not support instances of unknown classes, that's why we
need to define additional classes representing parameters of
asynchronous methods. These will be used in
LMI_ConcreteJob.JobOutParameters and CIM_InstMethodCall.MethodParameters
properties.


Diffs
-----

  mof/60_LMI_Software.mof bb02e3f17f52c6c76cb62868fc0f7613d52dbcf6 

Diff: http://reviewboard-openlmi.rhcloud.com/r/683/diff/


Testing
-------


Thanks,

Michal Minar

[Stephen Gallagher]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/683/#review882
-----------------------------------------------------------

Ship it!


Ship It!


mof/60_LMI_Software.mof
<http://reviewboard-openlmi.rhcloud.com/r/683/#comment471>

    does it make sense to put job here? Instances of this class *are* properties of the
job... I did not put jobs here in Storage.


- Jan Safranek


On Aug. 8, 2013, 10:04 a.m., Michal Minar wrote:
>  
>  -----------------------------------------------------------
>  This is an automatically generated e-mail. To reply, visit:
>  http://reviewboard-openlmi.rhcloud.com/r/683/
>  -----------------------------------------------------------
>  
>  (Updated Aug. 8, 2013, 10:04 a.m.)
>  
>  
>  Review request for OpenLMI Developers.
>  
>  
>  Repository: openlmi-providers
>  
>  
>  Description
>  -------
>  
>  software: added classes for asynchronous methods
>  
>  Pegasus does not support instances of unknown classes, that's why we
>  need to define additional classes representing parameters of
>  asynchronous methods. These will be used in
>  LMI_ConcreteJob.JobOutParameters and CIM_InstMethodCall.MethodParameters
>  properties.
>  
>  
>  Diffs
>  -----
>  
>    mof/60_LMI_Software.mof bb02e3f17f52c6c76cb62868fc0f7613d52dbcf6 
>  
>  Diff: http://reviewboard-openlmi.rhcloud.com/r/683/diff/
>  
>  
>  Testing
>  -------
>  
>  
>  Thanks,
>  
>  Michal Minar
>  
>

[Stephen Gallagher]

>  On Aug. 8, 2013, 8:27 a.m., Jan Safranek wrote:
>  > mof/60_LMI_Software.mof, line 1012
>  >
> <http://reviewboard-openlmi.rhcloud.com/r/683/diff/1/?file=3930#file3930li...
>  >
>  >     does it make sense to put job here? Instances of this class *are* properties of
> the job... I did not put jobs here in Storage. 
You are right. Makes no sense :). Thanks for observation.


- Michal


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/683/#review882
-----------------------------------------------------------


On Aug. 8, 2013, 8:04 a.m., Michal Minar wrote:
>  
>  -----------------------------------------------------------
>  This is an automatically generated e-mail. To reply, visit:
>  http://reviewboard-openlmi.rhcloud.com/r/683/
>  -----------------------------------------------------------
>  
>  (Updated Aug. 8, 2013, 8:04 a.m.)
>  
>  
>  Review request for OpenLMI Developers.
>  
>  
>  Repository: openlmi-providers
>  
>  
>  Description
>  -------
>  
>  software: added classes for asynchronous methods
>  
>  Pegasus does not support instances of unknown classes, that's why we
>  need to define additional classes representing parameters of
>  asynchronous methods. These will be used in
>  LMI_ConcreteJob.JobOutParameters and CIM_InstMethodCall.MethodParameters
>  properties.
>  
>  
>  Diffs
>  -----
>  
>    mof/60_LMI_Software.mof bb02e3f17f52c6c76cb62868fc0f7613d52dbcf6 
>  
>  Diff: http://reviewboard-openlmi.rhcloud.com/r/683/diff/
>  
>  
>  Testing
>  -------
>  
>  
>  Thanks,
>  
>  Michal Minar
>  
>

[Stephen Gallagher]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/683/
-----------------------------------------------------------

(Updated Aug. 8, 2013, 10:57 a.m.)


Status
------

This change has been marked as submitted.


Review request for OpenLMI Developers.


Repository: openlmi-providers


Description
-------

software: added classes for asynchronous methods

Pegasus does not support instances of unknown classes, that's why we
need to define additional classes representing parameters of
asynchronous methods. These will be used in
LMI_ConcreteJob.JobOutParameters and CIM_InstMethodCall.MethodParameters
properties.


Diffs
-----

  mof/60_LMI_Software.mof bb02e3f17f52c6c76cb62868fc0f7613d52dbcf6 

Diff: http://reviewboard-openlmi.rhcloud.com/r/683/diff/


Testing
-------


Thanks,

Michal Minar