)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5469aba035b25798b4cf38c2bffed663ba4aa177","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding a writable network-id extension for ports"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The network-id of a project\u0027s port can be changed when it"},{"line_number":10,"context_line":"has no fixed IPs on it and also the new network belongs"},{"line_number":11,"context_line":"to the same project."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I7cd0f76cbfc27f44923bba7a57839f3a5364c5d1"},{"line_number":14,"context_line":"NeutronLibImpact"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_389a635f","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":20},"updated":"2020-01-29 08:43:11.000000000","message":"This should be an rfe, and discussed with drivers team.","commit_id":"15c073aa5ae99571a1a4d06a5a5a64440dc9f09d"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"inactive contributor"},"change_message_id":"c72132a25b9daf9e626571dc2bab23e7e9fefc27","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding a writable network-id extension for ports"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The network-id of a project\u0027s port can be changed when it"},{"line_number":10,"context_line":"has no fixed IPs on it and also the new network belongs"},{"line_number":11,"context_line":"to the same project."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I7cd0f76cbfc27f44923bba7a57839f3a5364c5d1"},{"line_number":14,"context_line":"NeutronLibImpact"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_eba6220f","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":20},"in_reply_to":"3fa7e38b_389a635f","updated":"2020-01-30 12:31:13.000000000","message":"In the RFE please write about the problem you want to solve with this API change. That\u0027s the possible benefit we need to balance with the disadvantages of a change like this, just a few for example:\n\n* what if the port\u0027s MAC collides in the new network?\n* what if the old and new networks have different MTUs?\n* there are L2-only port use cases, how does the user of an L2-only port detect the network change?\n* how does a neutron API user tell when the network change got  completed in the backend?\n\nIf your intent is to change the network of a port that\u0027s not yet used, what does prevent you to delete this port and create a new instead? If that\u0027s a limitation of the orchestration system you have, I don\u0027t think that should be worked around in neutron.","commit_id":"15c073aa5ae99571a1a4d06a5a5a64440dc9f09d"}],"neutron_lib/api/definitions/port.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5469aba035b25798b4cf38c2bffed663ba4aa177","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                 \u0027is_filter\u0027: True,"},{"line_number":42,"context_line":"                 \u0027is_sort_key\u0027: True,"},{"line_number":43,"context_line":"                 \u0027is_visible\u0027: True},"},{"line_number":44,"context_line":"        PORT_NETWORK_ID: {\u0027allow_post\u0027: True, \u0027allow_put\u0027: False,"},{"line_number":45,"context_line":"                          \u0027required_by_policy\u0027: True,"},{"line_number":46,"context_line":"                          \u0027validate\u0027: {\u0027type:uuid\u0027: None},"},{"line_number":47,"context_line":"                          \u0027is_filter\u0027: True,"},{"line_number":48,"context_line":"                          \u0027is_sort_key\u0027: True,"},{"line_number":49,"context_line":"                          \u0027is_visible\u0027: True},"},{"line_number":50,"context_line":"        \u0027admin_state_up\u0027: {\u0027allow_post\u0027: True, \u0027allow_put\u0027: True,"},{"line_number":51,"context_line":"                           \u0027default\u0027: True,"},{"line_number":52,"context_line":"                           \u0027convert_to\u0027: converters.convert_to_boolean,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_988b5715","line":49,"range":{"start_line":44,"start_character":0,"end_line":49,"end_character":46},"updated":"2020-01-29 08:43:11.000000000","message":"If you have the new extension you don\u0027t need/ not allowed to modify an already existing extension","commit_id":"15c073aa5ae99571a1a4d06a5a5a64440dc9f09d"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c87978d8a31881d435e0cd734224ba1659624fd0","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                 \u0027is_filter\u0027: True,"},{"line_number":42,"context_line":"                 \u0027is_sort_key\u0027: True,"},{"line_number":43,"context_line":"                 \u0027is_visible\u0027: True},"},{"line_number":44,"context_line":"        PORT_NETWORK_ID: {\u0027allow_post\u0027: True, \u0027allow_put\u0027: False,"},{"line_number":45,"context_line":"                          \u0027required_by_policy\u0027: True,"},{"line_number":46,"context_line":"                          \u0027validate\u0027: {\u0027type:uuid\u0027: None},"},{"line_number":47,"context_line":"                          \u0027is_filter\u0027: True,"},{"line_number":48,"context_line":"                          \u0027is_sort_key\u0027: True,"},{"line_number":49,"context_line":"                          \u0027is_visible\u0027: True},"},{"line_number":50,"context_line":"        \u0027admin_state_up\u0027: {\u0027allow_post\u0027: True, \u0027allow_put\u0027: True,"},{"line_number":51,"context_line":"                           \u0027default\u0027: True,"},{"line_number":52,"context_line":"                           \u0027convert_to\u0027: converters.convert_to_boolean,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_a394a642","line":49,"range":{"start_line":44,"start_character":0,"end_line":49,"end_character":46},"in_reply_to":"3fa7e38b_988b5715","updated":"2020-02-03 16:18:13.000000000","message":"Well, if I\u0027m not wrong, this is only changing the key to this new constant. I\u0027m OK with this.","commit_id":"15c073aa5ae99571a1a4d06a5a5a64440dc9f09d"}]}