On 02/11/2014 03:39 PM, Rodrigo Trujillo wrote:
The module vmscreenshot is preventing this error using Image.load function inside a try/except. However right after the except block, the code calls Image.thumbnail() function, which calls the same load() function internally. Then the error could happen again. This patch fixes this possible problem.
Which error? I remembered Royce added im.load() to fix a problem in POWER machines Take a look at: git show 5831b4de
Signed-off-by: Rodrigo Trujillo <rodrigo.trujillo@linux.vnet.ibm.com> --- src/kimchi/screenshot.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/kimchi/screenshot.py b/src/kimchi/screenshot.py index 5cfea96..91c14a6 100644 --- a/src/kimchi/screenshot.py +++ b/src/kimchi/screenshot.py @@ -177,10 +177,9 @@ class VMScreenshot(object): try: # Prevent Image lib from lazy load, # work around pic truncate validation in thumbnail generation - im.load() + im.thumbnail(self.THUMBNAIL_SIZE) except Exception as e: kimchi_log.warning("Image load with warning: %s." % e) - im.thumbnail(self.THUMBNAIL_SIZE) im.save(thumbnail, "PNG")
self.info['thumbnail'] = thumbnail