)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"58aeeb6e46a1b8cbbddbaf30fcc8393d5091a02e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     xu-haiwei \u003chai-xu@xr.jp.nec.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-02 04:10:16 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Deprecate \u0027--config\u0027 option when create/update a vim"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When create/update a vim, only enable \u0027--config-file\u0027 option, will"},{"line_number":10,"context_line":"deprecate \u0027--config\u0027 option."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"9ad45d7e_39f32ed8","line":7,"range":{"start_line":7,"start_character":13,"end_line":7,"end_character":19},"updated":"2016-08-10 02:09:36.000000000","message":"Is it to deprecate or remove?","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"42dfd2f42d600a9037e536d4cb999d686925f671","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     xu-haiwei \u003chai-xu@xr.jp.nec.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-02 04:10:16 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Deprecate \u0027--config\u0027 option when create/update a vim"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When create/update a vim, only enable \u0027--config-file\u0027 option, will"},{"line_number":10,"context_line":"deprecate \u0027--config\u0027 option."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"9ad45d7e_f7b51a31","line":7,"range":{"start_line":7,"start_character":13,"end_line":7,"end_character":19},"in_reply_to":"9ad45d7e_39f32ed8","updated":"2016-08-10 07:35:54.000000000","message":"Remove","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"}],"tackerclient/common/utils.py":[{"author":{"_account_id":10487,"name":"Kanagaraj Manickam","email":"mkr1481@gmail.com","username":"kanagarajm"},"change_message_id":"440868569bd93a139e5e94769ec9573898522d91","unresolved":false,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"def get_file_path(filename):"},{"line_number":185,"context_line":"    file_path \u003d os.path.abspath(os.path.join(os.path.dirname(__file__),"},{"line_number":186,"context_line":"                                             \u0027../\u0027 + str(filename)))"},{"line_number":187,"context_line":"    return file_path"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_a99dd167","line":186,"updated":"2016-06-24 04:08:47.000000000","message":"kindly use string formatting instead of add operator","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"42b3094c49030e9d64d2c48ba89b4471ce6af7db","unresolved":false,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"def get_file_path(filename):"},{"line_number":185,"context_line":"    file_path \u003d os.path.abspath(os.path.join(os.path.dirname(__file__),"},{"line_number":186,"context_line":"                                             \u0027../\u0027 + str(filename)))"},{"line_number":187,"context_line":"    return file_path"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_f614c94d","line":186,"in_reply_to":"3aaa91ec_a99dd167","updated":"2016-06-24 09:03:42.000000000","message":"Done","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"}],"tackerclient/tacker/v1_0/nfvo/vim.py":[{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"1386f1f69b95988382a547daefab75694427cb2b","unresolved":false,"context_lines":[{"line_number":78,"context_line":"                                                           \u0027specified\u0027,"},{"line_number":79,"context_line":"                                                   status_code\u003d404)"},{"line_number":80,"context_line":"        except AttributeError:"},{"line_number":81,"context_line":"            message \u003d \u0027The imput config data is not in correct format.\u0027"},{"line_number":82,"context_line":"            raise exceptions.TackerClientException(message\u003dmessage,"},{"line_number":83,"context_line":"                                                   status_code\u003d400)"},{"line_number":84,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_6400dd02","line":81,"range":{"start_line":81,"start_character":27,"end_line":81,"end_character":32},"updated":"2016-06-02 05:27:18.000000000","message":"imput --\u003e input","commit_id":"70ba30fe8af9a5682355325b20887804c9e210ee"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"fb745900712047d60015521d84f1822fe1651fe0","unresolved":false,"context_lines":[{"line_number":78,"context_line":"                                                           \u0027specified\u0027,"},{"line_number":79,"context_line":"                                                   status_code\u003d404)"},{"line_number":80,"context_line":"        except AttributeError:"},{"line_number":81,"context_line":"            message \u003d \u0027The imput config data is not in correct format.\u0027"},{"line_number":82,"context_line":"            raise exceptions.TackerClientException(message\u003dmessage,"},{"line_number":83,"context_line":"                                                   status_code\u003d400)"},{"line_number":84,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_22708571","line":81,"range":{"start_line":81,"start_character":27,"end_line":81,"end_character":32},"in_reply_to":"9abb7d3a_6400dd02","updated":"2016-06-02 06:02:21.000000000","message":"Done","commit_id":"70ba30fe8af9a5682355325b20887804c9e210ee"},{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"1386f1f69b95988382a547daefab75694427cb2b","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        except AttributeError:"},{"line_number":81,"context_line":"            message \u003d \u0027The imput config data is not in correct format.\u0027"},{"line_number":82,"context_line":"            raise exceptions.TackerClientException(message\u003dmessage,"},{"line_number":83,"context_line":"                                                   status_code\u003d400)"},{"line_number":84,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"},{"line_number":85,"context_line":"        vim_obj[\u0027type\u0027] \u003d config_param.pop(\u0027type\u0027, \u0027openstack\u0027)"},{"line_number":86,"context_line":"        vim_utils.args2body_vim(config_param, vim_obj)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_27e53708","line":83,"updated":"2016-06-02 05:27:18.000000000","message":"Instead of handling \"AttributeError\", we should remove support for \"--config\" (i.e. line #55 #56)\n\n@Sripriya: Please suggest your opinion.","commit_id":"70ba30fe8af9a5682355325b20887804c9e210ee"},{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"9f1bc5a5b58ad9fbad324a26b772c0cb0c1425bd","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        except AttributeError:"},{"line_number":81,"context_line":"            message \u003d \u0027The imput config data is not in correct format.\u0027"},{"line_number":82,"context_line":"            raise exceptions.TackerClientException(message\u003dmessage,"},{"line_number":83,"context_line":"                                                   status_code\u003d400)"},{"line_number":84,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"},{"line_number":85,"context_line":"        vim_obj[\u0027type\u0027] \u003d config_param.pop(\u0027type\u0027, \u0027openstack\u0027)"},{"line_number":86,"context_line":"        vim_utils.args2body_vim(config_param, vim_obj)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_9e3c3094","line":83,"in_reply_to":"7aa08908_e2eb1c11","updated":"2016-06-09 09:00:15.000000000","message":"Sorry, I mixed it with VNF\u0027s congifs","commit_id":"70ba30fe8af9a5682355325b20887804c9e210ee"},{"author":{"_account_id":13485,"name":"Sripriya Seetharam","email":"sseetha@brocade.com","username":"Sripriya"},"change_message_id":"adbf809a1dcc8309a051cb00df0a7809db473242","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        except AttributeError:"},{"line_number":81,"context_line":"            message \u003d \u0027The imput config data is not in correct format.\u0027"},{"line_number":82,"context_line":"            raise exceptions.TackerClientException(message\u003dmessage,"},{"line_number":83,"context_line":"                                                   status_code\u003d400)"},{"line_number":84,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"},{"line_number":85,"context_line":"        vim_obj[\u0027type\u0027] \u003d config_param.pop(\u0027type\u0027, \u0027openstack\u0027)"},{"line_number":86,"context_line":"        vim_utils.args2body_vim(config_param, vim_obj)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_e2eb1c11","line":83,"in_reply_to":"7aa08908_ea497561","updated":"2016-06-09 05:56:59.000000000","message":"no, for VIM in Horizon, we provide input through form data. We do not support direct input.","commit_id":"70ba30fe8af9a5682355325b20887804c9e210ee"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"fb745900712047d60015521d84f1822fe1651fe0","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        except AttributeError:"},{"line_number":81,"context_line":"            message \u003d \u0027The imput config data is not in correct format.\u0027"},{"line_number":82,"context_line":"            raise exceptions.TackerClientException(message\u003dmessage,"},{"line_number":83,"context_line":"                                                   status_code\u003d400)"},{"line_number":84,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"},{"line_number":85,"context_line":"        vim_obj[\u0027type\u0027] \u003d config_param.pop(\u0027type\u0027, \u0027openstack\u0027)"},{"line_number":86,"context_line":"        vim_utils.args2body_vim(config_param, vim_obj)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_e2103d02","line":83,"in_reply_to":"9abb7d3a_27e53708","updated":"2016-06-02 06:02:21.000000000","message":"Can you explain your reason of removing support for \u0027--config\u0027 option.","commit_id":"70ba30fe8af9a5682355325b20887804c9e210ee"},{"author":{"_account_id":13485,"name":"Sripriya Seetharam","email":"sseetha@brocade.com","username":"Sripriya"},"change_message_id":"adbf809a1dcc8309a051cb00df0a7809db473242","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        except AttributeError:"},{"line_number":81,"context_line":"            message \u003d \u0027The imput config data is not in correct format.\u0027"},{"line_number":82,"context_line":"            raise exceptions.TackerClientException(message\u003dmessage,"},{"line_number":83,"context_line":"                                                   status_code\u003d400)"},{"line_number":84,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"},{"line_number":85,"context_line":"        vim_obj[\u0027type\u0027] \u003d config_param.pop(\u0027type\u0027, \u0027openstack\u0027)"},{"line_number":86,"context_line":"        vim_utils.args2body_vim(config_param, vim_obj)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_828a1862","line":83,"in_reply_to":"9abb7d3a_27e53708","updated":"2016-06-09 05:56:59.000000000","message":"Yes, we decided to remove direct input option from CLI and only support input through a --config-file option.\n\n@Sridhar updated the bug with these comments. See [1]\n\nAnd I think it is better to update the bug description to incorporate these comments.\n\n[1] https://bugs.launchpad.net/tacker/+bug/1587216","commit_id":"70ba30fe8af9a5682355325b20887804c9e210ee"},{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"4193eb7f207a9a7b0de460c3d1fdbccc3322c04f","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        except AttributeError:"},{"line_number":81,"context_line":"            message \u003d \u0027The imput config data is not in correct format.\u0027"},{"line_number":82,"context_line":"            raise exceptions.TackerClientException(message\u003dmessage,"},{"line_number":83,"context_line":"                                                   status_code\u003d400)"},{"line_number":84,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"},{"line_number":85,"context_line":"        vim_obj[\u0027type\u0027] \u003d config_param.pop(\u0027type\u0027, \u0027openstack\u0027)"},{"line_number":86,"context_line":"        vim_utils.args2body_vim(config_param, vim_obj)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_ea497561","line":83,"in_reply_to":"9abb7d3a_e2103d02","updated":"2016-06-08 09:08:06.000000000","message":"In case of horizon, we have two input methods(1: direct input, 2: config file). But in case of cli, we can input through config file only. So we should not support --config at cli level.","commit_id":"70ba30fe8af9a5682355325b20887804c9e210ee"},{"author":{"_account_id":10487,"name":"Kanagaraj Manickam","email":"mkr1481@gmail.com","username":"kanagarajm"},"change_message_id":"ab628c1d4e27d50873ace49b9d8f96aa1049e20c","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        except KeyError:"},{"line_number":72,"context_line":"            raise exceptions.TackerClientException(message\u003d\u0027Auth URL must be \u0027"},{"line_number":73,"context_line":"                                                           \u0027specified\u0027,"},{"line_number":74,"context_line":"                                                   status_code\u003d400)"},{"line_number":75,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"},{"line_number":76,"context_line":"        vim_obj[\u0027type\u0027] \u003d config_param.pop(\u0027type\u0027, \u0027openstack\u0027)"},{"line_number":77,"context_line":"        vim_utils.args2body_vim(config_param, vim_obj)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7aa08908_4fd9df21","line":74,"updated":"2016-06-13 09:37:55.000000000","message":"Kindly note that, this kind of status code should be returned from the tacker server, client should not make any such code setting, because status code represents the http status.","commit_id":"8dcc89515058f90612b775fa6655f50dd273500a"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"3a7ffdd005b0e2ce73c1ba9ec569b78c8a5267e8","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        except KeyError:"},{"line_number":72,"context_line":"            raise exceptions.TackerClientException(message\u003d\u0027Auth URL must be \u0027"},{"line_number":73,"context_line":"                                                           \u0027specified\u0027,"},{"line_number":74,"context_line":"                                                   status_code\u003d400)"},{"line_number":75,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"},{"line_number":76,"context_line":"        vim_obj[\u0027type\u0027] \u003d config_param.pop(\u0027type\u0027, \u0027openstack\u0027)"},{"line_number":77,"context_line":"        vim_utils.args2body_vim(config_param, vim_obj)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7aa08908_7592084d","line":74,"in_reply_to":"7aa08908_4fd9df21","updated":"2016-06-16 04:27:20.000000000","message":"Your comment makes sense to me. But because client side also verify the input info, it doesn\u0027t matter returning a status code to tell user what is wrong. Other project like novaclient also has the similar implementation.","commit_id":"8dcc89515058f90612b775fa6655f50dd273500a"},{"author":{"_account_id":10487,"name":"Kanagaraj Manickam","email":"mkr1481@gmail.com","username":"kanagarajm"},"change_message_id":"d5e90903105e787edeb5473e40d9b4579145e8ac","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        except KeyError:"},{"line_number":72,"context_line":"            raise exceptions.TackerClientException(message\u003d\u0027Auth URL must be \u0027"},{"line_number":73,"context_line":"                                                           \u0027specified\u0027,"},{"line_number":74,"context_line":"                                                   status_code\u003d400)"},{"line_number":75,"context_line":"        vim_obj[\u0027auth_url\u0027] \u003d utils.validate_url(auth_url).geturl()"},{"line_number":76,"context_line":"        vim_obj[\u0027type\u0027] \u003d config_param.pop(\u0027type\u0027, \u0027openstack\u0027)"},{"line_number":77,"context_line":"        vim_utils.args2body_vim(config_param, vim_obj)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7aa08908_ff4df6c4","line":74,"in_reply_to":"7aa08908_7592084d","updated":"2016-06-17 05:36:11.000000000","message":"ok","commit_id":"8dcc89515058f90612b775fa6655f50dd273500a"},{"author":{"_account_id":13485,"name":"Sripriya Seetharam","email":"sseetha@brocade.com","username":"Sripriya"},"change_message_id":"a3327063d9fbc0e400e3085e2f354b01af04533c","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":52,"context_line":"        group \u003d parser.add_mutually_exclusive_group(required\u003dTrue)"},{"line_number":53,"context_line":"        group.add_argument(\u0027--config-file\u0027, help\u003d\u0027specify VIM specific \u0027"},{"line_number":54,"context_line":"                                                 \u0027config parameters in a file\u0027)"},{"line_number":55,"context_line":"        parser.add_argument("},{"line_number":56,"context_line":"            \u0027--name\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a9d85d2_255fb442","line":53,"range":{"start_line":53,"start_character":8,"end_line":53,"end_character":13},"updated":"2016-06-20 18:35:15.000000000","message":"now that we removed the direct config input, --config-file is now a mandatory argument instead of optional argument even though it is listed under optional arguments when --help is typed. also we don\u0027t need the group anymore. we can just directly add the argument and set required\u003dtrue","commit_id":"8ce69fef41e4c56853eb3f998cc87bc0c39846ca"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"6f3c3c0dd0b2a594c97ce991efdb8e38b86ad01a","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":52,"context_line":"        group \u003d parser.add_mutually_exclusive_group(required\u003dTrue)"},{"line_number":53,"context_line":"        group.add_argument(\u0027--config-file\u0027, help\u003d\u0027specify VIM specific \u0027"},{"line_number":54,"context_line":"                                                 \u0027config parameters in a file\u0027)"},{"line_number":55,"context_line":"        parser.add_argument("},{"line_number":56,"context_line":"            \u0027--name\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a9d85d2_f7543955","line":53,"range":{"start_line":53,"start_character":8,"end_line":53,"end_character":13},"in_reply_to":"5a9d85d2_255fb442","updated":"2016-06-20 23:41:38.000000000","message":"Done","commit_id":"8ce69fef41e4c56853eb3f998cc87bc0c39846ca"},{"author":{"_account_id":13485,"name":"Sripriya Seetharam","email":"sseetha@brocade.com","username":"Sripriya"},"change_message_id":"0f70797b786ab34c18d4ace8eb4878ea8250c403","unresolved":false,"context_lines":[{"line_number":102,"context_line":"            with open(parsed_args.config_file) as f:"},{"line_number":103,"context_line":"                config_yaml \u003d f.read()"},{"line_number":104,"context_line":"            config_param \u003d yaml.load(config_yaml)"},{"line_number":105,"context_line":"        if parsed_args.config:"},{"line_number":106,"context_line":"            parsed_args.config \u003d parsed_args.config.decode(\u0027unicode_escape\u0027)"},{"line_number":107,"context_line":"            config_param \u003d yaml.load(parsed_args.config)"},{"line_number":108,"context_line":"        if \u0027auth_url\u0027 in config_param:"},{"line_number":109,"context_line":"            raise exceptions.TackerClientException(message\u003d\u0027Auth URL cannot \u0027"},{"line_number":110,"context_line":"                                                           \u0027be updated\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"3aaa91ec_ee240661","line":107,"range":{"start_line":105,"start_character":8,"end_line":107,"end_character":56},"updated":"2016-06-23 22:56:40.000000000","message":"this needs to be removed as well. sorry missed this earlier.","commit_id":"c9ac5d681906913fef9b19b1649dfd51cdfcede7"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"a1d78840b48344c4ffc69c8a6864eebc497eb7da","unresolved":false,"context_lines":[{"line_number":102,"context_line":"            with open(parsed_args.config_file) as f:"},{"line_number":103,"context_line":"                config_yaml \u003d f.read()"},{"line_number":104,"context_line":"            config_param \u003d yaml.load(config_yaml)"},{"line_number":105,"context_line":"        if parsed_args.config:"},{"line_number":106,"context_line":"            parsed_args.config \u003d parsed_args.config.decode(\u0027unicode_escape\u0027)"},{"line_number":107,"context_line":"            config_param \u003d yaml.load(parsed_args.config)"},{"line_number":108,"context_line":"        if \u0027auth_url\u0027 in config_param:"},{"line_number":109,"context_line":"            raise exceptions.TackerClientException(message\u003d\u0027Auth URL cannot \u0027"},{"line_number":110,"context_line":"                                                           \u0027be updated\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"3aaa91ec_a97b18f4","line":107,"range":{"start_line":105,"start_character":8,"end_line":107,"end_character":56},"in_reply_to":"3aaa91ec_ee240661","updated":"2016-06-23 23:46:08.000000000","message":"Done","commit_id":"c9ac5d681906913fef9b19b1649dfd51cdfcede7"},{"author":{"_account_id":10487,"name":"Kanagaraj Manickam","email":"mkr1481@gmail.com","username":"kanagarajm"},"change_message_id":"440868569bd93a139e5e94769ec9573898522d91","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        parser.add_argument("},{"line_number":53,"context_line":"            \u0027--config-file\u0027,"},{"line_number":54,"context_line":"            required\u003dTrue,"},{"line_number":55,"context_line":"            help\u003d\u0027specify VIM specific config parameters in a file\u0027)"},{"line_number":56,"context_line":"        parser.add_argument("},{"line_number":57,"context_line":"            \u0027--name\u0027,"},{"line_number":58,"context_line":"            help\u003d\u0027Set a name for the vim\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_a9ab1174","line":55,"updated":"2016-06-24 04:08:47.000000000","message":"kindly rephrase to \u0027specify path to VIM config file\u0027 or something similar... here the word specific and specify are kind of repetitive in meaning","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"42b3094c49030e9d64d2c48ba89b4471ce6af7db","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        parser.add_argument("},{"line_number":53,"context_line":"            \u0027--config-file\u0027,"},{"line_number":54,"context_line":"            required\u003dTrue,"},{"line_number":55,"context_line":"            help\u003d\u0027specify VIM specific config parameters in a file\u0027)"},{"line_number":56,"context_line":"        parser.add_argument("},{"line_number":57,"context_line":"            \u0027--name\u0027,"},{"line_number":58,"context_line":"            help\u003d\u0027Set a name for the vim\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_111ebf5e","line":55,"in_reply_to":"3aaa91ec_a9ab1174","updated":"2016-06-24 09:03:42.000000000","message":"Done","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":13485,"name":"Sripriya Seetharam","email":"sseetha@brocade.com","username":"Sripriya"},"change_message_id":"7a62779209c8bdc7ac2b6df0896e33d1ab376d68","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    resource \u003d _VIM"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":90,"context_line":"        group \u003d parser.add_mutually_exclusive_group(required\u003dTrue)"},{"line_number":91,"context_line":"        group.add_argument("},{"line_number":92,"context_line":"            \u0027--config-file\u0027,"},{"line_number":93,"context_line":"            help\u003d\u0027specify VIM specific config parameters in a file\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_c9bc9dbd","line":90,"range":{"start_line":90,"start_character":8,"end_line":90,"end_character":13},"updated":"2016-06-24 04:08:08.000000000","message":"group needs to be removed  as well","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"42b3094c49030e9d64d2c48ba89b4471ce6af7db","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    resource \u003d _VIM"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":90,"context_line":"        group \u003d parser.add_mutually_exclusive_group(required\u003dTrue)"},{"line_number":91,"context_line":"        group.add_argument("},{"line_number":92,"context_line":"            \u0027--config-file\u0027,"},{"line_number":93,"context_line":"            help\u003d\u0027specify VIM specific config parameters in a file\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_9132aff0","line":90,"range":{"start_line":90,"start_character":8,"end_line":90,"end_character":13},"in_reply_to":"3aaa91ec_c9bc9dbd","updated":"2016-06-24 09:03:42.000000000","message":"Done","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":10487,"name":"Kanagaraj Manickam","email":"mkr1481@gmail.com","username":"kanagarajm"},"change_message_id":"440868569bd93a139e5e94769ec9573898522d91","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        group \u003d parser.add_mutually_exclusive_group(required\u003dTrue)"},{"line_number":91,"context_line":"        group.add_argument("},{"line_number":92,"context_line":"            \u0027--config-file\u0027,"},{"line_number":93,"context_line":"            help\u003d\u0027specify VIM specific config parameters in a file\u0027)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def args2body(self, parsed_args):"},{"line_number":96,"context_line":"        body \u003d {self.resource: {}}"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_697e491a","line":93,"updated":"2016-06-24 04:08:47.000000000","message":"why do you want to keep the groups while the config option is removed now. you done the same in create cli","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"42b3094c49030e9d64d2c48ba89b4471ce6af7db","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        group \u003d parser.add_mutually_exclusive_group(required\u003dTrue)"},{"line_number":91,"context_line":"        group.add_argument("},{"line_number":92,"context_line":"            \u0027--config-file\u0027,"},{"line_number":93,"context_line":"            help\u003d\u0027specify VIM specific config parameters in a file\u0027)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def args2body(self, parsed_args):"},{"line_number":96,"context_line":"        body \u003d {self.resource: {}}"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_31393bd8","line":93,"in_reply_to":"3aaa91ec_697e491a","updated":"2016-06-24 09:03:42.000000000","message":"Done","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":10487,"name":"Kanagaraj Manickam","email":"mkr1481@gmail.com","username":"kanagarajm"},"change_message_id":"4562acf7920baf245b2a11e17a8f1ccda2b31faa","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        parser.add_argument("},{"line_number":53,"context_line":"            \u0027--config-file\u0027,"},{"line_number":54,"context_line":"            required\u003dTrue,"},{"line_number":55,"context_line":"            help\u003d\u0027specify config parameters in a file for VIM\u0027)"},{"line_number":56,"context_line":"        parser.add_argument("},{"line_number":57,"context_line":"            \u0027--name\u0027,"},{"line_number":58,"context_line":"            help\u003d\u0027Set a name for the VIM\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"3aaa91ec_2c0ba42b","line":55,"range":{"start_line":55,"start_character":18,"end_line":55,"end_character":19},"updated":"2016-06-24 09:43:28.000000000","message":"Capital S","commit_id":"1427119d72a36483c7107637298ad96d8b08bb4f"},{"author":{"_account_id":13380,"name":"Sridhar Ramaswamy","email":"srics.r@gmail.com","username":"srics-r"},"change_message_id":"e9c4d8079937a4f6cf4d7e9d6d0c5353a97269fa","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--config-file\u0027,"},{"line_number":49,"context_line":"            required\u003dTrue,"},{"line_number":50,"context_line":"            help\u003d\u0027specify config parameters in a file for VIM\u0027)"},{"line_number":51,"context_line":"        parser.add_argument("},{"line_number":52,"context_line":"            \u0027--name\u0027,"},{"line_number":53,"context_line":"            help\u003d\u0027Set a name for the VIM\u0027)"}],"source_content_type":"text/x-python","patch_set":14,"id":"9ad45d7e_2d5770cf","line":50,"range":{"start_line":50,"start_character":18,"end_line":50,"end_character":19},"updated":"2016-08-10 00:27:28.000000000","message":"Please use uppercase to start your help string.","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"42dfd2f42d600a9037e536d4cb999d686925f671","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--config-file\u0027,"},{"line_number":49,"context_line":"            required\u003dTrue,"},{"line_number":50,"context_line":"            help\u003d\u0027specify config parameters in a file for VIM\u0027)"},{"line_number":51,"context_line":"        parser.add_argument("},{"line_number":52,"context_line":"            \u0027--name\u0027,"},{"line_number":53,"context_line":"            help\u003d\u0027Set a name for the VIM\u0027)"}],"source_content_type":"text/x-python","patch_set":14,"id":"9ad45d7e_d7de166c","line":50,"range":{"start_line":50,"start_character":18,"end_line":50,"end_character":19},"in_reply_to":"9ad45d7e_2d5770cf","updated":"2016-08-10 07:35:54.000000000","message":"Done","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"},{"author":{"_account_id":20986,"name":"Manikantha Srinivas Tadi","email":"manikantha.tadi@gmail.com","username":"mtadi"},"change_message_id":"d3a7883203f460191d06d320b9c27a58badc913d","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--config-file\u0027,"},{"line_number":49,"context_line":"            required\u003dTrue,"},{"line_number":50,"context_line":"            help\u003d\u0027specify config parameters in a file for VIM\u0027)"},{"line_number":51,"context_line":"        parser.add_argument("},{"line_number":52,"context_line":"            \u0027--name\u0027,"},{"line_number":53,"context_line":"            help\u003d\u0027Set a name for the VIM\u0027)"}],"source_content_type":"text/x-python","patch_set":14,"id":"9ad45d7e_6c9de158","line":50,"range":{"start_line":50,"start_character":18,"end_line":50,"end_character":19},"in_reply_to":"9ad45d7e_2d5770cf","updated":"2016-08-10 06:00:39.000000000","message":"The earlier help text \"Specify VIM specific config parameters in a file\" looks appropriate, Why should we rephrase it ?","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"42dfd2f42d600a9037e536d4cb999d686925f671","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--config-file\u0027,"},{"line_number":49,"context_line":"            required\u003dTrue,"},{"line_number":50,"context_line":"            help\u003d\u0027specify config parameters in a file for VIM\u0027)"},{"line_number":51,"context_line":"        parser.add_argument("},{"line_number":52,"context_line":"            \u0027--name\u0027,"},{"line_number":53,"context_line":"            help\u003d\u0027Set a name for the VIM\u0027)"}],"source_content_type":"text/x-python","patch_set":14,"id":"9ad45d7e_77f76af5","line":50,"range":{"start_line":50,"start_character":18,"end_line":50,"end_character":19},"in_reply_to":"9ad45d7e_6c9de158","updated":"2016-08-10 07:35:54.000000000","message":"Done","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"},{"author":{"_account_id":13380,"name":"Sridhar Ramaswamy","email":"srics.r@gmail.com","username":"srics-r"},"change_message_id":"ef7fe12669a47f9217f6760058fb26b8bec18213","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--config-file\u0027,"},{"line_number":49,"context_line":"            required\u003dTrue,"},{"line_number":50,"context_line":"            help\u003d\u0027specify config parameters in a file for VIM\u0027)"},{"line_number":51,"context_line":"        parser.add_argument("},{"line_number":52,"context_line":"            \u0027--name\u0027,"},{"line_number":53,"context_line":"            help\u003d\u0027Set a name for the VIM\u0027)"}],"source_content_type":"text/x-python","patch_set":14,"id":"9ad45d7e_0be694af","line":50,"range":{"start_line":50,"start_character":18,"end_line":50,"end_character":19},"in_reply_to":"9ad45d7e_6c9de158","updated":"2016-08-11 01:19:41.000000000","message":"Nothing wrong other than gongysh had a suggestion in the earlier patchset to change it (which reads better IMO) and xu-haiwei accepted the change.","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"}],"tackerclient/tests/unit/test_utils.py":[{"author":{"_account_id":13485,"name":"Sripriya Seetharam","email":"sseetha@brocade.com","username":"Sripriya"},"change_message_id":"a3327063d9fbc0e400e3085e2f354b01af04533c","unresolved":false,"context_lines":[{"line_number":198,"context_line":"            utils.get_client_class, \u0027image\u0027, \u00272\u0027, {\u0027image\u0027: \u00272\u0027})"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"def get_file_path(filename):"},{"line_number":202,"context_line":"    file_path \u003d os.path.abspath(os.path.join(os.path.dirname(__file__),"},{"line_number":203,"context_line":"                                             \u0027vm/samples/\u0027 + str(filename)))"},{"line_number":204,"context_line":"    return file_path"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a9d85d2_452da0a6","line":204,"range":{"start_line":201,"start_character":0,"end_line":204,"end_character":20},"updated":"2016-06-20 18:35:15.000000000","message":"this needs to be added in [1] . Not in test file.\n\n\n[1]https://github.com/openstack/python-tackerclient/blob/master/tackerclient/common/utils.py","commit_id":"8ce69fef41e4c56853eb3f998cc87bc0c39846ca"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"6f3c3c0dd0b2a594c97ce991efdb8e38b86ad01a","unresolved":false,"context_lines":[{"line_number":198,"context_line":"            utils.get_client_class, \u0027image\u0027, \u00272\u0027, {\u0027image\u0027: \u00272\u0027})"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"def get_file_path(filename):"},{"line_number":202,"context_line":"    file_path \u003d os.path.abspath(os.path.join(os.path.dirname(__file__),"},{"line_number":203,"context_line":"                                             \u0027vm/samples/\u0027 + str(filename)))"},{"line_number":204,"context_line":"    return file_path"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a9d85d2_174c65c6","line":204,"range":{"start_line":201,"start_character":0,"end_line":204,"end_character":20},"in_reply_to":"5a9d85d2_452da0a6","updated":"2016-06-20 23:41:38.000000000","message":"Done","commit_id":"8ce69fef41e4c56853eb3f998cc87bc0c39846ca"}],"tackerclient/tests/unit/vm/samples/vim_config_no_auth_url.yaml":[{"author":{"_account_id":10487,"name":"Kanagaraj Manickam","email":"mkr1481@gmail.com","username":"kanagarajm"},"change_message_id":"440868569bd93a139e5e94769ec9573898522d91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3aaa91ec_e9deb90f","updated":"2016-06-24 04:08:47.000000000","message":"this might not be required, instead use the other file and remove the auth url from the config list in the unit testing.","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"8e182d7567c9ebb7254a98f8594c0af60683171a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3aaa91ec_2c88c4c4","in_reply_to":"3aaa91ec_e9deb90f","updated":"2016-06-24 09:36:54.000000000","message":"I think it easy to understand by remaining this file.","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"}],"tackerclient/tests/unit/vm/test_cli10_vim.py":[{"author":{"_account_id":13485,"name":"Sripriya Seetharam","email":"sseetha@brocade.com","username":"Sripriya"},"change_message_id":"0f70797b786ab34c18d4ace8eb4878ea8250c403","unresolved":false,"context_lines":[{"line_number":153,"context_line":"        update_config \u003d {\u0027username\u0027: \u0027xyz\u0027, \u0027password\u0027: \u002712345\u0027,"},{"line_number":154,"context_line":"                         \u0027project_name\u0027: \u0027abc\u0027}"},{"line_number":155,"context_line":"        my_id \u003d \u0027my-id\u0027"},{"line_number":156,"context_line":"        key \u003d \u0027config\u0027"},{"line_number":157,"context_line":"        value \u003d str(update_config)"},{"line_number":158,"context_line":"        extra_fields \u003d {\u0027vim_project\u0027: self.vim_project, \u0027auth_cred\u0027:"},{"line_number":159,"context_line":"                        self.auth_cred}"}],"source_content_type":"text/x-python","patch_set":7,"id":"3aaa91ec_2ed1ee23","line":156,"range":{"start_line":156,"start_character":15,"end_line":156,"end_character":21},"updated":"2016-06-23 22:56:40.000000000","message":"should be updated to config-file","commit_id":"c9ac5d681906913fef9b19b1649dfd51cdfcede7"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"a1d78840b48344c4ffc69c8a6864eebc497eb7da","unresolved":false,"context_lines":[{"line_number":153,"context_line":"        update_config \u003d {\u0027username\u0027: \u0027xyz\u0027, \u0027password\u0027: \u002712345\u0027,"},{"line_number":154,"context_line":"                         \u0027project_name\u0027: \u0027abc\u0027}"},{"line_number":155,"context_line":"        my_id \u003d \u0027my-id\u0027"},{"line_number":156,"context_line":"        key \u003d \u0027config\u0027"},{"line_number":157,"context_line":"        value \u003d str(update_config)"},{"line_number":158,"context_line":"        extra_fields \u003d {\u0027vim_project\u0027: self.vim_project, \u0027auth_cred\u0027:"},{"line_number":159,"context_line":"                        self.auth_cred}"}],"source_content_type":"text/x-python","patch_set":7,"id":"3aaa91ec_2970c80a","line":156,"range":{"start_line":156,"start_character":15,"end_line":156,"end_character":21},"in_reply_to":"3aaa91ec_2ed1ee23","updated":"2016-06-23 23:46:08.000000000","message":"Done","commit_id":"c9ac5d681906913fef9b19b1649dfd51cdfcede7"},{"author":{"_account_id":10487,"name":"Kanagaraj Manickam","email":"mkr1481@gmail.com","username":"kanagarajm"},"change_message_id":"440868569bd93a139e5e94769ec9573898522d91","unresolved":false,"context_lines":[{"line_number":71,"context_line":"            \u0027--name\u0027, name,"},{"line_number":72,"context_line":"            \u0027--description\u0027, description]"},{"line_number":73,"context_line":"        position_names \u003d [\u0027auth_cred\u0027, \u0027vim_project\u0027, \u0027auth_url\u0027]"},{"line_number":74,"context_line":"        position_values \u003d [self.auth_cred, self.vim_project, self.auth_url]"},{"line_number":75,"context_line":"        extra_body \u003d {\u0027type\u0027: \u0027openstack\u0027, \u0027name\u0027: name, \u0027description\u0027:"},{"line_number":76,"context_line":"                      description}"},{"line_number":77,"context_line":"        message \u003d \u0027Auth URL must be specified\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_e905197a","line":74,"updated":"2016-06-24 04:08:47.000000000","message":"here you could skip the auth url instead","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"42b3094c49030e9d64d2c48ba89b4471ce6af7db","unresolved":false,"context_lines":[{"line_number":71,"context_line":"            \u0027--name\u0027, name,"},{"line_number":72,"context_line":"            \u0027--description\u0027, description]"},{"line_number":73,"context_line":"        position_names \u003d [\u0027auth_cred\u0027, \u0027vim_project\u0027, \u0027auth_url\u0027]"},{"line_number":74,"context_line":"        position_values \u003d [self.auth_cred, self.vim_project, self.auth_url]"},{"line_number":75,"context_line":"        extra_body \u003d {\u0027type\u0027: \u0027openstack\u0027, \u0027name\u0027: name, \u0027description\u0027:"},{"line_number":76,"context_line":"                      description}"},{"line_number":77,"context_line":"        message \u003d \u0027Auth URL must be specified\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_d19c37c2","line":74,"in_reply_to":"3aaa91ec_e905197a","updated":"2016-06-24 09:03:42.000000000","message":"Done","commit_id":"af1cd0837a57d573d245b8e723750c01e74ef99f"},{"author":{"_account_id":13380,"name":"Sridhar Ramaswamy","email":"srics.r@gmail.com","username":"srics-r"},"change_message_id":"e9c4d8079937a4f6cf4d7e9d6d0c5353a97269fa","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        my_id \u003d \u0027my-id\u0027"},{"line_number":48,"context_line":"        name \u003d \u0027test_vim\u0027"},{"line_number":49,"context_line":"        description \u003d \u0027Vim Description\u0027"},{"line_number":50,"context_line":"        vim_config \u003d utils.get_file_path("},{"line_number":51,"context_line":"            \u0027tests/unit/vm/vim_config.yaml\u0027)"},{"line_number":52,"context_line":"        file_content \u003d [\"username: \u0027xyz\u0027\\n\", \"password: \u002712345\u0027\\n\","},{"line_number":53,"context_line":"                        \"project_name: \u0027abc\u0027\\n\","},{"line_number":54,"context_line":"                        \"auth_url: \u0027http://1.2.3.4:5000\u0027\\n\","}],"source_content_type":"text/x-python","patch_set":14,"id":"9ad45d7e_6dbb28e6","line":51,"range":{"start_line":50,"start_character":0,"end_line":51,"end_character":44},"updated":"2016-08-10 00:27:28.000000000","message":"I\u0027m worried about this solution of having these files written to the filesystem... particularly in this dir tests/unit/vm. These files could be left over in a git backed path on a test failure and could cause side effects in other test cases. \n\nInstead can you consider looking into six.StringIO [1] which gives an file like representation for a string buffer ?\n\n[1] https://pythonhosted.org/six/#six.StringIO","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"},{"author":{"_account_id":20986,"name":"Manikantha Srinivas Tadi","email":"manikantha.tadi@gmail.com","username":"mtadi"},"change_message_id":"d3a7883203f460191d06d320b9c27a58badc913d","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        my_id \u003d \u0027my-id\u0027"},{"line_number":48,"context_line":"        name \u003d \u0027test_vim\u0027"},{"line_number":49,"context_line":"        description \u003d \u0027Vim Description\u0027"},{"line_number":50,"context_line":"        vim_config \u003d utils.get_file_path("},{"line_number":51,"context_line":"            \u0027tests/unit/vm/vim_config.yaml\u0027)"},{"line_number":52,"context_line":"        file_content \u003d [\"username: \u0027xyz\u0027\\n\", \"password: \u002712345\u0027\\n\","},{"line_number":53,"context_line":"                        \"project_name: \u0027abc\u0027\\n\","},{"line_number":54,"context_line":"                        \"auth_url: \u0027http://1.2.3.4:5000\u0027\\n\","}],"source_content_type":"text/x-python","patch_set":14,"id":"9ad45d7e_6cb4c1d0","line":51,"range":{"start_line":50,"start_character":0,"end_line":51,"end_character":44},"in_reply_to":"9ad45d7e_6dbb28e6","updated":"2016-08-10 06:00:39.000000000","message":"+1","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"757ef77e9faa7af1b81360a98089f546ffc7eb0c","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        my_id \u003d \u0027my-id\u0027"},{"line_number":48,"context_line":"        name \u003d \u0027test_vim\u0027"},{"line_number":49,"context_line":"        description \u003d \u0027Vim Description\u0027"},{"line_number":50,"context_line":"        vim_config \u003d utils.get_file_path("},{"line_number":51,"context_line":"            \u0027tests/unit/vm/vim_config.yaml\u0027)"},{"line_number":52,"context_line":"        file_content \u003d [\"username: \u0027xyz\u0027\\n\", \"password: \u002712345\u0027\\n\","},{"line_number":53,"context_line":"                        \"project_name: \u0027abc\u0027\\n\","},{"line_number":54,"context_line":"                        \"auth_url: \u0027http://1.2.3.4:5000\u0027\\n\","}],"source_content_type":"text/x-python","patch_set":14,"id":"9ad45d7e_99133a0d","line":51,"range":{"start_line":50,"start_character":0,"end_line":51,"end_character":44},"in_reply_to":"9ad45d7e_6dbb28e6","updated":"2016-08-10 02:06:30.000000000","message":"or a temp file","commit_id":"a5fd12d0aab962e3eee0cae5a05f6bf9b540d518"}]}