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

Ship it!

Those were just nitpicks. Otherwise it's OK.

- Michal


On April 12th, 2013, 2:13 p.m. UTC, Jan Safranek wrote:

Review request for OpenLMI Developers.
By Jan Safranek.

Updated April 12, 2013, 2:13 p.m.

Repository: openlmi-providers

Description

Synchronize JobManager with storage.

JobManager here is older version of the one in openlmi-storage.
Let's sync it with the storage so I can use it from there.

The patch includes:

Fixed job expiration under SFCB.
Added LMI_StorageJob.JobInParameters and .JobOutParameters properties.
Allow python to exit the provider even if there are threads running.
Fixed job returning an error
Added possibility to set AffectedElements when a job finishes.
Added workaround for rhbz#920763

Diffs

  • src/python/openlmi/common/JobManager.py (a923f296d3eba40f1997a267b68525a44f50ffc7)

View Diff