)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"23865cc5674eb002bc97ef0a2c399235bd24192c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This implements the neutron-vpnaas-agent container for Ubuntu/Debian"},{"line_number":10,"context_line":"with installation from source packages."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"blueprint neutron-vpnaas-agent"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I3e4b513fb8e1670c4df0edb4951610f777ff72e6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3aaa91ec_2ddf143b","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":30},"updated":"2016-06-30 11:51:13.000000000","message":"Implements: blueprint neutron-vpnaas-agent","commit_id":"276e13ce30b763b23421cefac14f49f67bc0ce1a"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"80d0c94dfcebb159e72bb09f11ac06c2bb09d60d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add neutron-vpnaas-agent container"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This implements the neutron-vpnaas-agent container for Ubuntu/Debian"},{"line_number":10,"context_line":"with installation from source packages."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Implements: blueprint neutron-vpnaas-agent"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":22,"id":"7a8ec9b2_f85a8329","line":9,"updated":"2016-09-14 21:46:08.000000000","message":"CentOS should also be supported i think.","commit_id":"09a873fb74eafe279391e62a5dba8aba216e10fa"}],"docker/neutron/neutron-base/Dockerfile.j2":[{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"ae3d57a8bf39e8814b3e01c2da85ae6ba35b2ffa","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# TODO(Jeffrey4l): UCA latest Newton release lack of neutron-lbaas-common"},{"line_number":20,"context_line":"# package, just remove neutron-lbaas-agent in neutron_base_packages. Once they"},{"line_number":21,"context_line":"# fix it, this should be reverted."},{"line_number":22,"context_line":"        {% set neutron_base_packages \u003d ["},{"line_number":23,"context_line":"            \u0027iproute2\u0027,"},{"line_number":24,"context_line":"            \u0027neutron-plugin-ml2\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":29,"id":"7a8ec9b2_c40fc0df","line":21,"range":{"start_line":19,"start_character":0,"end_line":21,"end_character":34},"updated":"2016-09-14 13:35:15.000000000","message":"Can this TODO be removed now?","commit_id":"b00530cc54a0774522f90c3e65a978fe04a3ac68"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"fe54743fb012d55689c9c567ba5b17014c1c56a8","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    {% elif base_distro in [\u0027ubuntu\u0027] %}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# TODO(Jeffrey4l): UCA latest Newton release lack of neutron-lbaas-common"},{"line_number":20,"context_line":"# package, just remove neutron-lbaas-agent in neutron_base_packages. Once they"},{"line_number":21,"context_line":"# fix it, this should be reverted."},{"line_number":22,"context_line":"        {% set neutron_base_packages \u003d ["},{"line_number":23,"context_line":"            \u0027iproute2\u0027,"},{"line_number":24,"context_line":"            \u0027neutron-plugin-ml2\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":29,"id":"7a8ec9b2_aa799d95","line":21,"range":{"start_line":19,"start_character":0,"end_line":21,"end_character":34},"in_reply_to":"7a8ec9b2_c40fc0df","updated":"2016-09-14 13:56:41.000000000","message":"Done","commit_id":"b00530cc54a0774522f90c3e65a978fe04a3ac68"}],"docker/neutron/neutron-vpnaas-agent/Dockerfile.j2":[{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"3c61f53b0fdf2bc038a93bda82c279d2bed9a9c1","unresolved":false,"context_lines":[{"line_number":26,"context_line":"ADD neutron-vpnaas-agent-archive /neutron-vpnaas-agent-source"},{"line_number":27,"context_line":"RUN ln -s neutron-vpnaas-agent-source/* neutron_vpnaas \\"},{"line_number":28,"context_line":"    \u0026\u0026 /var/lib/kolla/venv/bin/pip --no-cache-dir install --upgrade -c requirements/upper-constraints.txt /neutron_vpnaas \\"},{"line_number":29,"context_line":"    \u0026\u0026 cp /neutron_vpnaas/etc/neutron/rootwrap.d/* /etc/neutron/rootwrap.d"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"{% endif %}"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"1aa78d24_ebb3f108","line":29,"updated":"2016-07-01 22:02:30.000000000","message":"\u0026 chown -R neutron: /etc/neutron","commit_id":"d2d0bcc578de94e2e1796d119a68ab23510ac99f"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"56c5d469b323738c5be83cd4541b64684e77abae","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}neutron-base:{{ tag }}"},{"line_number":2,"context_line":"MAINTAINER {{ maintainer }}"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% if base_distro in [\u0027centos\u0027, \u0027fedora\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"bacf61ea_431a2018","line":1,"updated":"2016-08-01 11:20:21.000000000","message":"We are now introducing Dockerfile customization, please adapt this Dockerfile to this pattern. https://blueprints.launchpad.net/kolla/+spec/third-party-plugin-support","commit_id":"31e140e9d0fd652952ac4cbbafb26eb6d1d77776"},{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"077476a7f64ba6224217507c5b4571f1de4f5773","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# FIXME: installation from binary packages has to be implemented"},{"line_number":20,"context_line":"RUN echo \u0027{{ install_type }} not yet available for {{ base_distro }}\u0027 \\"},{"line_number":21,"context_line":"    \u0026\u0026 /bin/false"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":17,"id":"1ac06dbe_984e0195","line":19,"range":{"start_line":19,"start_character":2,"end_line":19,"end_character":7},"updated":"2016-08-19 12:10:28.000000000","message":"IMO, it will be better to use  # TODO(name):","commit_id":"0b22fbdf8f9ebdb6a565ef8bae86e56315e1d18c"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"625866049179b5528a49ee77bcad8e10fd9a77ad","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# FIXME: installation from binary packages has to be implemented"},{"line_number":20,"context_line":"RUN echo \u0027{{ install_type }} not yet available for {{ base_distro }}\u0027 \\"},{"line_number":21,"context_line":"    \u0026\u0026 /bin/false"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":17,"id":"1ac06dbe_a824cfc4","line":19,"range":{"start_line":19,"start_character":2,"end_line":19,"end_character":7},"in_reply_to":"1ac06dbe_984e0195","updated":"2016-08-22 20:29:05.000000000","message":"TODO(berendt) is wrong here because I will not fix it in the future... :)","commit_id":"0b22fbdf8f9ebdb6a565ef8bae86e56315e1d18c"},{"author":{"_account_id":19370,"name":"Prithiv","email":"prithiv.mohan@intel.com","username":"prithiv"},"change_message_id":"980101a8e77d19e5151dd05ff8da3581e76c3488","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN {{ macros.install_packages(neutron_vpnaas_agent_packages | customizable(\"packages\")) }}"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":18,"id":"fa7ab95a_96078080","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":91},"updated":"2016-09-01 09:00:20.000000000","message":"Is \u0027RUN\u0027 really required here ? I believe that is handled by macros.j2 file","commit_id":"8da19f08ce80e72b38f1904eadeb59b42a1268cc"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"34f0e7c8a6799172f199a7fc7bc8c3fd04784a74","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN {{ macros.install_packages(neutron_vpnaas_agent_packages | customizable(\"packages\")) }}"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":18,"id":"fa7ab95a_7982ef53","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":91},"in_reply_to":"fa7ab95a_96078080","updated":"2016-09-01 09:05:52.000000000","message":"Yes, good catch, this was changed after I prepared this change...","commit_id":"8da19f08ce80e72b38f1904eadeb59b42a1268cc"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"f91fcc3722a138a0470dc4d2a637d91bf1621f62","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN {{ macros.install_packages(neutron_vpnaas_agent_packages | customizable(\"packages\")) }}"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"{% if install_type \u003d\u003d \u0027binary\u0027 %}"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":25,"id":"9a89bdaa_626dbd03","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":4},"updated":"2016-09-12 13:39:09.000000000","message":"Please, remove RUN from macros install, RUN command is included in the macro itself, this way can handle empty package definitions.","commit_id":"f75d9a117eb3f940c53cb840119a3076c72836c3"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"05afee1e3a129cd957a16b05d8c31d99c12f393c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}neutron-base:{{ tag }}"},{"line_number":2,"context_line":"MAINTAINER {{ maintainer }}"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% block neutorn_vpnass_agent__header %}{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% import \"macros.j2\" as macros with context %}"},{"line_number":7,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":27,"id":"9a89bdaa_fafbe0be","line":4,"range":{"start_line":4,"start_character":29,"end_line":4,"end_character":31},"updated":"2016-09-13 16:36:07.000000000","message":"one _ too many","commit_id":"837d97aaf69f72ad07c833f20b00ae358252be7a"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"05afee1e3a129cd957a16b05d8c31d99c12f393c","unresolved":false,"context_lines":[{"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, \u0027fedora\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":10,"context_line":"       {% set neutron_vpnaas_agent_packages \u003d [ "},{"line_number":11,"context_line":"            \u0027openswan\u0027,"},{"line_number":12,"context_line":"            \u0027openstack-neutron-vpn-agent\u0027"},{"line_number":13,"context_line":"       ] %}"}],"source_content_type":"text/x-jinja2","patch_set":27,"id":"9a89bdaa_fa74c068","line":10,"range":{"start_line":10,"start_character":47,"end_line":10,"end_character":48},"updated":"2016-09-13 16:36:07.000000000","message":"whitespace","commit_id":"837d97aaf69f72ad07c833f20b00ae358252be7a"}],"kolla/common/config.py":[{"author":{"_account_id":10419,"name":"Ryan Hallisey","email":"rhallise@redhat.com","username":"rhallisey"},"change_message_id":"c3eee0f00e19dd7c627d933f5fa7f96eb4dce664","unresolved":false,"context_lines":[{"line_number":238,"context_line":"        \u0027type\u0027: \u0027url\u0027,"},{"line_number":239,"context_line":"        \u0027location\u0027: (\u0027http://tarballs.openstack.org/neutron-vpnaas/\u0027"},{"line_number":240,"context_line":"                     \u0027neutron-vpnaas-master.tar.gz\u0027)},"},{"line_number":241,"context_line":"    \u0027neutron-server-plugin-neutron-vpnaas\u0027: {"},{"line_number":242,"context_line":"        \u0027type\u0027: \u0027url\u0027,"},{"line_number":243,"context_line":"        \u0027location\u0027: (\u0027http://tarballs.openstack.org/neutron-vpnaas/\u0027"},{"line_number":244,"context_line":"                     \u0027neutron-vpnaas-master.tar.gz\u0027)},"}],"source_content_type":"text/x-python","patch_set":8,"id":"bacf61ea_0f29699e","line":241,"updated":"2016-08-01 12:47:04.000000000","message":"You should only have to include the neutron-vpnaas source repo once.","commit_id":"8d7f4ea72affaeb36e2853e90cef3a80f58741bb"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"e6b34834d7d6e4c7a2c4aa5ea402cd40bfbde1e7","unresolved":false,"context_lines":[{"line_number":238,"context_line":"        \u0027type\u0027: \u0027url\u0027,"},{"line_number":239,"context_line":"        \u0027location\u0027: (\u0027http://tarballs.openstack.org/neutron-vpnaas/\u0027"},{"line_number":240,"context_line":"                     \u0027neutron-vpnaas-master.tar.gz\u0027)},"},{"line_number":241,"context_line":"    \u0027neutron-server-plugin-neutron-vpnaas\u0027: {"},{"line_number":242,"context_line":"        \u0027type\u0027: \u0027url\u0027,"},{"line_number":243,"context_line":"        \u0027location\u0027: (\u0027http://tarballs.openstack.org/neutron-vpnaas/\u0027"},{"line_number":244,"context_line":"                     \u0027neutron-vpnaas-master.tar.gz\u0027)},"}],"source_content_type":"text/x-python","patch_set":8,"id":"bacf61ea_369abdb9","line":241,"in_reply_to":"bacf61ea_0f29699e","updated":"2016-08-01 14:39:24.000000000","message":"This entry is required to install the neutron-vpnaas sources inside the neutron-server image.","commit_id":"8d7f4ea72affaeb36e2853e90cef3a80f58741bb"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"d54e698aa1da469b4c0dcfa7fce005b0886733d4","unresolved":false,"context_lines":[{"line_number":249,"context_line":"        \u0027type\u0027: \u0027url\u0027,"},{"line_number":250,"context_line":"        \u0027location\u0027: (\u0027http://tarballs.openstack.org/neutron-vpnaas/\u0027"},{"line_number":251,"context_line":"                     \u0027neutron-vpnaas-master.tar.gz\u0027)},"},{"line_number":252,"context_line":"    \u0027neutron-server-plugin-neutron-vpnaas\u0027: {"},{"line_number":253,"context_line":"        \u0027type\u0027: \u0027url\u0027,"},{"line_number":254,"context_line":"        \u0027location\u0027: (\u0027http://tarballs.openstack.org/neutron-vpnaas/\u0027"},{"line_number":255,"context_line":"                     \u0027neutron-vpnaas-master.tar.gz\u0027)},"},{"line_number":256,"context_line":"    \u0027nova-base\u0027: {"},{"line_number":257,"context_line":"        \u0027type\u0027: \u0027url\u0027,"},{"line_number":258,"context_line":"        \u0027location\u0027: (\u0027http://tarballs.openstack.org/nova/\u0027"}],"source_content_type":"text/x-python","patch_set":28,"id":"7a8ec9b2_80794432","line":255,"range":{"start_line":252,"start_character":4,"end_line":255,"end_character":54},"updated":"2016-09-14 03:05:08.000000000","message":"Why do we need neutron-server-plugin-neutron-vpnass? If so the tar ball file is incorrect.","commit_id":"f8ef2ed2f24111164891036298b41bc64da0fa1e"}]}