)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"1b5e664527b7204cf6f2d55574103fb13f03116c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     lihaijing \u003clihaijing@fiberhome.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-11-29 08:29:56 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix the way to get backup metadata"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I82cb96b1f89b4009d4b3f4a0e64db076a7a04977"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df87a7cf_7dd8e9ed","line":7,"updated":"2017-12-11 09:37:06.000000000","message":"Please, add more details to the commit message","commit_id":"f9900dacafe2ea21037020fe1c8c5b86320ec021"}],"cinderclient/v3/shell.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"e0c6cf93b8c419ff73a054ad73787a75498886bd","unresolved":false,"context_lines":[{"line_number":858,"context_line":"                                   args.disk_format))"},{"line_number":859,"context_line":""},{"line_number":860,"context_line":""},{"line_number":861,"context_line":"@api_versions.wraps(\u00273.9\u0027, \u00273.43\u0027)"},{"line_number":862,"context_line":"@utils.arg(\u0027backup\u0027, metavar\u003d\u0027\u003cbackup\u003e\u0027,"},{"line_number":863,"context_line":"           help\u003d\u0027Name or ID of backup to rename.\u0027)"},{"line_number":864,"context_line":"@utils.arg(\u0027--name\u0027, nargs\u003d\u0027?\u0027, metavar\u003d\u0027\u003cname\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_2d70eb07","line":861,"range":{"start_line":861,"start_character":18,"end_line":861,"end_character":34},"updated":"2017-11-29 01:03:21.000000000","message":"It\u0027s not related to this change, but why the max version is specified here?","commit_id":"f9900dacafe2ea21037020fe1c8c5b86320ec021"},{"author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"change_message_id":"c17964c6341be009c76643303d6483bb39ce3e40","unresolved":false,"context_lines":[{"line_number":858,"context_line":"                                   args.disk_format))"},{"line_number":859,"context_line":""},{"line_number":860,"context_line":""},{"line_number":861,"context_line":"@api_versions.wraps(\u00273.9\u0027, \u00273.43\u0027)"},{"line_number":862,"context_line":"@utils.arg(\u0027backup\u0027, metavar\u003d\u0027\u003cbackup\u003e\u0027,"},{"line_number":863,"context_line":"           help\u003d\u0027Name or ID of backup to rename.\u0027)"},{"line_number":864,"context_line":"@utils.arg(\u0027--name\u0027, nargs\u003d\u0027?\u0027, metavar\u003d\u0027\u003cname\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_eef2f66f","line":861,"range":{"start_line":861,"start_character":18,"end_line":861,"end_character":34},"in_reply_to":"ff82abbf_2d70eb07","updated":"2017-11-29 06:11:20.000000000","message":"This is original master code. I think that because cinder server supportes updating backup metadata since v3.43.","commit_id":"f9900dacafe2ea21037020fe1c8c5b86320ec021"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"1b5e664527b7204cf6f2d55574103fb13f03116c","unresolved":false,"context_lines":[{"line_number":882,"context_line":"        kwargs[\u0027description\u0027] \u003d args.description"},{"line_number":883,"context_line":""},{"line_number":884,"context_line":"    if cs.api_version \u003e\u003d api_versions.APIVersion(\"3.43\"):"},{"line_number":885,"context_line":"        if args.metadata is not None:"},{"line_number":886,"context_line":"            kwargs[\u0027metadata\u0027] \u003d shell_utils.extract_metadata(args)"},{"line_number":887,"context_line":""},{"line_number":888,"context_line":"    if not kwargs:"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_fdf5d957","line":885,"updated":"2017-12-11 09:37:06.000000000","message":"It would be good to add unit tests for this change","commit_id":"f9900dacafe2ea21037020fe1c8c5b86320ec021"},{"author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"change_message_id":"7a29622b920b8bc7d846e0fca507f08fe3d6e16d","unresolved":false,"context_lines":[{"line_number":882,"context_line":"        kwargs[\u0027description\u0027] \u003d args.description"},{"line_number":883,"context_line":""},{"line_number":884,"context_line":"    if cs.api_version \u003e\u003d api_versions.APIVersion(\"3.43\"):"},{"line_number":885,"context_line":"        if args.metadata is not None:"},{"line_number":886,"context_line":"            kwargs[\u0027metadata\u0027] \u003d shell_utils.extract_metadata(args)"},{"line_number":887,"context_line":""},{"line_number":888,"context_line":"    if not kwargs:"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_af50380c","line":885,"in_reply_to":"df87a7cf_fdf5d957","updated":"2017-12-13 08:17:22.000000000","message":"Done","commit_id":"f9900dacafe2ea21037020fe1c8c5b86320ec021"}]}