)]}'
{"neutron/extensions/ovn.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"50e0bab1351e7f0d4e59d143a26b8c878329b8ac","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":3,"context_line":"#    not use this file except in compliance with the License. You may obtain"},{"line_number":4,"context_line":"#    a copy of the License at"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_e1e6ddb0","line":1,"updated":"2019-12-04 11:45:31.000000000","message":"I\u0027m not sure if neutron/extensions is good place for this file as in this location we have API extensions defined and this isn\u0027t new API extension in fact.","commit_id":"c05731d58dc79295bb0ceb8aab34b6af3708231a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"eead7c36746accfc519e9724fee1e93805a97f89","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":3,"context_line":"#    not use this file except in compliance with the License. You may obtain"},{"line_number":4,"context_line":"#    a copy of the License at"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_818a89b9","line":1,"in_reply_to":"3fa7e38b_e1e6ddb0","updated":"2019-12-04 11:50:50.000000000","message":"I agree with that, it looks a bit off to be added here. Any suggestions ?","commit_id":"c05731d58dc79295bb0ceb8aab34b6af3708231a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c003059d321d645e4be54783970b5e80b012351b","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# NOTE(russellb) If you update these lists, please also update"},{"line_number":19,"context_line":"# doc/source/features.rst and the current release note."},{"line_number":20,"context_line":"ML2_SUPPORTED_API_EXTENSIONS_NEUTRON_L3 \u003d ["},{"line_number":21,"context_line":"    \u0027dns-integration\u0027,"},{"line_number":22,"context_line":"    \u0027dvr\u0027,"},{"line_number":23,"context_line":"    \u0027extraroute\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_73f08e92","line":20,"updated":"2019-12-03 18:26:01.000000000","message":"This variable is not used","commit_id":"c05731d58dc79295bb0ceb8aab34b6af3708231a"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"50e0bab1351e7f0d4e59d143a26b8c878329b8ac","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# NOTE(russellb) If you update these lists, please also update"},{"line_number":19,"context_line":"# doc/source/features.rst and the current release note."},{"line_number":20,"context_line":"ML2_SUPPORTED_API_EXTENSIONS_NEUTRON_L3 \u003d ["},{"line_number":21,"context_line":"    \u0027dns-integration\u0027,"},{"line_number":22,"context_line":"    \u0027dvr\u0027,"},{"line_number":23,"context_line":"    \u0027extraroute\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_8147e9a3","line":20,"in_reply_to":"3fa7e38b_73f08e92","updated":"2019-12-04 11:45:31.000000000","message":"this is used in devstack/lib/ovn_agent\nbut I don\u0027t know if it will not be used also in other places in code too","commit_id":"c05731d58dc79295bb0ceb8aab34b6af3708231a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"eead7c36746accfc519e9724fee1e93805a97f89","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# NOTE(russellb) If you update these lists, please also update"},{"line_number":19,"context_line":"# doc/source/features.rst and the current release note."},{"line_number":20,"context_line":"ML2_SUPPORTED_API_EXTENSIONS_NEUTRON_L3 \u003d ["},{"line_number":21,"context_line":"    \u0027dns-integration\u0027,"},{"line_number":22,"context_line":"    \u0027dvr\u0027,"},{"line_number":23,"context_line":"    \u0027extraroute\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_e1b49d84","line":20,"in_reply_to":"3fa7e38b_8147e9a3","updated":"2019-12-04 11:50:50.000000000","message":"Yeah, only used in the devstack module","commit_id":"c05731d58dc79295bb0ceb8aab34b6af3708231a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0a1b405c70c82565cc9888a02c1eacf8bbfb5975","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# NOTE(russellb) If you update these lists, please also update"},{"line_number":19,"context_line":"# doc/source/features.rst and the current release note."},{"line_number":20,"context_line":"ML2_SUPPORTED_API_EXTENSIONS_NEUTRON_L3 \u003d ["},{"line_number":21,"context_line":"    \u0027dns-integration\u0027,"},{"line_number":22,"context_line":"    \u0027dvr\u0027,"},{"line_number":23,"context_line":"    \u0027extraroute\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_1cfcbe0a","line":20,"in_reply_to":"3fa7e38b_e1b49d84","updated":"2019-12-04 11:58:18.000000000","message":"Sorry, I still don\u0027t see it:\n\nhttp://codesearch.openstack.org/?q\u003dML2_SUPPORTED_API_EXTENSIONS_NEUTRON_L3\u0026i\u003dnope\u0026files\u003d\u0026repos\u003d","commit_id":"c05731d58dc79295bb0ceb8aab34b6af3708231a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c003059d321d645e4be54783970b5e80b012351b","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    \u0027project-id\u0027,"},{"line_number":37,"context_line":"    \u0027dns-integration\u0027,"},{"line_number":38,"context_line":"]"},{"line_number":39,"context_line":"ML2_SUPPORTED_API_EXTENSIONS \u003d ["},{"line_number":40,"context_line":"    \u0027address-scope\u0027,"},{"line_number":41,"context_line":"    \u0027agent\u0027,"},{"line_number":42,"context_line":"    \u0027allowed-address-pairs\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_b3528667","line":39,"range":{"start_line":39,"start_character":0,"end_line":39,"end_character":28},"updated":"2019-12-03 18:26:01.000000000","message":"Why do we need this one?","commit_id":"c05731d58dc79295bb0ceb8aab34b6af3708231a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"eead7c36746accfc519e9724fee1e93805a97f89","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    \u0027project-id\u0027,"},{"line_number":37,"context_line":"    \u0027dns-integration\u0027,"},{"line_number":38,"context_line":"]"},{"line_number":39,"context_line":"ML2_SUPPORTED_API_EXTENSIONS \u003d ["},{"line_number":40,"context_line":"    \u0027address-scope\u0027,"},{"line_number":41,"context_line":"    \u0027agent\u0027,"},{"line_number":42,"context_line":"    \u0027allowed-address-pairs\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_a1aea52c","line":39,"range":{"start_line":39,"start_character":0,"end_line":39,"end_character":28},"in_reply_to":"3fa7e38b_a1d785c0","updated":"2019-12-04 11:50:50.000000000","message":"Sure, I can work on that!","commit_id":"c05731d58dc79295bb0ceb8aab34b6af3708231a"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"50e0bab1351e7f0d4e59d143a26b8c878329b8ac","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    \u0027project-id\u0027,"},{"line_number":37,"context_line":"    \u0027dns-integration\u0027,"},{"line_number":38,"context_line":"]"},{"line_number":39,"context_line":"ML2_SUPPORTED_API_EXTENSIONS \u003d ["},{"line_number":40,"context_line":"    \u0027address-scope\u0027,"},{"line_number":41,"context_line":"    \u0027agent\u0027,"},{"line_number":42,"context_line":"    \u0027allowed-address-pairs\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_a1d785c0","line":39,"range":{"start_line":39,"start_character":0,"end_line":39,"end_character":28},"in_reply_to":"3fa7e38b_b3528667","updated":"2019-12-04 11:45:31.000000000","message":"same here.\nBut IMO if this is only used for devstack module, maybe we could simply list all those extensions in devstack/lib/ovn_agent and drop this file completly?","commit_id":"c05731d58dc79295bb0ceb8aab34b6af3708231a"}]}