)]}'
{"cinder/volume/drivers/tegile.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"a6aaf1e2e5251d4bb27ea4fc57da00767b5c0283","unresolved":false,"context_lines":[{"line_number":570,"context_line":"            params\u003dparams)"},{"line_number":571,"context_line":"        target_portal \u003d mapping_info[\u0027target_portal\u0027]"},{"line_number":572,"context_line":"        target_iqn \u003d mapping_info[\u0027target_iqn\u0027]"},{"line_number":573,"context_line":"        target_lun \u003d int(mapping_info[\u0027target_lun\u0027])"},{"line_number":574,"context_line":""},{"line_number":575,"context_line":"        provider_location \u003d \u0027%s %s %s\u0027 % (target_portal,"},{"line_number":576,"context_line":"                                          target_iqn,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_7c845e5d","line":573,"range":{"start_line":573,"start_character":21,"end_line":573,"end_character":25},"updated":"2016-10-07 09:35:46.000000000","message":"As I mentioned in the Launchpad bug[1], there\u0027s no proper explanation of what the problem really is and how this is fixing it.\n\nFor example the change in this line is completely useless, as in L575 it is converted to a string, so it doesn\u0027t matter if it\u0027s an int or a string to begin with.  So how is this helping fix a bug?\n\n[1]: https://bugs.launchpad.net/cinder/+bug/1630035/comments/2","commit_id":"d63b455b97fe1562c6150bcfc3b6023a60b160e0"},{"author":{"_account_id":17357,"name":"Abhilash Divakaran","email":"abhilashd@tegile.com","username":"abhilashd"},"change_message_id":"49ac41ca804de0f40d6ad7b1545e22efb063aa3a","unresolved":false,"context_lines":[{"line_number":570,"context_line":"            params\u003dparams)"},{"line_number":571,"context_line":"        target_portal \u003d mapping_info[\u0027target_portal\u0027]"},{"line_number":572,"context_line":"        target_iqn \u003d mapping_info[\u0027target_iqn\u0027]"},{"line_number":573,"context_line":"        target_lun \u003d int(mapping_info[\u0027target_lun\u0027])"},{"line_number":574,"context_line":""},{"line_number":575,"context_line":"        provider_location \u003d \u0027%s %s %s\u0027 % (target_portal,"},{"line_number":576,"context_line":"                                          target_iqn,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_a4123a55","line":573,"range":{"start_line":573,"start_character":21,"end_line":573,"end_character":25},"in_reply_to":"1a95cdbc_7c845e5d","updated":"2016-10-07 17:36:41.000000000","message":"Backend API was returning a string for LUN number, It was causing failures in our tests. \n\nI do see that change line 573 is useless. \n\nChange in lines 529 and 626 were the ones that fixed the problem.\n\nThanks for pointing it out.","commit_id":"d63b455b97fe1562c6150bcfc3b6023a60b160e0"}]}