)]}'
{".gitignore":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"d58e46cd01952b6b5cac4be4a8aec767d251d917","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5f2577a7_f6f0bf69","updated":"2017-09-14 05:25:00.000000000","message":"Do we need this?","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"f52e6052d3300a75a424012b32201e1a852cda4a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff5d4b2f_9050dcd6","in_reply_to":"5f2577a7_f6f0bf69","updated":"2017-09-19 05:23:09.000000000","message":"Removed.","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"}],"/COMMIT_MSG":[{"author":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"change_message_id":"969070ac9faf740dda9d3b583eabe4f398470b77","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I1f828ef965d5b117539b88a290e2b9b966016336"},{"line_number":12,"context_line":"Partial-Bug: #1590908"},{"line_number":13,"context_line":"Depends-On: I1f828ef965d5b117539b88a290e2b9b966016336"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7f515b1d_7fae2f2f","line":13,"range":{"start_line":13,"start_character":12,"end_line":13,"end_character":53},"updated":"2017-09-22 23:06:06.000000000","message":"this seems like it is depending on itself.. you may want to modify this to add the real dependent change ID and if i am not mistaken it should be I3d47000bb0ad8434f88d016acb5b05fb7142e47a","commit_id":"407c394674827603b3e7a99d7babc81bc5cd05fd"}],"neutronclient/osc/v2/networking_bgpvpn/bgpvpn.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"d58e46cd01952b6b5cac4be4a8aec767d251d917","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    parser.add_argument("},{"line_number":149,"context_line":"        \u0027--vni\u0027,"},{"line_number":150,"context_line":"        metavar\u003d\"\u003cvni\u003e\","},{"line_number":151,"context_line":"        help\u003d_(\u0027VXLAN Network Identifier to be used for this BGPVPN \u0027"},{"line_number":152,"context_line":"               \u0027when a VXLAN encapsulation is used\u0027))"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f2577a7_7604cf4a","line":151,"range":{"start_line":151,"start_character":57,"end_line":151,"end_character":65},"updated":"2017-09-14 05:25:00.000000000","message":"maybe it can be reworded?","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"b6e92940ec41f7e61ccc3fbeacc3b94404cce38b","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    parser.add_argument("},{"line_number":149,"context_line":"        \u0027--vni\u0027,"},{"line_number":150,"context_line":"        metavar\u003d\"\u003cvni\u003e\","},{"line_number":151,"context_line":"        help\u003d_(\u0027VXLAN Network Identifier to be used for this BGPVPN \u0027"},{"line_number":152,"context_line":"               \u0027when a VXLAN encapsulation is used\u0027))"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"def _args2body(client_manager, id, action, args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f2577a7_c5de0c6c","line":152,"range":{"start_line":151,"start_character":16,"end_line":152,"end_character":52},"updated":"2017-09-15 01:30:45.000000000","message":"The sentence is clear enough for me, but perhaps I\u0027m too familiar with the idea.\n\nReedip, would you describe more why you\u0027d reword or perhaps suggest an alternative text?","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"d58e46cd01952b6b5cac4be4a8aec767d251d917","unresolved":false,"context_lines":[{"line_number":149,"context_line":"        \u0027--vni\u0027,"},{"line_number":150,"context_line":"        metavar\u003d\"\u003cvni\u003e\","},{"line_number":151,"context_line":"        help\u003d_(\u0027VXLAN Network Identifier to be used for this BGPVPN \u0027"},{"line_number":152,"context_line":"               \u0027when a VXLAN encapsulation is used\u0027))"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"def _args2body(client_manager, id, action, args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f2577a7_b6ea3772","line":152,"range":{"start_line":152,"start_character":42,"end_line":152,"end_character":50},"updated":"2017-09-14 05:25:00.000000000","message":"is enabled?","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"d58e46cd01952b6b5cac4be4a8aec767d251d917","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        attrs[\u0027name\u0027] \u003d str(args.name)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    if \u0027vni\u0027 in args and args.vni is not None:"},{"line_number":169,"context_line":"        attrs[\u0027vni\u0027] \u003d str(args.vni)"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    if args.purge_route_target:"},{"line_number":172,"context_line":"        attrs[\u0027route_targets\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f2577a7_36fe4737","line":169,"range":{"start_line":169,"start_character":23,"end_line":169,"end_character":26},"updated":"2017-09-14 05:25:00.000000000","message":"The input would be in STR, you dont need to change it","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"f52e6052d3300a75a424012b32201e1a852cda4a","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        attrs[\u0027name\u0027] \u003d str(args.name)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    if \u0027vni\u0027 in args and args.vni is not None:"},{"line_number":169,"context_line":"        attrs[\u0027vni\u0027] \u003d str(args.vni)"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    if args.purge_route_target:"},{"line_number":172,"context_line":"        attrs[\u0027route_targets\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff5d4b2f_d0f414d2","line":169,"range":{"start_line":169,"start_character":23,"end_line":169,"end_character":26},"in_reply_to":"5f2577a7_057c1470","updated":"2017-09-19 05:23:09.000000000","message":"Done","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"b6e92940ec41f7e61ccc3fbeacc3b94404cce38b","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        attrs[\u0027name\u0027] \u003d str(args.name)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    if \u0027vni\u0027 in args and args.vni is not None:"},{"line_number":169,"context_line":"        attrs[\u0027vni\u0027] \u003d str(args.vni)"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    if args.purge_route_target:"},{"line_number":172,"context_line":"        attrs[\u0027route_targets\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f2577a7_057c1470","line":169,"range":{"start_line":169,"start_character":23,"end_line":169,"end_character":26},"in_reply_to":"5f2577a7_36fe4737","updated":"2017-09-15 01:30:45.000000000","message":"In fact, you need to convert to an integer.","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"d58e46cd01952b6b5cac4be4a8aec767d251d917","unresolved":false,"context_lines":[{"line_number":244,"context_line":"            attrs[\u0027export_targets\u0027] \u003d parsed_args.export_targets"},{"line_number":245,"context_line":"        if parsed_args.route_distinguishers is not None:"},{"line_number":246,"context_line":"            attrs[\u0027route_distinguishers\u0027] \u003d parsed_args.route_distinguishers"},{"line_number":247,"context_line":"        if parsed_args.vni is not None:"},{"line_number":248,"context_line":"            attrs[\u0027vni\u0027] \u003d parsed_args.vni"},{"line_number":249,"context_line":"        if \u0027project\u0027 in parsed_args and parsed_args.project is not None:"},{"line_number":250,"context_line":"            project_id \u003d nc_osc_utils.find_project("}],"source_content_type":"text/x-python","patch_set":1,"id":"5f2577a7_36ac0733","line":247,"range":{"start_line":247,"start_character":0,"end_line":247,"end_character":39},"updated":"2017-09-14 05:25:00.000000000","message":"I couldnt find the API definition in neutron-lib repo or networking-bgpvpn.\n\nJust want to know why we cannot modify ( Set/Unset ) the VNI in BGPVPN.\nThis is just a question","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"b6e92940ec41f7e61ccc3fbeacc3b94404cce38b","unresolved":false,"context_lines":[{"line_number":244,"context_line":"            attrs[\u0027export_targets\u0027] \u003d parsed_args.export_targets"},{"line_number":245,"context_line":"        if parsed_args.route_distinguishers is not None:"},{"line_number":246,"context_line":"            attrs[\u0027route_distinguishers\u0027] \u003d parsed_args.route_distinguishers"},{"line_number":247,"context_line":"        if parsed_args.vni is not None:"},{"line_number":248,"context_line":"            attrs[\u0027vni\u0027] \u003d parsed_args.vni"},{"line_number":249,"context_line":"        if \u0027project\u0027 in parsed_args and parsed_args.project is not None:"},{"line_number":250,"context_line":"            project_id \u003d nc_osc_utils.find_project("}],"source_content_type":"text/x-python","patch_set":1,"id":"5f2577a7_858fe473","line":247,"range":{"start_line":247,"start_character":0,"end_line":247,"end_character":39},"in_reply_to":"5f2577a7_36ac0733","updated":"2017-09-15 01:30:45.000000000","message":"The API definition is in change Iae5fe901f0cddfbf8b1ef3afd529f9131b0aa165\n\nPUT is allowed in this definition","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"2a7576695a715fdaceb67c006d7877d9fb2441e0","unresolved":false,"context_lines":[{"line_number":244,"context_line":"            attrs[\u0027export_targets\u0027] \u003d parsed_args.export_targets"},{"line_number":245,"context_line":"        if parsed_args.route_distinguishers is not None:"},{"line_number":246,"context_line":"            attrs[\u0027route_distinguishers\u0027] \u003d parsed_args.route_distinguishers"},{"line_number":247,"context_line":"        if parsed_args.vni is not None:"},{"line_number":248,"context_line":"            attrs[\u0027vni\u0027] \u003d parsed_args.vni"},{"line_number":249,"context_line":"        if \u0027project\u0027 in parsed_args and parsed_args.project is not None:"},{"line_number":250,"context_line":"            project_id \u003d nc_osc_utils.find_project("}],"source_content_type":"text/x-python","patch_set":1,"id":"5f2577a7_e5cea8b2","line":247,"range":{"start_line":247,"start_character":0,"end_line":247,"end_character":39},"in_reply_to":"5f2577a7_858fe473","updated":"2017-09-15 01:33:25.000000000","message":"So I agree both need to be made consistent.","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"f52e6052d3300a75a424012b32201e1a852cda4a","unresolved":false,"context_lines":[{"line_number":244,"context_line":"            attrs[\u0027export_targets\u0027] \u003d parsed_args.export_targets"},{"line_number":245,"context_line":"        if parsed_args.route_distinguishers is not None:"},{"line_number":246,"context_line":"            attrs[\u0027route_distinguishers\u0027] \u003d parsed_args.route_distinguishers"},{"line_number":247,"context_line":"        if parsed_args.vni is not None:"},{"line_number":248,"context_line":"            attrs[\u0027vni\u0027] \u003d parsed_args.vni"},{"line_number":249,"context_line":"        if \u0027project\u0027 in parsed_args and parsed_args.project is not None:"},{"line_number":250,"context_line":"            project_id \u003d nc_osc_utils.find_project("}],"source_content_type":"text/x-python","patch_set":1,"id":"ff5d4b2f_102cec45","line":247,"range":{"start_line":247,"start_character":0,"end_line":247,"end_character":39},"in_reply_to":"5f2577a7_e5cea8b2","updated":"2017-09-19 05:23:09.000000000","message":"As discussed in API definitions review: https://review.openstack.org/#/c/503954/2/neutron_lib/api/definitions/bgpvpn_vni.py, the vni attribute will be read-only.","commit_id":"dae69bce32f8562c346a62b4a4fb8cae4c09f6a5"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"e4072a80f766a1222a9596aff9ceb6a7de4c7d60","unresolved":false,"context_lines":[{"line_number":146,"context_line":"            help\u003d_(\u0027Empty route distinguisher list\u0027),"},{"line_number":147,"context_line":"        )"},{"line_number":148,"context_line":"    parser.add_argument("},{"line_number":149,"context_line":"        \u0027--vni\u0027,"},{"line_number":150,"context_line":"        metavar\u003d\"\u003cvni\u003e\","},{"line_number":151,"context_line":"        help\u003d_(\u0027VXLAN Network Identifier to be used for this BGPVPN \u0027"},{"line_number":152,"context_line":"               \u0027when a VXLAN encapsulation is used\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf57530f_288780da","line":149,"updated":"2017-09-20 08:04:59.000000000","message":"I think we want \u0027type\u003dint\u0027 here.","commit_id":"ae460c38ebb7ec39faaa6812d2deb9f327f4b76b"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"a770bdc04844df9f872b0ff3aa0bd7d086c35baa","unresolved":false,"context_lines":[{"line_number":146,"context_line":"            help\u003d_(\u0027Empty route distinguisher list\u0027),"},{"line_number":147,"context_line":"        )"},{"line_number":148,"context_line":"    parser.add_argument("},{"line_number":149,"context_line":"        \u0027--vni\u0027,"},{"line_number":150,"context_line":"        metavar\u003d\"\u003cvni\u003e\","},{"line_number":151,"context_line":"        help\u003d_(\u0027VXLAN Network Identifier to be used for this BGPVPN \u0027"},{"line_number":152,"context_line":"               \u0027when a VXLAN encapsulation is used\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf57530f_14da91e8","line":149,"in_reply_to":"bf57530f_288780da","updated":"2017-09-20 10:55:32.000000000","message":"Done","commit_id":"ae460c38ebb7ec39faaa6812d2deb9f327f4b76b"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"60fcab12b688b0edeb6d2d57557c7fbfc5d50beb","unresolved":false,"context_lines":[{"line_number":147,"context_line":"        )"},{"line_number":148,"context_line":"    parser.add_argument("},{"line_number":149,"context_line":"        \u0027--vni\u0027,"},{"line_number":150,"context_line":"        metavar\u003d\"\u003cvni\u003e\","},{"line_number":151,"context_line":"        help\u003d_(\u0027VXLAN Network Identifier to be used for this BGPVPN \u0027"},{"line_number":152,"context_line":"               \u0027when a VXLAN encapsulation is used\u0027))"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"bf57530f_88754c9b","line":150,"range":{"start_line":150,"start_character":2,"end_line":150,"end_character":24},"updated":"2017-09-20 07:35:41.000000000","message":"this is useless (equals to the default which, inferred from \u0027--vni\u0027, is \u0027vni\u0027)","commit_id":"ae460c38ebb7ec39faaa6812d2deb9f327f4b76b"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"a770bdc04844df9f872b0ff3aa0bd7d086c35baa","unresolved":false,"context_lines":[{"line_number":147,"context_line":"        )"},{"line_number":148,"context_line":"    parser.add_argument("},{"line_number":149,"context_line":"        \u0027--vni\u0027,"},{"line_number":150,"context_line":"        metavar\u003d\"\u003cvni\u003e\","},{"line_number":151,"context_line":"        help\u003d_(\u0027VXLAN Network Identifier to be used for this BGPVPN \u0027"},{"line_number":152,"context_line":"               \u0027when a VXLAN encapsulation is used\u0027))"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"bf57530f_34d595ba","line":150,"range":{"start_line":150,"start_character":2,"end_line":150,"end_character":24},"in_reply_to":"bf57530f_88754c9b","updated":"2017-09-20 10:55:32.000000000","message":"Done","commit_id":"ae460c38ebb7ec39faaa6812d2deb9f327f4b76b"},{"author":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"change_message_id":"cb1deaaf14a72f801c98aa0c9935466b42b2a360","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        attrs[\u0027name\u0027] \u003d str(args.name)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    if \u0027vni\u0027 in args and args.vni is not None:"},{"line_number":168,"context_line":"        attrs[\u0027vni\u0027] \u003d int(args.vni)"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    if args.purge_route_target:"},{"line_number":171,"context_line":"        attrs[\u0027route_targets\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f5c4f37_3b0d8f90","line":168,"range":{"start_line":168,"start_character":23,"end_line":168,"end_character":26},"updated":"2017-09-21 02:19:05.000000000","message":"is int necessary? isnt the arg type int already?","commit_id":"afe7f4a3b24b5cf898b54d2c7d8c27cf1c04b1f6"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"ef72da169295f79a3159c737f78324e9ea5eb9bb","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        attrs[\u0027name\u0027] \u003d str(args.name)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    if \u0027vni\u0027 in args and args.vni is not None:"},{"line_number":168,"context_line":"        attrs[\u0027vni\u0027] \u003d int(args.vni)"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    if args.purge_route_target:"},{"line_number":171,"context_line":"        attrs[\u0027route_targets\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f5c4f37_66512b0f","line":168,"in_reply_to":"9f5c4f37_3b0d8f90","updated":"2017-09-21 05:18:59.000000000","message":"Ah yes, good catch.","commit_id":"afe7f4a3b24b5cf898b54d2c7d8c27cf1c04b1f6"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"155603a03f7126bba2e32d6b90f630d253c74fab","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        attrs[\u0027name\u0027] \u003d str(args.name)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    if \u0027vni\u0027 in args and args.vni is not None:"},{"line_number":168,"context_line":"        attrs[\u0027vni\u0027] \u003d int(args.vni)"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    if args.purge_route_target:"},{"line_number":171,"context_line":"        attrs[\u0027route_targets\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f5c4f37_0694af43","line":168,"in_reply_to":"9f5c4f37_66512b0f","updated":"2017-09-21 05:41:18.000000000","message":"Thanks Abhishek. Corrected it.","commit_id":"afe7f4a3b24b5cf898b54d2c7d8c27cf1c04b1f6"},{"author":{"_account_id":55,"name":"Édouard Thuleau","email":"ethuleau@juniper.net","username":"ethuleau"},"change_message_id":"58bc31afe19ef66afac9f25482c730c76ddf8c71","unresolved":false,"context_lines":[{"line_number":40,"context_line":"     nc_osc_utils.LIST_LONG_ONLY),"},{"line_number":41,"context_line":"    (\u0027networks\u0027, \u0027Associated Networks\u0027, nc_osc_utils.LIST_LONG_ONLY),"},{"line_number":42,"context_line":"    (\u0027routers\u0027, \u0027Associated Routers\u0027, nc_osc_utils.LIST_LONG_ONLY),"},{"line_number":43,"context_line":"    (\u0027vni\u0027, \u0027VNI\u0027, nc_osc_utils.LIST_BOTH),"},{"line_number":44,"context_line":")"},{"line_number":45,"context_line":"_formatters \u003d {"},{"line_number":46,"context_line":"    \u0027route_targets\u0027: format_columns.ListColumn,"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f515b1d_6212f60b","line":43,"updated":"2017-09-25 14:46:03.000000000","message":"It\u0027s an administrative property, users don\u0027t care about it. Show it only on long list command.","commit_id":"069e4d9c539b0d4dc7aa0f3b1fc53cc079c53770"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"56875792bfa81563a7dd9eb5f3e36a19952bb0f1","unresolved":false,"context_lines":[{"line_number":40,"context_line":"     nc_osc_utils.LIST_LONG_ONLY),"},{"line_number":41,"context_line":"    (\u0027networks\u0027, \u0027Associated Networks\u0027, nc_osc_utils.LIST_LONG_ONLY),"},{"line_number":42,"context_line":"    (\u0027routers\u0027, \u0027Associated Routers\u0027, nc_osc_utils.LIST_LONG_ONLY),"},{"line_number":43,"context_line":"    (\u0027vni\u0027, \u0027VNI\u0027, nc_osc_utils.LIST_BOTH),"},{"line_number":44,"context_line":")"},{"line_number":45,"context_line":"_formatters \u003d {"},{"line_number":46,"context_line":"    \u0027route_targets\u0027: format_columns.ListColumn,"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f515b1d_b5870762","line":43,"in_reply_to":"7f515b1d_6212f60b","updated":"2017-09-25 15:56:16.000000000","message":"+1 good point","commit_id":"069e4d9c539b0d4dc7aa0f3b1fc53cc079c53770"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"5e5c9be6a591ce1f0e8b8cac873636ac2cd04d69","unresolved":false,"context_lines":[{"line_number":40,"context_line":"     nc_osc_utils.LIST_LONG_ONLY),"},{"line_number":41,"context_line":"    (\u0027networks\u0027, \u0027Associated Networks\u0027, nc_osc_utils.LIST_LONG_ONLY),"},{"line_number":42,"context_line":"    (\u0027routers\u0027, \u0027Associated Routers\u0027, nc_osc_utils.LIST_LONG_ONLY),"},{"line_number":43,"context_line":"    (\u0027vni\u0027, \u0027VNI\u0027, nc_osc_utils.LIST_BOTH),"},{"line_number":44,"context_line":")"},{"line_number":45,"context_line":"_formatters \u003d {"},{"line_number":46,"context_line":"    \u0027route_targets\u0027: format_columns.ListColumn,"}],"source_content_type":"text/x-python","patch_set":7,"id":"7f515b1d_8d4d0c74","line":43,"in_reply_to":"7f515b1d_6212f60b","updated":"2017-10-06 05:40:18.000000000","message":"Done","commit_id":"069e4d9c539b0d4dc7aa0f3b1fc53cc079c53770"}],"releasenotes/notes/support-vni-in-networking-bgpvpn-cli-d284b73b40b79495.yaml":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"6fd4baf2e5a583b1e9b43a08d64ca82031f7b84d","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    CLI support for VXLAN VNI ID attribute in bgpvpn."},{"line_number":5,"context_line":"    An optional argument ``--vni`` is added to ``neutron bgpvpn``"},{"line_number":6,"context_line":"    commands to configure VXLAN Network Identifier when VXLAN"},{"line_number":7,"context_line":"    encapsulation is used for the bgpvpn."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"7f96bb07_4e758363","line":5,"range":{"start_line":5,"start_character":49,"end_line":5,"end_character":63},"updated":"2018-01-16 03:18:23.000000000","message":"I think the command name is not correct. openstack","commit_id":"14c3a456aadb969b00afc972deb858a84cb477ed"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"ac952e2a3f21001df78903f75675ac66fb5dd746","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    CLI support for VXLAN VNI ID attribute in bgpvpn."},{"line_number":5,"context_line":"    An optional argument ``--vni`` is added to ``neutron bgpvpn``"},{"line_number":6,"context_line":"    commands to configure VXLAN Network Identifier when VXLAN"},{"line_number":7,"context_line":"    encapsulation is used for the bgpvpn."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"7f96bb07_e8bf9f5e","line":5,"range":{"start_line":5,"start_character":49,"end_line":5,"end_character":63},"in_reply_to":"7f96bb07_0e19bb20","updated":"2018-01-16 06:43:17.000000000","message":"Done","commit_id":"14c3a456aadb969b00afc972deb858a84cb477ed"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"0efc1ec88d2f06c382ee76743c8d2243955e6321","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    CLI support for VXLAN VNI ID attribute in bgpvpn."},{"line_number":5,"context_line":"    An optional argument ``--vni`` is added to ``neutron bgpvpn``"},{"line_number":6,"context_line":"    commands to configure VXLAN Network Identifier when VXLAN"},{"line_number":7,"context_line":"    encapsulation is used for the bgpvpn."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"7f96bb07_0e19bb20","line":5,"range":{"start_line":5,"start_character":49,"end_line":5,"end_character":63},"in_reply_to":"7f96bb07_4e758363","updated":"2018-01-16 03:19:24.000000000","message":"I posted my comment before typing a whole command.\n\"openstack bgpvpn create\"?","commit_id":"14c3a456aadb969b00afc972deb858a84cb477ed"}]}