)]}'
{"nimbleclient/osc/v1/instance.py":[{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"0b2f2f9b7866da82f9c70713558b8cc59ebd51a0","unresolved":false,"context_lines":[{"line_number":41,"context_line":"            help\u003d_(\"New instance name\")"},{"line_number":42,"context_line":"        )"},{"line_number":43,"context_line":"        parser.add_argument("},{"line_number":44,"context_line":"            \"--instance-type\","},{"line_number":45,"context_line":"            metavar\u003d\"\u003ctype-id\u003e\","},{"line_number":46,"context_line":"            required\u003dTrue,"},{"line_number":47,"context_line":"            help\u003d_(\"ID of instance type\"),"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_9d7c2874","line":44,"range":{"start_line":44,"start_character":15,"end_line":44,"end_character":28},"updated":"2016-11-08 12:05:55.000000000","message":"should be \"type\", keep consistent with command \"baremetal compute type list\"","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"f3396144aec388db004e248f7ad2e2617da6a9d4","unresolved":false,"context_lines":[{"line_number":41,"context_line":"            help\u003d_(\"New instance name\")"},{"line_number":42,"context_line":"        )"},{"line_number":43,"context_line":"        parser.add_argument("},{"line_number":44,"context_line":"            \"--instance-type\","},{"line_number":45,"context_line":"            metavar\u003d\"\u003ctype-id\u003e\","},{"line_number":46,"context_line":"            required\u003dTrue,"},{"line_number":47,"context_line":"            help\u003d_(\"ID of instance type\"),"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_9e7ddde9","line":44,"range":{"start_line":44,"start_character":15,"end_line":44,"end_character":28},"in_reply_to":"9a629dbe_9d7c2874","updated":"2016-11-09 01:21:31.000000000","message":"agree","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"0b2f2f9b7866da82f9c70713558b8cc59ebd51a0","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        )"},{"line_number":43,"context_line":"        parser.add_argument("},{"line_number":44,"context_line":"            \"--instance-type\","},{"line_number":45,"context_line":"            metavar\u003d\"\u003ctype-id\u003e\","},{"line_number":46,"context_line":"            required\u003dTrue,"},{"line_number":47,"context_line":"            help\u003d_(\"ID of instance type\"),"},{"line_number":48,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_0224390b","line":45,"updated":"2016-11-08 12:05:55.000000000","message":"we should support both name and ID argument, that keep consistent with other OSC commands, and supporting resource name enhance the UX for end users.","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"f3396144aec388db004e248f7ad2e2617da6a9d4","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        )"},{"line_number":43,"context_line":"        parser.add_argument("},{"line_number":44,"context_line":"            \"--instance-type\","},{"line_number":45,"context_line":"            metavar\u003d\"\u003ctype-id\u003e\","},{"line_number":46,"context_line":"            required\u003dTrue,"},{"line_number":47,"context_line":"            help\u003d_(\"ID of instance type\"),"},{"line_number":48,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_5e4b45b0","line":45,"in_reply_to":"9a629dbe_0224390b","updated":"2016-11-09 01:21:31.000000000","message":"maybe you are right","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"0b2f2f9b7866da82f9c70713558b8cc59ebd51a0","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        )"},{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \"--image\","},{"line_number":51,"context_line":"            metavar\u003d\"\u003cimage-id\u003e\","},{"line_number":52,"context_line":"            required\u003dTrue,"},{"line_number":53,"context_line":"            help\u003d_(\"ID of image\"),"},{"line_number":54,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_627975fc","line":51,"updated":"2016-11-08 12:05:55.000000000","message":"ditto","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"f3396144aec388db004e248f7ad2e2617da6a9d4","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        )"},{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \"--image\","},{"line_number":51,"context_line":"            metavar\u003d\"\u003cimage-id\u003e\","},{"line_number":52,"context_line":"            required\u003dTrue,"},{"line_number":53,"context_line":"            help\u003d_(\"ID of image\"),"},{"line_number":54,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_7ec80923","line":51,"in_reply_to":"9a629dbe_627975fc","updated":"2016-11-09 01:21:31.000000000","message":"Done","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"0b2f2f9b7866da82f9c70713558b8cc59ebd51a0","unresolved":false,"context_lines":[{"line_number":53,"context_line":"            help\u003d_(\"ID of image\"),"},{"line_number":54,"context_line":"        )"},{"line_number":55,"context_line":"        parser.add_argument("},{"line_number":56,"context_line":"            \"--network\","},{"line_number":57,"context_line":"            metavar\u003d\"\u003cnetwork-id\u003d[,port-type\u003d]\u003e\","},{"line_number":58,"context_line":"            required\u003dTrue,"},{"line_number":59,"context_line":"            action\u003dparseractions.MultiKeyValueAction,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_5da1b0a8","line":56,"range":{"start_line":56,"start_character":15,"end_line":56,"end_character":22},"updated":"2016-11-08 12:05:55.000000000","message":"the option can be repeated, and one option is one nic configuration, not whole instance network configuration, so I think the option name should be \"--nic\".","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"f3396144aec388db004e248f7ad2e2617da6a9d4","unresolved":false,"context_lines":[{"line_number":53,"context_line":"            help\u003d_(\"ID of image\"),"},{"line_number":54,"context_line":"        )"},{"line_number":55,"context_line":"        parser.add_argument("},{"line_number":56,"context_line":"            \"--network\","},{"line_number":57,"context_line":"            metavar\u003d\"\u003cnetwork-id\u003d[,port-type\u003d]\u003e\","},{"line_number":58,"context_line":"            required\u003dTrue,"},{"line_number":59,"context_line":"            action\u003dparseractions.MultiKeyValueAction,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_1ed8edee","line":56,"range":{"start_line":56,"start_character":15,"end_line":56,"end_character":22},"in_reply_to":"9a629dbe_5da1b0a8","updated":"2016-11-09 01:21:31.000000000","message":"agree","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"0b2f2f9b7866da82f9c70713558b8cc59ebd51a0","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        )"},{"line_number":55,"context_line":"        parser.add_argument("},{"line_number":56,"context_line":"            \"--network\","},{"line_number":57,"context_line":"            metavar\u003d\"\u003cnetwork-id\u003d[,port-type\u003d]\u003e\","},{"line_number":58,"context_line":"            required\u003dTrue,"},{"line_number":59,"context_line":"            action\u003dparseractions.MultiKeyValueAction,"},{"line_number":60,"context_line":"            dest\u003d\u0027add_networks\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_e28905cc","line":57,"range":{"start_line":57,"start_character":22,"end_line":57,"end_character":32},"updated":"2016-11-08 12:05:55.000000000","message":"looks the \u0027network-id\u0027 is required arguments, please add \u0027required_keys\u0027 for the parser, see reference command:\n\n    openstack subnet unset","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"f3396144aec388db004e248f7ad2e2617da6a9d4","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        )"},{"line_number":55,"context_line":"        parser.add_argument("},{"line_number":56,"context_line":"            \"--network\","},{"line_number":57,"context_line":"            metavar\u003d\"\u003cnetwork-id\u003d[,port-type\u003d]\u003e\","},{"line_number":58,"context_line":"            required\u003dTrue,"},{"line_number":59,"context_line":"            action\u003dparseractions.MultiKeyValueAction,"},{"line_number":60,"context_line":"            dest\u003d\u0027add_networks\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_bee9c182","line":57,"range":{"start_line":57,"start_character":22,"end_line":57,"end_character":32},"in_reply_to":"9a629dbe_e28905cc","updated":"2016-11-09 01:21:31.000000000","message":"good suggestion","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"0b2f2f9b7866da82f9c70713558b8cc59ebd51a0","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            required\u003dTrue,"},{"line_number":59,"context_line":"            action\u003dparseractions.MultiKeyValueAction,"},{"line_number":60,"context_line":"            dest\u003d\u0027add_networks\u0027,"},{"line_number":61,"context_line":"            help\u003d_(\"Create a NIC on the instance. Specify option multiple \""},{"line_number":62,"context_line":"                   \"times to create multiple nics\"),"},{"line_number":63,"context_line":"        )"},{"line_number":64,"context_line":"        parser.add_argument("},{"line_number":65,"context_line":"            \"--description\","}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_5d8ff022","line":62,"range":{"start_line":61,"start_character":20,"end_line":62,"end_character":50},"updated":"2016-11-08 12:05:55.000000000","message":"the help message of repeat option looks like:\n\n    Property to add to this aggregate (repeat option to set multiple properties)\n\nrefer in OSC devref:\n\nhttp://docs.openstack.org/developer/python-openstackclient/command-options.html#id2","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"f3396144aec388db004e248f7ad2e2617da6a9d4","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            required\u003dTrue,"},{"line_number":59,"context_line":"            action\u003dparseractions.MultiKeyValueAction,"},{"line_number":60,"context_line":"            dest\u003d\u0027add_networks\u0027,"},{"line_number":61,"context_line":"            help\u003d_(\"Create a NIC on the instance. Specify option multiple \""},{"line_number":62,"context_line":"                   \"times to create multiple nics\"),"},{"line_number":63,"context_line":"        )"},{"line_number":64,"context_line":"        parser.add_argument("},{"line_number":65,"context_line":"            \"--description\","}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_7e9649f1","line":62,"range":{"start_line":61,"start_character":20,"end_line":62,"end_character":50},"in_reply_to":"9a629dbe_5d8ff022","updated":"2016-11-09 01:21:31.000000000","message":"let me check","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"0b2f2f9b7866da82f9c70713558b8cc59ebd51a0","unresolved":false,"context_lines":[{"line_number":76,"context_line":"    def take_action(self, parsed_args):"},{"line_number":77,"context_line":"        bc_client \u003d self.app.client_manager.baremetal_compute"},{"line_number":78,"context_line":"        networks \u003d list()"},{"line_number":79,"context_line":"        for network in parsed_args.add_networks:"},{"line_number":80,"context_line":"            net_info \u003d {\u0027uuid\u0027: network.get(\u0027network-id\u0027)}"},{"line_number":81,"context_line":"            port_type \u003d network.get(\u0027port-type\u0027)"},{"line_number":82,"context_line":"            if port_type:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_fdffe431","line":79,"range":{"start_line":79,"start_character":35,"end_line":79,"end_character":47},"updated":"2016-11-08 12:05:55.000000000","message":"avoid the verb in parser destination, that make the code looks like a method, not an attribute.","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"f3396144aec388db004e248f7ad2e2617da6a9d4","unresolved":false,"context_lines":[{"line_number":76,"context_line":"    def take_action(self, parsed_args):"},{"line_number":77,"context_line":"        bc_client \u003d self.app.client_manager.baremetal_compute"},{"line_number":78,"context_line":"        networks \u003d list()"},{"line_number":79,"context_line":"        for network in parsed_args.add_networks:"},{"line_number":80,"context_line":"            net_info \u003d {\u0027uuid\u0027: network.get(\u0027network-id\u0027)}"},{"line_number":81,"context_line":"            port_type \u003d network.get(\u0027port-type\u0027)"},{"line_number":82,"context_line":"            if port_type:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_7ebd6967","line":79,"range":{"start_line":79,"start_character":35,"end_line":79,"end_character":47},"in_reply_to":"9a629dbe_fdffe431","updated":"2016-11-09 01:21:31.000000000","message":"ok","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"0b2f2f9b7866da82f9c70713558b8cc59ebd51a0","unresolved":false,"context_lines":[{"line_number":175,"context_line":"                \"UUID\","},{"line_number":176,"context_line":"                \"Name\","},{"line_number":177,"context_line":"                \"Instance Type\","},{"line_number":178,"context_line":"                \"Status\""},{"line_number":179,"context_line":"            )"},{"line_number":180,"context_line":"            columns \u003d ("},{"line_number":181,"context_line":"                \"uuid\","}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_27822f95","line":178,"updated":"2016-11-08 12:05:55.000000000","message":"I would like to see the instance network information in short output, that\u0027s helpful in most cases.","commit_id":"bee33eb5ad5bf620c64cff12113e3f51856472af"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"633b3e88a770a1b49bb5a4760262f681d80ecf38","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            required_keys\u003d[\u0027uuid\u0027],"},{"line_number":59,"context_line":"            optional_keys\u003d[\u0027port-type\u0027],"},{"line_number":60,"context_line":"            action\u003dparseractions.MultiKeyValueAction,"},{"line_number":61,"context_line":"            help\u003d_(\"Create a NIC on the instance.\""},{"line_number":62,"context_line":"                   \"(repeat option to create multiple NICs)\"),"},{"line_number":63,"context_line":"        )"},{"line_number":64,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_336fc2c6","line":61,"updated":"2016-11-16 03:35:23.000000000","message":"please add a blank at the end of line.","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"fb82cc6054aaf1345e99f2953c77e0a92444fb79","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            required_keys\u003d[\u0027uuid\u0027],"},{"line_number":59,"context_line":"            optional_keys\u003d[\u0027port-type\u0027],"},{"line_number":60,"context_line":"            action\u003dparseractions.MultiKeyValueAction,"},{"line_number":61,"context_line":"            help\u003d_(\"Create a NIC on the instance.\""},{"line_number":62,"context_line":"                   \"(repeat option to create multiple NICs)\"),"},{"line_number":63,"context_line":"        )"},{"line_number":64,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_ea2b8492","line":61,"in_reply_to":"7a77a97e_336fc2c6","updated":"2016-11-16 08:41:49.000000000","message":"Done","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"}],"nimbleclient/tests/unit/fakes.py":[{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"633b3e88a770a1b49bb5a4760262f681d80ecf38","unresolved":false,"context_lines":[{"line_number":224,"context_line":"    \"\"\"Fake one instance.\"\"\""},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    @staticmethod"},{"line_number":227,"context_line":"    def create_fake_instance(name, type, image, networks,"},{"line_number":228,"context_line":"                             availability_zone\u003dNone,"},{"line_number":229,"context_line":"                             description\u003dNone, extra\u003dNone):"},{"line_number":230,"context_line":"        instance_info \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_0527d9b6","line":227,"range":{"start_line":227,"start_character":8,"end_line":227,"end_character":28},"updated":"2016-11-16 03:35:23.000000000","message":"seems the function can be replaced by \"create_one_instance\".","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"fb82cc6054aaf1345e99f2953c77e0a92444fb79","unresolved":false,"context_lines":[{"line_number":224,"context_line":"    \"\"\"Fake one instance.\"\"\""},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    @staticmethod"},{"line_number":227,"context_line":"    def create_fake_instance(name, type, image, networks,"},{"line_number":228,"context_line":"                             availability_zone\u003dNone,"},{"line_number":229,"context_line":"                             description\u003dNone, extra\u003dNone):"},{"line_number":230,"context_line":"        instance_info \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_8a2690cc","line":227,"range":{"start_line":227,"start_character":8,"end_line":227,"end_character":28},"in_reply_to":"7a77a97e_0527d9b6","updated":"2016-11-16 08:41:49.000000000","message":"agree","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"}],"nimbleclient/tests/unit/osc/v1/test_instance.py":[{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"633b3e88a770a1b49bb5a4760262f681d80ecf38","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class TestInstance(test_base.TestBaremetalComputeV1):"},{"line_number":28,"context_line":"    fake_instance \u003d fakes.FakeInstance.create_one_instance()"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    columns \u003d ("},{"line_number":31,"context_line":"        \u0027availability_zone\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_65240daf","line":28,"updated":"2016-11-16 03:35:23.000000000","message":"here","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"fb82cc6054aaf1345e99f2953c77e0a92444fb79","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class TestInstance(test_base.TestBaremetalComputeV1):"},{"line_number":28,"context_line":"    fake_instance \u003d fakes.FakeInstance.create_one_instance()"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    columns \u003d ("},{"line_number":31,"context_line":"        \u0027availability_zone\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_8a0db040","line":28,"in_reply_to":"7a77a97e_65240daf","updated":"2016-11-16 08:41:49.000000000","message":"Done","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"633b3e88a770a1b49bb5a4760262f681d80ecf38","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        image_obj \u003d mock.Mock()"},{"line_number":108,"context_line":"        image_obj.id \u003d image_id"},{"line_number":109,"context_line":"        mock_find.side_effect \u003d [type_obj, image_obj]"},{"line_number":110,"context_line":"        fk_instance \u003d fakes.FakeInstance.create_fake_instance("},{"line_number":111,"context_line":"            name, type_id, image_id, networks)"},{"line_number":112,"context_line":"        mock_create.return_value \u003d fk_instance"},{"line_number":113,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verifylist)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_252a157f","line":110,"range":{"start_line":110,"start_character":41,"end_line":110,"end_character":61},"updated":"2016-11-16 03:35:23.000000000","message":"could we reuse the self.fake_instance in line #28 to avoid to create new fake object?","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"fb82cc6054aaf1345e99f2953c77e0a92444fb79","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        image_obj \u003d mock.Mock()"},{"line_number":108,"context_line":"        image_obj.id \u003d image_id"},{"line_number":109,"context_line":"        mock_find.side_effect \u003d [type_obj, image_obj]"},{"line_number":110,"context_line":"        fk_instance \u003d fakes.FakeInstance.create_fake_instance("},{"line_number":111,"context_line":"            name, type_id, image_id, networks)"},{"line_number":112,"context_line":"        mock_create.return_value \u003d fk_instance"},{"line_number":113,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verifylist)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_ca07a81a","line":110,"range":{"start_line":110,"start_character":41,"end_line":110,"end_character":61},"in_reply_to":"7a77a97e_252a157f","updated":"2016-11-16 08:41:49.000000000","message":"Done","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"633b3e88a770a1b49bb5a4760262f681d80ecf38","unresolved":false,"context_lines":[{"line_number":111,"context_line":"            name, type_id, image_id, networks)"},{"line_number":112,"context_line":"        mock_create.return_value \u003d fk_instance"},{"line_number":113,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verifylist)"},{"line_number":114,"context_line":"        self.cmd.take_action(parsed_args)"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"        mock_create.assert_called_once_with("},{"line_number":117,"context_line":"            \u0027/instances\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_f3638a18","line":114,"range":{"start_line":114,"start_character":8,"end_line":114,"end_character":41},"updated":"2016-11-16 03:35:23.000000000","message":"please check the return is same as you expected.","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"},{"author":{"_account_id":8976,"name":"shihanzhang","email":"ayshihanzhang@gmail.com","username":"shihanzhang"},"change_message_id":"fb82cc6054aaf1345e99f2953c77e0a92444fb79","unresolved":false,"context_lines":[{"line_number":111,"context_line":"            name, type_id, image_id, networks)"},{"line_number":112,"context_line":"        mock_create.return_value \u003d fk_instance"},{"line_number":113,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verifylist)"},{"line_number":114,"context_line":"        self.cmd.take_action(parsed_args)"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"        mock_create.assert_called_once_with("},{"line_number":117,"context_line":"            \u0027/instances\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_aa08ac2e","line":114,"range":{"start_line":114,"start_character":8,"end_line":114,"end_character":41},"in_reply_to":"7a77a97e_f3638a18","updated":"2016-11-16 08:41:49.000000000","message":"got it","commit_id":"e8bd987e9513ebd064bcb6d0134f2b2859e8f192"}]}