)]}'
{"neutronclient/neutron/v2_0/port.py":[{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"50caa64ca22f92036014375d1eb61f820c281643","unresolved":false,"context_lines":[{"line_number":243,"context_line":"            choices\u003d[\u0027direct\u0027, \u0027macvtap\u0027, \u0027normal\u0027, \u0027baremetal\u0027],"},{"line_number":244,"context_line":"            help\u003d_(\u0027VNIC type for this port.\u0027))"},{"line_number":245,"context_line":"        parser.add_argument("},{"line_number":246,"context_line":"            \u0027--vnic_type\u0027,"},{"line_number":247,"context_line":"            choices\u003d[\u0027direct\u0027, \u0027macvtap\u0027, \u0027normal\u0027, \u0027baremetal\u0027],"},{"line_number":248,"context_line":"            help\u003dargparse.SUPPRESS)"},{"line_number":249,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_152eb4a5","line":246,"updated":"2015-11-30 13:46:01.000000000","message":"I may sound stupid asking this.\n\nWhy we don\u0027t have \"metavar\u003d\u0027\u003cdirect | macvtap | normal | baremetal\u003e\u0027\" here as well.","commit_id":"0042a39d9245b9262b95b7b124a2bb15ba2682e5"},{"author":{"_account_id":7448,"name":"Carl Baldwin","email":"carl@ecbaldwin.net","username":"carl-baldwin"},"change_message_id":"d5fa4d7daece308f8962bda0464eea558c326b4b","unresolved":false,"context_lines":[{"line_number":243,"context_line":"            choices\u003d[\u0027direct\u0027, \u0027macvtap\u0027, \u0027normal\u0027, \u0027baremetal\u0027],"},{"line_number":244,"context_line":"            help\u003d_(\u0027VNIC type for this port.\u0027))"},{"line_number":245,"context_line":"        parser.add_argument("},{"line_number":246,"context_line":"            \u0027--vnic_type\u0027,"},{"line_number":247,"context_line":"            choices\u003d[\u0027direct\u0027, \u0027macvtap\u0027, \u0027normal\u0027, \u0027baremetal\u0027],"},{"line_number":248,"context_line":"            help\u003dargparse.SUPPRESS)"},{"line_number":249,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_c7c0747f","line":246,"in_reply_to":"9a8ffd7b_152eb4a5","updated":"2015-11-30 17:24:47.000000000","message":"It is a good question but I don\u0027t think it should hold up this patch set.","commit_id":"0042a39d9245b9262b95b7b124a2bb15ba2682e5"}],"neutronclient/tests/unit/test_cli20_port.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c1c5263e1e948c220515f94e2cbf982f9b2740bf","unresolved":false,"context_lines":[{"line_number":172,"context_line":"        position_values \u003d [\u0027macvtap\u0027, netid]"},{"line_number":173,"context_line":"        self._test_create_resource(resource, cmd, name, myid, args,"},{"line_number":174,"context_line":"                                   position_names, position_values)"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    def test_create_port_vnic_type_baremetal(self):"},{"line_number":177,"context_line":"        \"\"\"Create port: --vnic_type baremetal netid.\"\"\""},{"line_number":178,"context_line":"        resource \u003d \u0027port\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_51e863b4","line":175,"updated":"2015-11-26 01:45:47.000000000","message":"A lot of duplicated code. I need to refactor, but okay.","commit_id":"0042a39d9245b9262b95b7b124a2bb15ba2682e5"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"2f80cd92f97c7dfc27516910a3f906522a3a620d","unresolved":false,"context_lines":[{"line_number":172,"context_line":"        position_values \u003d [\u0027macvtap\u0027, netid]"},{"line_number":173,"context_line":"        self._test_create_resource(resource, cmd, name, myid, args,"},{"line_number":174,"context_line":"                                   position_names, position_values)"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    def test_create_port_vnic_type_baremetal(self):"},{"line_number":177,"context_line":"        \"\"\"Create port: --vnic_type baremetal netid.\"\"\""},{"line_number":178,"context_line":"        resource \u003d \u0027port\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_c9a6e752","line":175,"in_reply_to":"9a8ffd7b_51e863b4","updated":"2015-11-26 04:52:10.000000000","message":"I agree, seems the same code is used everywhere :(","commit_id":"0042a39d9245b9262b95b7b124a2bb15ba2682e5"}]}