----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/882/ -----------------------------------------------------------
Review request for OpenLMI Developers.
Repository: openlmi-tools
Description -------
It does not make sense to refresh all the LMIInstances after method call. This patch introduces an optional (keyword) argument of a method call, by which you can instruct lmishell to refresh such instance or let it be. By default, no instance is refreshed.
Example:
$ instance.MethodCall({"param1" : val1, "param2" : val2}, RefreshInstance=True) $
Diffs -----
cli/lmi/shell/LMIMethod.py 80f422e117057f93e6b9c65519982a4d1ae435db
Diff: http://reviewboard-openlmi.rhcloud.com/r/882/diff/
Testing -------
Thanks,
Peter Hatina
----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/882/#review1220 -----------------------------------------------------------
Ship it!
Ship It!
- Roman Rakus
On Sept. 20, 2013, 11:45 a.m., Peter Hatina wrote:
This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/882/
(Updated Sept. 20, 2013, 11:45 a.m.)
Review request for OpenLMI Developers.
Repository: openlmi-tools
Description
It does not make sense to refresh all the LMIInstances after method call. This patch introduces an optional (keyword) argument of a method call, by which you can instruct lmishell to refresh such instance or let it be. By default, no instance is refreshed.
Example:
$ instance.MethodCall({"param1" : val1, "param2" : val2}, RefreshInstance=True) $
Diffs
cli/lmi/shell/LMIMethod.py 80f422e117057f93e6b9c65519982a4d1ae435db
Diff: http://reviewboard-openlmi.rhcloud.com/r/882/diff/
Testing
Thanks,
Peter Hatina
----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/882/ -----------------------------------------------------------
(Updated Sept. 20, 2013, 12:54 p.m.)
Status ------
This change has been marked as submitted.
Review request for OpenLMI Developers.
Repository: openlmi-tools
Description -------
It does not make sense to refresh all the LMIInstances after method call. This patch introduces an optional (keyword) argument of a method call, by which you can instruct lmishell to refresh such instance or let it be. By default, no instance is refreshed.
Example:
$ instance.MethodCall({"param1" : val1, "param2" : val2}, RefreshInstance=True) $
Diffs -----
cli/lmi/shell/LMIMethod.py 80f422e117057f93e6b9c65519982a4d1ae435db
Diff: http://reviewboard-openlmi.rhcloud.com/r/882/diff/
Testing -------
Thanks,
Peter Hatina
openlmi-reviews@lists.fedorahosted.org
-
Stephen Gallagher