----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/1896/#review3134 -----------------------------------------------------------
cli/lmi/shell/LMIMethod.py http://reviewboard-openlmi.rhcloud.com/r/1896/#comment1751
I am OK with the code; but please, don't use this style of indenting.
- Peter Hatina
On July 14, 2014, 7:37 a.m., Michal Minar wrote:
This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/1896/
(Updated July 14, 2014, 7:37 a.m.)
Review request for OpenLMI Developers.
Repository: openlmi-tools
Description
shell: better handle job output parameters
Due to openlmi's workaround of pegasus' shortcoming, JobOutParameters property of job instance is something else then it should be. It's an instance of __MethodParameters_<method_name>_Result class which derives from __MethodParameters_<method_name> class. The latter contains input parameters of asynchronous method. Therefor JobOutParameters may contain input parameters as well. Let's filter them out because this may confuse client scripts.
Diffs
cli/lmi/shell/LMIMethod.py 97b8cb66066ec968c0b05e72de0e7074482abdcb
Diff: http://reviewboard-openlmi.rhcloud.com/r/1896/diff/
Testing
Thanks,
Michal Minar