)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"9a28715d9db4572b58b107e0d9bc949b142d5615","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Chao Guo \u003cguochao@winhong.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-24 10:48:39 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding strongswan/openswan to source branch of neutron-vpnaas-agent"},{"line_number":8,"context_line":"docker image."},{"line_number":9,"context_line":"Fixes Bug1644159"},{"line_number":10,"context_line":"Closes-Bug:1644158"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a74a57a_0b328410","line":7,"updated":"2016-11-24 10:46:27.000000000","message":"Please follow commit message guidelines:\n\n    \u003c short title \u003e\n\n    \u003c body \u003e\n\n    \u003c metadata \u003e\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"f5080886037c3f0c17f0310e43b984fb5a63aaaf","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Chao Guo \u003cguochao@winhong.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-24 10:48:39 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding strongswan/openswan to source branch of neutron-vpnaas-agent"},{"line_number":8,"context_line":"docker image."},{"line_number":9,"context_line":"Fixes Bug1644159"},{"line_number":10,"context_line":"Closes-Bug:1644158"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a74a57a_b73276f0","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":13},"updated":"2016-11-28 09:45:15.000000000","message":"Reduce commit message to a maximum of 52 characters.\nAlso add a commit description","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"f5080886037c3f0c17f0310e43b984fb5a63aaaf","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding strongswan/openswan to source branch of neutron-vpnaas-agent"},{"line_number":8,"context_line":"docker image."},{"line_number":9,"context_line":"Fixes Bug1644159"},{"line_number":10,"context_line":"Closes-Bug:1644158"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Id08e60c7132dd4e8723e588dc1dab20cfc541722"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a74a57a_572da3a6","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":16},"updated":"2016-11-28 09:45:15.000000000","message":"Please, remove this line","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"c413c95dd223b48ea0b612b4734fbdab5b39f4a2","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adding strongswan/openswan to source branch of neutron-vpnaas-agent"},{"line_number":8,"context_line":"docker image."},{"line_number":9,"context_line":"Fixes Bug1644159"},{"line_number":10,"context_line":"Closes-Bug:1644158"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Id08e60c7132dd4e8723e588dc1dab20cfc541722"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a74a57a_450f042e","line":10,"range":{"start_line":10,"start_character":11,"end_line":10,"end_character":18},"updated":"2016-11-24 12:51:08.000000000","message":"This bug id is wrong, please reference the one you logged originally","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"f5080886037c3f0c17f0310e43b984fb5a63aaaf","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adding strongswan/openswan to source branch of neutron-vpnaas-agent"},{"line_number":8,"context_line":"docker image."},{"line_number":9,"context_line":"Fixes Bug1644159"},{"line_number":10,"context_line":"Closes-Bug:1644158"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Id08e60c7132dd4e8723e588dc1dab20cfc541722"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a74a57a_570cc23a","line":10,"range":{"start_line":10,"start_character":11,"end_line":10,"end_character":18},"in_reply_to":"5a74a57a_253be8d2","updated":"2016-11-28 09:45:15.000000000","message":"Yes, bug id is wrong use the correct one #1643755","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"84110c173fe4887b00aa5a9617cc9ebaea6e8397","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adding strongswan/openswan to source branch of neutron-vpnaas-agent"},{"line_number":8,"context_line":"docker image."},{"line_number":9,"context_line":"Fixes Bug1644159"},{"line_number":10,"context_line":"Closes-Bug:1644158"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Id08e60c7132dd4e8723e588dc1dab20cfc541722"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a74a57a_253be8d2","line":10,"range":{"start_line":10,"start_character":11,"end_line":10,"end_character":18},"in_reply_to":"5a74a57a_450f042e","updated":"2016-11-24 12:51:45.000000000","message":"1643755","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"}],"docker/neutron/neutron-vpnaas-agent/Dockerfile.j2":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"f5080886037c3f0c17f0310e43b984fb5a63aaaf","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    \u0026\u0026 /var/lib/kolla/venv/bin/pip --no-cache-dir install --upgrade -c requirements/upper-constraints.txt /neutron_vpnaas \\"},{"line_number":38,"context_line":"    \u0026\u0026 cp /neutron_vpnaas/etc/neutron/rootwrap.d/* /etc/neutron/rootwrap.d"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"{{ macros.install_packages(neutron_vpnaas_agent_packages | customizable(\"packages\")) }}"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"{% endif %}"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5a74a57a_575a022b","line":40,"range":{"start_line":40,"start_character":0,"end_line":40,"end_character":87},"updated":"2016-11-28 09:45:15.000000000","message":"Maybe makes more sense to put it before source code install","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"9a28715d9db4572b58b107e0d9bc949b142d5615","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    \u0026\u0026 /var/lib/kolla/venv/bin/pip --no-cache-dir install --upgrade -c requirements/upper-constraints.txt /neutron_vpnaas \\"},{"line_number":38,"context_line":"    \u0026\u0026 cp /neutron_vpnaas/etc/neutron/rootwrap.d/* /etc/neutron/rootwrap.d"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"{{ macros.install_packages(neutron_vpnaas_agent_packages | customizable(\"packages\")) }}"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"{% endif %}"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5a74a57a_8b7854c6","line":40,"range":{"start_line":40,"start_character":27,"end_line":40,"end_character":56},"updated":"2016-11-24 10:46:27.000000000","message":"Please change the name of this variable so it doesn\u0027t clash with the one above if users wish to override","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"6453913606385c295f04460f4b3742a36b19859a","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    \u0026\u0026 /var/lib/kolla/venv/bin/pip --no-cache-dir install --upgrade -c requirements/upper-constraints.txt /neutron_vpnaas \\"},{"line_number":38,"context_line":"    \u0026\u0026 cp /neutron_vpnaas/etc/neutron/rootwrap.d/* /etc/neutron/rootwrap.d"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"{{ macros.install_packages(neutron_vpnaas_agent_packages | customizable(\"packages\")) }}"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"{% endif %}"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5a74a57a_fab7cff2","line":40,"range":{"start_line":40,"start_character":27,"end_line":40,"end_character":56},"in_reply_to":"5a74a57a_7fff31aa","updated":"2016-11-24 12:37:42.000000000","message":"Ah you\u0027re right actually, thanks","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"7dffcc4f80b25afd66e8c898ef1abbd4a3a0c332","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    \u0026\u0026 /var/lib/kolla/venv/bin/pip --no-cache-dir install --upgrade -c requirements/upper-constraints.txt /neutron_vpnaas \\"},{"line_number":38,"context_line":"    \u0026\u0026 cp /neutron_vpnaas/etc/neutron/rootwrap.d/* /etc/neutron/rootwrap.d"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"{{ macros.install_packages(neutron_vpnaas_agent_packages | customizable(\"packages\")) }}"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"{% endif %}"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5a74a57a_7fff31aa","line":40,"range":{"start_line":40,"start_character":27,"end_line":40,"end_character":56},"in_reply_to":"5a74a57a_8b7854c6","updated":"2016-11-24 12:20:05.000000000","message":"I think the variable name is correct, is the same for binary and source installs.","commit_id":"85be8a0a666849627fc98e815b6708d7a10332d0"}]}