)]}'
{"docker/neutron/neutron-base/Dockerfile.j2":[{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"eb27b7aac8de6824c84e73e47051d03589f4b5c9","unresolved":false,"context_lines":[{"line_number":137,"context_line":"        {{ macros.install_pip(neutron_base_plugins_pip_packages) }} \\"},{"line_number":138,"context_line":"        \u0026\u0026 for neutron_plugins in $(ls /plugins); do \\"},{"line_number":139,"context_line":"            if [[ -d \"/plugins/$neutron_plugins/etc/neutron/rootwrap.d\" ]]; then \\"},{"line_number":140,"context_line":"                cp  /plugins/$neutron_plugins/etc/neutron/rootwrap.d/* /etc/neutron/rootwrap.d; \\"},{"line_number":141,"context_line":"            fi; \\"},{"line_number":142,"context_line":"        done; \\"},{"line_number":143,"context_line":"    fi"}],"source_content_type":"text/x-jinja2","patch_set":10,"id":"3fa0c359_ec919e48","line":140,"range":{"start_line":140,"start_character":19,"end_line":140,"end_character":20},"updated":"2018-02-13 12:08:46.000000000","message":"too space","commit_id":"e50bb500ee710336feb299b6e8cad0b816c542ba"}],"docker/neutron/neutron-l3-agent/Dockerfile.j2":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8fb661575bf49864e84c8cb392103a329e8b1ef6","unresolved":false,"context_lines":[{"line_number":40,"context_line":"    \u0027/neutron_vpnaas\u0027"},{"line_number":41,"context_line":"] %}"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"RUN git clone https://github.com/openstack/neutron-vpnaas /neutron-vpnaas-agent-source \\"},{"line_number":44,"context_line":"    \u0026\u0026 ln -s neutron-vpnaas-agent-source/* neutron_vpnaas \\"},{"line_number":45,"context_line":"    \u0026\u0026 {{ macros.install_pip(neutron_vpnaas_agent_pip_packages | customizable(\"pip_packages\")) }} \\"},{"line_number":46,"context_line":"    \u0026\u0026 cp /neutron_vpnaas/etc/neutron/rootwrap.d/* /etc/neutron/rootwrap.d"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"9f91af0f_5e7d8daf","line":43,"updated":"2018-01-02 02:38:36.000000000","message":"please use the kolla plugin mechanism\n\nhttps://github.com/openstack/kolla/blob/master/kolla/common/config.py#L518-L521","commit_id":"1a9cb2b49a8c0e4f2d9aec0eb782534616e6b8b8"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"cf9bb48b8506dbb7b2f540d7186058250ba8a20a","unresolved":false,"context_lines":[{"line_number":44,"context_line":"# TODO(pbourke): The libreswan driver required by redhat derived bases"},{"line_number":45,"context_line":"# currently requires root in Neutron. Remove this if condition once"},{"line_number":46,"context_line":"# https://bugs.launchpad.net/neutron/+bug/1644517 is addressed."},{"line_number":47,"context_line":"{% if base_distro in [\u0027debian\u0027, \u0027ubuntu\u0027] %}"},{"line_number":48,"context_line":"USER neutron"},{"line_number":49,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"7f96bb07_d2f4465e","line":49,"range":{"start_line":47,"start_character":0,"end_line":49,"end_character":11},"updated":"2018-01-11 07:27:18.000000000","message":"will surely cause issues to users upgrading from pike when user owning resources is neutron and changed to root","commit_id":"09ba38ca788f935a14c00ebc0bf659d2353db6db"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"511760440bc3e2806954520a07bb2ba04e090e75","unresolved":false,"context_lines":[{"line_number":44,"context_line":"# TODO(pbourke): The libreswan driver required by redhat derived bases"},{"line_number":45,"context_line":"# currently requires root in Neutron. Remove this if condition once"},{"line_number":46,"context_line":"# https://bugs.launchpad.net/neutron/+bug/1644517 is addressed."},{"line_number":47,"context_line":"{% if base_distro in [\u0027debian\u0027, \u0027ubuntu\u0027] %}"},{"line_number":48,"context_line":"USER neutron"},{"line_number":49,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"7f96bb07_31b7e13c","line":49,"range":{"start_line":47,"start_character":0,"end_line":49,"end_character":11},"in_reply_to":"7f96bb07_5f7b3d8b","updated":"2018-01-18 07:37:13.000000000","message":"Since this is a bug in vpnaas bug, i think we can leave a TODO/FIXME here and fix the issue on the vpnaas side.","commit_id":"09ba38ca788f935a14c00ebc0bf659d2353db6db"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"0fa96596a143cde3eb5605ffa1a7da232208fc8a","unresolved":false,"context_lines":[{"line_number":44,"context_line":"# TODO(pbourke): The libreswan driver required by redhat derived bases"},{"line_number":45,"context_line":"# currently requires root in Neutron. Remove this if condition once"},{"line_number":46,"context_line":"# https://bugs.launchpad.net/neutron/+bug/1644517 is addressed."},{"line_number":47,"context_line":"{% if base_distro in [\u0027debian\u0027, \u0027ubuntu\u0027] %}"},{"line_number":48,"context_line":"USER neutron"},{"line_number":49,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"7f96bb07_5f7b3d8b","line":49,"range":{"start_line":47,"start_character":0,"end_line":49,"end_character":11},"in_reply_to":"7f96bb07_d2f4465e","updated":"2018-01-12 07:42:34.000000000","message":"guess no workaround here.\n\nand run this as ROOT should be OK, i guess....","commit_id":"09ba38ca788f935a14c00ebc0bf659d2353db6db"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"eb27b7aac8de6824c84e73e47051d03589f4b5c9","unresolved":false,"context_lines":[{"line_number":6,"context_line":"{% import \"macros.j2\" as macros with context %}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":9,"context_line":"   {% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":10,"context_line":"       {% set neutron_l3_agent_packages \u003d ["},{"line_number":11,"context_line":"            \u0027openstack-neutron-vpn-agent\u0027,"},{"line_number":12,"context_line":"            \u0027libreswan\u0027"}],"source_content_type":"text/x-jinja2","patch_set":10,"id":"3fa0c359_ccb702f5","line":9,"range":{"start_line":9,"start_character":27,"end_line":9,"end_character":55},"updated":"2018-02-13 12:08:46.000000000","message":"please alphabet it","commit_id":"e50bb500ee710336feb299b6e8cad0b816c542ba"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"eb27b7aac8de6824c84e73e47051d03589f4b5c9","unresolved":false,"context_lines":[{"line_number":22,"context_line":"{{ macros.install_packages(neutron_l3_agent_packages | customizable(\"packages\")) }}"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"{% elif install_type \u003d\u003d \u0027source\u0027 %}"},{"line_number":25,"context_line":"   {% if base_distro in [\u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":26,"context_line":"       {% set neutron_l3_agent_packages \u003d ["},{"line_number":27,"context_line":"            \u0027libreswan\u0027,"},{"line_number":28,"context_line":"            \u0027radvd\u0027"}],"source_content_type":"text/x-jinja2","patch_set":10,"id":"3fa0c359_8ccd7a86","line":25,"range":{"start_line":25,"start_character":27,"end_line":25,"end_character":60},"updated":"2018-02-13 12:08:46.000000000","message":"ditto","commit_id":"e50bb500ee710336feb299b6e8cad0b816c542ba"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"eb27b7aac8de6824c84e73e47051d03589f4b5c9","unresolved":false,"context_lines":[{"line_number":42,"context_line":"{% block footer %}{% endblock %}"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"USER neutron"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":10,"id":"3fa0c359_4cc3f257","line":45,"range":{"start_line":45,"start_character":0,"end_line":45,"end_character":0},"updated":"2018-02-13 12:08:46.000000000","message":"unnecessary blank line, better remove it","commit_id":"e50bb500ee710336feb299b6e8cad0b816c542ba"},{"author":{"_account_id":19853,"name":"Nguyen Hung Phuong","email":"phuongnh@vn.fujitsu.com","username":"phuongnh"},"change_message_id":"3cdad6b5a2f95f8df18ca0bab6f2f703bc3bc940","unresolved":false,"context_lines":[{"line_number":41,"context_line":"{% block neutron_l3_agent_footer %}{% endblock %}"},{"line_number":42,"context_line":"{% block footer %}{% endblock %}"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"USER neutron"}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"3fa0c359_0d243a5f","line":44,"range":{"start_line":44,"start_character":0,"end_line":44,"end_character":12},"updated":"2018-02-22 03:48:18.000000000","message":"I downloaded the file and do not found new line at the bottom. Please add to fix pep8 issue.","commit_id":"eb7e10efcd8f8dd7d7aed55a55ee72840bc77e0d"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"2ba4e418949db1c2d9a2fd4f1b1e5297b26fe5c0","unresolved":false,"context_lines":[{"line_number":13,"context_line":"       ] %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"# TODO(chenxing) Currently, there is no Ubuntu vpnaas package for Queens release."},{"line_number":16,"context_line":"# Add Ubuntu binary install_type after it releases."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"   {% endif %}"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"df7087c5_d01f9d05","line":16,"updated":"2018-03-15 07:08:11.000000000","message":"at lease, you need install \"neutron-l3-agent\" package. like\n\n    # TODO(xxx): there is no vpnaas package for queens release.\n    {% set neutron_l3_agent_packages \u003d [\n        \u0027neutron-l3-agent\u0027\n    ] %}","commit_id":"80672717c509a5f9e2c6f92d7a5f90dc081d9a62"}],"docker/neutron/neutron-l3-agent/extend_start.sh":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8fb661575bf49864e84c8cb392103a329e8b1ef6","unresolved":false,"context_lines":[{"line_number":5,"context_line":"if [[ \"${!KOLLA_BOOTSTRAP[@]}\" ]]; then"},{"line_number":6,"context_line":"    neutron-db-manage --subproject neutron-vpnaas --config-file /etc/neutron/neutron.conf upgrade head"},{"line_number":7,"context_line":"    exit 0"},{"line_number":8,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":5,"id":"9f91af0f_de4afd7e","line":8,"updated":"2018-01-02 02:38:36.000000000","message":"i think no need this here.\n\n1. neutron-l3-agent image hasn\u0027t bootstrap progress.\n2. bootstrap neutron-server image will run this automatically.","commit_id":"1a9cb2b49a8c0e4f2d9aec0eb782534616e6b8b8"}],"docker/neutron/neutron-vpnaas-agent/Dockerfile.j2":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"fea631729aa7a616fd9cbe68c9d6f50dfa01ec49","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# TODO(chenxing)  As neutron-vpnaas-agent is loaded just inside of the"},{"line_number":2,"context_line":"# existing l3 agent rather than requiring operators to run a completely"},{"line_number":3,"context_line":"# different binary, this Dockerfile will be removed in Rocky release."},{"line_number":4,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":15,"id":"ff6b8bd7_c39a3c6d","line":1,"updated":"2018-03-05 09:04:53.000000000","message":"This file should be removed.","commit_id":"584e5cc31b2e2708e51389df6883ee35ab694bc1"}],"releasenotes/notes/remove-neutron-vpnaas-agent-216810affb495ad0.yaml":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8fb661575bf49864e84c8cb392103a329e8b1ef6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    The neutron-vpnaas-agent has been loaded just inside of the existing l3 agent"},{"line_number":5,"context_line":"    rather than requiring operators to run a completely different binary with a"},{"line_number":6,"context_line":"    subclass of the existing L3 agent. We will remove the neutron-vpnaas-agent"},{"line_number":7,"context_line":"    Dockerfile in Rocky release."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f91af0f_1e514537","line":7,"updated":"2018-01-02 02:38:36.000000000","message":"i think we need add a deprecation section here.","commit_id":"1a9cb2b49a8c0e4f2d9aec0eb782534616e6b8b8"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"b62ef79a2ab1061a4c27058369f9283126dfd9a9","unresolved":false,"context_lines":[{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    As neutron-vpnaas-agent can be loaded by the neutron l3 agent, there is no need"},{"line_number":11,"context_line":"    to keep its Dockerfile. We will remove the neutron-vpnaas-agent Dockerfile in"},{"line_number":12,"context_line":"    Rocky release."}],"source_content_type":"text/x-yaml","patch_set":20,"id":"ff6b8bd7_23519fd9","line":12,"updated":"2018-03-06 13:14:43.000000000","message":"Since vpnaas standalone mode is not support since queens. It is not proper to say that will remove it in rocky release.\n\nJust say vpnaas standalone mode is not support it and remove the containers.","commit_id":"fe8629da3e8029869304e7ec13bfd666d3b3ae8d"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"443783da0fc1b0c0a71d7f899581e7d4577a96d8","unresolved":false,"context_lines":[{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    As neutron-vpnaas-agent can be loaded by the neutron l3 agent, there is no need"},{"line_number":11,"context_line":"    to keep its Dockerfile. We will remove the neutron-vpnaas-agent Dockerfile in"},{"line_number":12,"context_line":"    Rocky release."}],"source_content_type":"text/x-yaml","patch_set":20,"id":"df7087c5_30d6192a","line":12,"updated":"2018-03-15 07:08:28.000000000","message":"we should not use the deprecation policy. the neutron vpnaas standalone mode is\nalready removed since queens. So just say \"remove neutron-vpnaas-agent image\"","commit_id":"fe8629da3e8029869304e7ec13bfd666d3b3ae8d"}]}