)]}'
{"bin/cinder-rtstool":[{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"1bd7cdaa0551ebd96b03602b8c3b2cf723963bad","unresolved":false,"context_lines":[{"line_number":132,"context_line":"    acl_new.chap_userid \u003d userid"},{"line_number":133,"context_line":"    acl_new.chap_password \u003d password"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"    m \u003d rtslib.MappedLUN(acl_new, 0, tpg_lun\u003d0)"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"def get_targets():"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAN3%2F%2F3cE%3D","line":135,"updated":"2013-03-08 08:59:18.000000000","message":"Variable \u0027m\u0027 is neither used, not returned. One similar instance in a function above (not changed in this commit). Am I missing something?","commit_id":"3a2bf66e38fe0000a20a015fd363d79b0634344f"},{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"b9358323ef80d778e23489d74ba3583d11312361","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"    acl_new.chap_userid \u003d userid"},{"line_number":79,"context_line":"    acl_new.chap_password \u003d password"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    m \u003d rtslib.MappedLUN(acl_new, lun_new.lun, lun_new.lun)"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    if initiator_iqns:"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAN3%2F%2F2xU%3D","line":80,"updated":"2013-03-08 18:47:05.000000000","message":"Can we remove the variable \u0027m\u0027 here too? Would make the code look more consistent. Sorry if my comment on previous patch set didnt pointed out this clearly.","commit_id":"fc391a9ec8a931bd3bdb5e872454d5afe3e2d1ae"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"1845dffb4013152429d921e3d2731e73c4aefae5","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"    acl_new.chap_userid \u003d userid"},{"line_number":79,"context_line":"    acl_new.chap_password \u003d password"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    m \u003d rtslib.MappedLUN(acl_new, lun_new.lun, lun_new.lun)"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    if initiator_iqns:"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAN3%2F%2F2vE%3D","line":80,"in_reply_to":"AAAAN3%2F%2F2xU%3D","updated":"2013-03-08 18:59:42.000000000","message":"Sure, no problem, I had just opted to change only the new code before. :)","commit_id":"fc391a9ec8a931bd3bdb5e872454d5afe3e2d1ae"},{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"b9358323ef80d778e23489d74ba3583d11312361","unresolved":false,"context_lines":[{"line_number":86,"context_line":"            acl_new \u003d rtslib.NodeACL(tpg_new, i, mode\u003d\u0027create\u0027)"},{"line_number":87,"context_line":"            acl_new.chap_userid \u003d userid"},{"line_number":88,"context_line":"            acl_new.chap_password \u003d password"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"            m \u003d rtslib.MappedLUN(acl_new, lun_new.lun, lun_new.lun)"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    tpg_new.enable \u003d 1"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAN3%2F%2F2xQ%3D","line":89,"updated":"2013-03-08 18:47:05.000000000","message":"Ditto.","commit_id":"fc391a9ec8a931bd3bdb5e872454d5afe3e2d1ae"}]}