)]}'
{"cinder/brick/local_dev/lvm.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"c0957c2798f516298edf63dcf533387f38411093","unresolved":false,"context_lines":[{"line_number":17,"context_line":"LVM class for performing LVM operations."},{"line_number":18,"context_line":"\"\"\""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import itertools"},{"line_number":21,"context_line":"import math"},{"line_number":22,"context_line":"import os"},{"line_number":23,"context_line":"import re"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2fa921_dc1321b7","line":20,"updated":"2015-10-07 13:49:10.000000000","message":"Have to remove this unused import","commit_id":"679a90283628eee9a58ee96ccd211d5c2df9b03c"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"d5ecc59fd1aff322ae37e89edae7c05be25c0242","unresolved":false,"context_lines":[{"line_number":290,"context_line":"        lv_list \u003d []"},{"line_number":291,"context_line":"        if out is not None:"},{"line_number":292,"context_line":"            volumes \u003d out.split()"},{"line_number":293,"context_line":"            for vg, name, size in moves.zip(*[iter(volumes)] * 3):"},{"line_number":294,"context_line":"                lv_list.append({\"vg\": vg, \"name\": name, \"size\": size})"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        return lv_list"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a2fa921_450de28e","line":293,"updated":"2015-10-07 19:00:10.000000000","message":"There\u0027s a pylint error here:\n\ncinder/brick/local_dev/lvm.py:293: [E1101, LVM.get_lv_info] Instance of \u0027_MovedItems\u0027 has no \u0027zip\u0027 member","commit_id":"87355d6fdd2f87ba464f375d6c6c21f9f310648d"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"92b7f01243414d71ebbad2987f2e8a023782f12c","unresolved":false,"context_lines":[{"line_number":290,"context_line":"        lv_list \u003d []"},{"line_number":291,"context_line":"        if out is not None:"},{"line_number":292,"context_line":"            volumes \u003d out.split()"},{"line_number":293,"context_line":"            for vg, name, size in moves.zip(*[iter(volumes)] * 3):"},{"line_number":294,"context_line":"                lv_list.append({\"vg\": vg, \"name\": name, \"size\": size})"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        return lv_list"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a2fa921_eb215c4e","line":293,"in_reply_to":"7a2fa921_450de28e","updated":"2015-10-07 21:03:42.000000000","message":"I think it\u0027s worth checking out that pylint error.  I had just noticed that in the CI log failure myself.","commit_id":"87355d6fdd2f87ba464f375d6c6c21f9f310648d"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"eab820230e0e856e336e455250a3cdca8d6c2efa","unresolved":false,"context_lines":[{"line_number":290,"context_line":"        lv_list \u003d []"},{"line_number":291,"context_line":"        if out is not None:"},{"line_number":292,"context_line":"            volumes \u003d out.split()"},{"line_number":293,"context_line":"            for vg, name, size in moves.zip(*[iter(volumes)] * 3):"},{"line_number":294,"context_line":"                lv_list.append({\"vg\": vg, \"name\": name, \"size\": size})"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        return lv_list"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a2fa921_3d6d4531","line":293,"in_reply_to":"7a2fa921_450de28e","updated":"2015-10-08 10:35:14.000000000","message":"Sadly, it\u0027s a bug in pylint: https://bitbucket.org/logilab/pylint/issues/200/\n\nI updated my change to disable pylint checks on moves.zip() until the bug is fixed.\n\nThe bug says that pylint 1.5 will better handle the six module, but I don\u0027t know if six.moves.zip() will be handled correctly or not, and pylint 1.5 is not released yet (it was announced in june :-/).","commit_id":"87355d6fdd2f87ba464f375d6c6c21f9f310648d"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"306ea1bebde70b6124b842bc796774a1afc0ef77","unresolved":false,"context_lines":[{"line_number":290,"context_line":"        lv_list \u003d []"},{"line_number":291,"context_line":"        if out is not None:"},{"line_number":292,"context_line":"            volumes \u003d out.split()"},{"line_number":293,"context_line":"            for vg, name, size in moves.zip(*[iter(volumes)] * 3):"},{"line_number":294,"context_line":"                lv_list.append({\"vg\": vg, \"name\": name, \"size\": size})"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        return lv_list"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a2fa921_bca96c7d","line":293,"in_reply_to":"7a2fa921_eb215c4e","updated":"2015-10-07 22:40:08.000000000","message":"This seems to be an artifact of how six.moves dynamically generates members vs how pylint scans for errors.\n\nThis code is unit tested, changing it will cause expected failures, so I think it\u0027s correct:\n\nhttp://paste.openstack.org/raw/475669/","commit_id":"87355d6fdd2f87ba464f375d6c6c21f9f310648d"}]}