)]}'
{"cinder/volume/api.py":[{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"782ba5f43dd57c77f95142a2f53ac538452a831b","unresolved":false,"context_lines":[{"line_number":1918,"context_line":"            if instance_uuid:"},{"line_number":1919,"context_line":"                override \u003d True"},{"line_number":1920,"context_line":"                for attachment in vref.volume_attachment:"},{"line_number":1921,"context_line":"                    if attachment.instance_uuid !\u003d instance_uuid:"},{"line_number":1922,"context_line":"                        override \u003d False"},{"line_number":1923,"context_line":"                        break"},{"line_number":1924,"context_line":""},{"line_number":1925,"context_line":"            if not override:"}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_b67def1d","line":1922,"range":{"start_line":1921,"start_character":20,"end_line":1922,"end_character":40},"updated":"2017-05-30 21:39:28.000000000","message":"Wouldn\u0027t it be better this way:\n\nif attachment.instance_uuid \u003d\u003d instance_uuid:\n    override \u003d True","commit_id":"220dafc65cdffaefa867380a476aa7fcd55268c4"},{"author":{"_account_id":9961,"name":"tinytmy","email":"tangmeiyan77@gmail.com","username":"tinytmy"},"change_message_id":"ef3716914b5fa6f56871bd665f6ec1268f2b1e06","unresolved":false,"context_lines":[{"line_number":1918,"context_line":"            if instance_uuid:"},{"line_number":1919,"context_line":"                override \u003d True"},{"line_number":1920,"context_line":"                for attachment in vref.volume_attachment:"},{"line_number":1921,"context_line":"                    if attachment.instance_uuid !\u003d instance_uuid:"},{"line_number":1922,"context_line":"                        override \u003d False"},{"line_number":1923,"context_line":"                        break"},{"line_number":1924,"context_line":""},{"line_number":1925,"context_line":"            if not override:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_7a485489","line":1922,"range":{"start_line":1921,"start_character":20,"end_line":1922,"end_character":40},"in_reply_to":"df140735_b271680f","updated":"2018-09-27 07:43:53.000000000","message":"volume\u0027s status check does not work.\nFor example:\nvolume\u0027s status is backing-up,we can also do attach","commit_id":"220dafc65cdffaefa867380a476aa7fcd55268c4"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"ef695488eb6bcd49eb01c89fed34fd594f953657","unresolved":false,"context_lines":[{"line_number":1918,"context_line":"            if instance_uuid:"},{"line_number":1919,"context_line":"                override \u003d True"},{"line_number":1920,"context_line":"                for attachment in vref.volume_attachment:"},{"line_number":1921,"context_line":"                    if attachment.instance_uuid !\u003d instance_uuid:"},{"line_number":1922,"context_line":"                        override \u003d False"},{"line_number":1923,"context_line":"                        break"},{"line_number":1924,"context_line":""},{"line_number":1925,"context_line":"            if not override:"}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_d6bce36a","line":1922,"range":{"start_line":1921,"start_character":20,"end_line":1922,"end_character":40},"in_reply_to":"df140735_b67def1d","updated":"2017-05-30 21:48:49.000000000","message":"Wanted to make sure it covered the case of multiple attachment records, probably not a likely scenario, but could happen.","commit_id":"220dafc65cdffaefa867380a476aa7fcd55268c4"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"a776c66d08c6a8c770d804afe105a4ae67cb2def","unresolved":false,"context_lines":[{"line_number":1918,"context_line":"            if instance_uuid:"},{"line_number":1919,"context_line":"                override \u003d True"},{"line_number":1920,"context_line":"                for attachment in vref.volume_attachment:"},{"line_number":1921,"context_line":"                    if attachment.instance_uuid !\u003d instance_uuid:"},{"line_number":1922,"context_line":"                        override \u003d False"},{"line_number":1923,"context_line":"                        break"},{"line_number":1924,"context_line":""},{"line_number":1925,"context_line":"            if not override:"}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_b271680f","line":1922,"range":{"start_line":1921,"start_character":20,"end_line":1922,"end_character":40},"in_reply_to":"df140735_d6bce36a","updated":"2017-05-31 12:53:09.000000000","message":"Fair point :)","commit_id":"220dafc65cdffaefa867380a476aa7fcd55268c4"}]}