)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"68b4338e41c4fd5f564d11ce32e1af13e61bf1c5","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix: Instances page crashes on repeated confirm/revert resize actions"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add error handling to \"ConfirmResize.single\" and \"RevertResize.single\" methods."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I128049091f38e8db3c1524a5c4cb932f3e809714"},{"line_number":12,"context_line":"Closes-Bug: #1882918"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f560f44_bbe921b8","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":79},"updated":"2020-08-05 13:23:33.000000000","message":"Could you wrap this line in 80 chars?","commit_id":"fda24962e7773d000762b929f6e5bc52d73f3f61"},{"author":{"_account_id":27815,"name":"Huang Cheng","email":"hcn518@gmail.com","username":"pedh"},"change_message_id":"8c5d134ad948312133d016e72d09eaaa7c7ef613","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix: Instances page crashes on repeated confirm/revert resize actions"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add error handling to \"ConfirmResize.single\" and \"RevertResize.single\" methods."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I128049091f38e8db3c1524a5c4cb932f3e809714"},{"line_number":12,"context_line":"Closes-Bug: #1882918"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f560f44_2529e29a","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":79},"in_reply_to":"9f560f44_bbe921b8","updated":"2020-08-17 10:03:48.000000000","message":"Thank you for your advice, I\u0027ve updated the code also my emacs.d settings to meet the git commit message 50/72 rule. XD","commit_id":"fda24962e7773d000762b929f6e5bc52d73f3f61"}],"openstack_dashboard/dashboards/project/instances/tables.py":[{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"bd5dff570f5cba0043e1811f39b9ed866551fa3d","unresolved":false,"context_lines":[{"line_number":630,"context_line":"        except Exception:"},{"line_number":631,"context_line":"            exceptions.handle(request,"},{"line_number":632,"context_line":"                              _(\u0027Unable to confirm resize instance %s\u0027)"},{"line_number":633,"context_line":"                              % instance)"},{"line_number":634,"context_line":"        return shortcuts.redirect(request.get_full_path())"},{"line_number":635,"context_line":""},{"line_number":636,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_48b86b29","line":633,"range":{"start_line":633,"start_character":32,"end_line":633,"end_character":40},"updated":"2020-08-17 10:01:05.000000000","message":"It needs to be one of the below options:\n1) instance.name or instance.id\n2) instance.id\nI prefer the first option.","commit_id":"2e78bb22f79df4d09ee0df44b517c133e0c2cdb6"},{"author":{"_account_id":27815,"name":"Huang Cheng","email":"hcn518@gmail.com","username":"pedh"},"change_message_id":"f40961d7e5b1cfc07ca4a487669f183741c1c442","unresolved":false,"context_lines":[{"line_number":630,"context_line":"        except Exception:"},{"line_number":631,"context_line":"            exceptions.handle(request,"},{"line_number":632,"context_line":"                              _(\u0027Unable to confirm resize instance %s\u0027)"},{"line_number":633,"context_line":"                              % instance)"},{"line_number":634,"context_line":"        return shortcuts.redirect(request.get_full_path())"},{"line_number":635,"context_line":""},{"line_number":636,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_fa876eb5","line":633,"range":{"start_line":633,"start_character":32,"end_line":633,"end_character":40},"in_reply_to":"9f560f44_48b86b29","updated":"2020-08-18 11:47:24.000000000","message":"Thank you, the argument instance is actually instance_id, I change the name to instance_id to avoid misunderstanding.","commit_id":"2e78bb22f79df4d09ee0df44b517c133e0c2cdb6"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"bd5dff570f5cba0043e1811f39b9ed866551fa3d","unresolved":false,"context_lines":[{"line_number":649,"context_line":"        except Exception:"},{"line_number":650,"context_line":"            exceptions.handle(request,"},{"line_number":651,"context_line":"                              _(\u0027Unable to revert resize instance %s\u0027)"},{"line_number":652,"context_line":"                              % instance)"},{"line_number":653,"context_line":""},{"line_number":654,"context_line":""},{"line_number":655,"context_line":"class RebuildInstance(policy.PolicyTargetMixin, tables.LinkAction):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_28b3774f","line":652,"range":{"start_line":652,"start_character":32,"end_line":652,"end_character":40},"updated":"2020-08-17 10:01:05.000000000","message":"ditto.","commit_id":"2e78bb22f79df4d09ee0df44b517c133e0c2cdb6"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"339317198ac76dd998690756c3995fb308410275","unresolved":false,"context_lines":[{"line_number":630,"context_line":"        except Exception:"},{"line_number":631,"context_line":"            exceptions.handle(request,"},{"line_number":632,"context_line":"                              _(\u0027Unable to confirm resize instance %s\u0027)"},{"line_number":633,"context_line":"                              % instance_id)"},{"line_number":634,"context_line":"        return shortcuts.redirect(request.get_full_path())"},{"line_number":635,"context_line":""},{"line_number":636,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_4fabec86","line":633,"range":{"start_line":633,"start_character":32,"end_line":633,"end_character":43},"updated":"2020-09-08 21:25:55.000000000","message":"We usually do not use UUID in the instance (\u003d server) page.\nCould you show \"name\" or \"name_or_id\" instead?\nYou can get a corresponding instance object by table.get_object_by_id.","commit_id":"95522e326f6e40bd75d523f19f563e09fefc8dd4"},{"author":{"_account_id":27815,"name":"Huang Cheng","email":"hcn518@gmail.com","username":"pedh"},"change_message_id":"0ab12a2b3f0d55d24ad5db0e59937389dd56cc63","unresolved":false,"context_lines":[{"line_number":630,"context_line":"        except Exception:"},{"line_number":631,"context_line":"            exceptions.handle(request,"},{"line_number":632,"context_line":"                              _(\u0027Unable to confirm resize instance %s\u0027)"},{"line_number":633,"context_line":"                              % instance_id)"},{"line_number":634,"context_line":"        return shortcuts.redirect(request.get_full_path())"},{"line_number":635,"context_line":""},{"line_number":636,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_71adef82","line":633,"range":{"start_line":633,"start_character":32,"end_line":633,"end_character":43},"in_reply_to":"9f560f44_4fabec86","updated":"2020-09-09 07:10:43.000000000","message":"Thank you for the advice, the fix was committed.","commit_id":"95522e326f6e40bd75d523f19f563e09fefc8dd4"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"17818d5fb7eb38992ff6b0481bbfe4435f276605","unresolved":false,"context_lines":[{"line_number":631,"context_line":"        except Exception:"},{"line_number":632,"context_line":"            exceptions.handle(request,"},{"line_number":633,"context_line":"                              _(\u0027Unable to confirm resize instance \"%s\"\u0027)"},{"line_number":634,"context_line":"                              % instance.name or instance.id)"},{"line_number":635,"context_line":"        return shortcuts.redirect(request.get_full_path())"},{"line_number":636,"context_line":""},{"line_number":637,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_4863d0b1","line":634,"range":{"start_line":634,"start_character":32,"end_line":634,"end_character":61},"updated":"2020-09-24 01:42:33.000000000","message":"You need parentheses around here.\nThe current code does not work when name is an empty string.\n\n \u003e\u003e\u003e \u0027[%s]\u0027 % \u0027\u0027 or \u0027id-xxx\u0027\n \u0027[]\u0027\n \u003e\u003e\u003e \u0027[%s]\u0027 % (\u0027\u0027 or \u0027id-xxx\u0027)\n \u0027[id-xxx]\u0027","commit_id":"8f8f7cd2db12fc79f61f8bf8389d33b9ec15de62"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"531c80bf1f7b4c70ab5a47f4e10af2a59020be8f","unresolved":false,"context_lines":[{"line_number":631,"context_line":"        except Exception:"},{"line_number":632,"context_line":"            exceptions.handle(request,"},{"line_number":633,"context_line":"                              _(\u0027Unable to confirm resize instance \"%s\"\u0027)"},{"line_number":634,"context_line":"                              % instance.name or instance.id)"},{"line_number":635,"context_line":"        return shortcuts.redirect(request.get_full_path())"},{"line_number":636,"context_line":""},{"line_number":637,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_15b56ced","line":634,"range":{"start_line":634,"start_character":32,"end_line":634,"end_character":61},"in_reply_to":"9f560f44_4863d0b1","updated":"2020-09-24 06:58:10.000000000","message":"Done.","commit_id":"8f8f7cd2db12fc79f61f8bf8389d33b9ec15de62"}]}
