)]}'
{"setup.cfg":[{"author":{"_account_id":7505,"name":"Jaume Devesa","email":"devvesa@gmail.com","username":"devesa"},"change_message_id":"63851383d450b2cf5a8b8e23461e7c91ef953a4a","unresolved":false,"context_lines":[{"line_number":30,"context_line":"packages \u003d"},{"line_number":31,"context_line":"    kuryr"},{"line_number":32,"context_line":"data_files \u003d"},{"line_number":33,"context_line":"    /usr/libexec \u003d usr/libexec/*"},{"line_number":34,"context_line":"    /usr/lib/docker/plugins/kuryr \u003d etc/kuryr.spec"},{"line_number":35,"context_line":"    /etc/kuryr \u003d"},{"line_number":36,"context_line":"        etc/kuryr.conf.sample"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"1a122d0e_09d854ee","line":33,"updated":"2016-05-05 15:39:44.000000000","message":"1. use relative paths. otherwise a common user will find permission errors when trying to install in a virtual env\n2. it is a \"string: list\" dict. Better put the second argument in a new line\n3. Wildcards lead to wild behaviour. Please enumerate all the drivers\n\n    usr/libexec \u003d\n        usr/libexec/midonet\n        usr/libexec/ovs\n        usr/libexec/bridge\n        ...","commit_id":"17765e02c9eae4d0bc83a7dc98c61e01c44dd135"},{"author":{"_account_id":14352,"name":"Antoni Segura Puimedon","email":"celebdor@gmail.com","username":"toni"},"change_message_id":"523b0af949fd6fae3907727a6ab40b8f702a63a1","unresolved":false,"context_lines":[{"line_number":30,"context_line":"packages \u003d"},{"line_number":31,"context_line":"    kuryr"},{"line_number":32,"context_line":"data_files \u003d"},{"line_number":33,"context_line":"    /usr/libexec \u003d usr/libexec/*"},{"line_number":34,"context_line":"    /usr/lib/docker/plugins/kuryr \u003d etc/kuryr.spec"},{"line_number":35,"context_line":"    /etc/kuryr \u003d"},{"line_number":36,"context_line":"        etc/kuryr.conf.sample"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"1a122d0e_ff6946c1","line":33,"in_reply_to":"1a122d0e_09d854ee","updated":"2016-05-05 16:04:34.000000000","message":"Done","commit_id":"17765e02c9eae4d0bc83a7dc98c61e01c44dd135"},{"author":{"_account_id":7505,"name":"Jaume Devesa","email":"devvesa@gmail.com","username":"devesa"},"change_message_id":"63851383d450b2cf5a8b8e23461e7c91ef953a4a","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    kuryr"},{"line_number":32,"context_line":"data_files \u003d"},{"line_number":33,"context_line":"    /usr/libexec \u003d usr/libexec/*"},{"line_number":34,"context_line":"    /usr/lib/docker/plugins/kuryr \u003d etc/kuryr.spec"},{"line_number":35,"context_line":"    /etc/kuryr \u003d"},{"line_number":36,"context_line":"        etc/kuryr.conf.sample"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"1a122d0e_69e1089b","line":34,"updated":"2016-05-05 15:39:44.000000000","message":"ditto","commit_id":"17765e02c9eae4d0bc83a7dc98c61e01c44dd135"},{"author":{"_account_id":14352,"name":"Antoni Segura Puimedon","email":"celebdor@gmail.com","username":"toni"},"change_message_id":"523b0af949fd6fae3907727a6ab40b8f702a63a1","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    kuryr"},{"line_number":32,"context_line":"data_files \u003d"},{"line_number":33,"context_line":"    /usr/libexec \u003d usr/libexec/*"},{"line_number":34,"context_line":"    /usr/lib/docker/plugins/kuryr \u003d etc/kuryr.spec"},{"line_number":35,"context_line":"    /etc/kuryr \u003d"},{"line_number":36,"context_line":"        etc/kuryr.conf.sample"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"1a122d0e_5f44ba45","line":34,"in_reply_to":"1a122d0e_69e1089b","updated":"2016-05-05 16:04:34.000000000","message":"Docker only looks in absolute paths:\n\n    UNIX domain socket files must be located under /run/docker/plugins, whereas spec files can be located either under /etc/docker/plugins or /usr/lib/docker/plugins","commit_id":"17765e02c9eae4d0bc83a7dc98c61e01c44dd135"},{"author":{"_account_id":7505,"name":"Jaume Devesa","email":"devvesa@gmail.com","username":"devesa"},"change_message_id":"63851383d450b2cf5a8b8e23461e7c91ef953a4a","unresolved":false,"context_lines":[{"line_number":32,"context_line":"data_files \u003d"},{"line_number":33,"context_line":"    /usr/libexec \u003d usr/libexec/*"},{"line_number":34,"context_line":"    /usr/lib/docker/plugins/kuryr \u003d etc/kuryr.spec"},{"line_number":35,"context_line":"    /etc/kuryr \u003d"},{"line_number":36,"context_line":"        etc/kuryr.conf.sample"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"[build_sphinx]"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"1a122d0e_a9ede08f","line":35,"updated":"2016-05-05 15:39:44.000000000","message":"ditto","commit_id":"17765e02c9eae4d0bc83a7dc98c61e01c44dd135"},{"author":{"_account_id":14352,"name":"Antoni Segura Puimedon","email":"celebdor@gmail.com","username":"toni"},"change_message_id":"523b0af949fd6fae3907727a6ab40b8f702a63a1","unresolved":false,"context_lines":[{"line_number":32,"context_line":"data_files \u003d"},{"line_number":33,"context_line":"    /usr/libexec \u003d usr/libexec/*"},{"line_number":34,"context_line":"    /usr/lib/docker/plugins/kuryr \u003d etc/kuryr.spec"},{"line_number":35,"context_line":"    /etc/kuryr \u003d"},{"line_number":36,"context_line":"        etc/kuryr.conf.sample"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"[build_sphinx]"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"1a122d0e_7f3096e9","line":35,"in_reply_to":"1a122d0e_a9ede08f","updated":"2016-05-05 16:04:34.000000000","message":"Done","commit_id":"17765e02c9eae4d0bc83a7dc98c61e01c44dd135"},{"author":{"_account_id":7505,"name":"Jaume Devesa","email":"devvesa@gmail.com","username":"devesa"},"change_message_id":"4e0903641ab9e6ba2a77f10e3e04ab8716477aeb","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        usr/libexec/kuryr/unbound"},{"line_number":39,"context_line":"    /usr/lib/docker/plugins/kuryr \u003d etc/kuryr.spec"},{"line_number":40,"context_line":"    etc/kuryr \u003d"},{"line_number":41,"context_line":"        etc/kuryr.conf.sample"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"[build_sphinx]"},{"line_number":44,"context_line":"source-dir \u003d doc/source"}],"source_content_type":"text/x-ttcn-cfg","patch_set":3,"id":"dab17558_28884de7","line":41,"updated":"2016-05-06 09:17:12.000000000","message":"You should provide the file `etc/kuryr.conf.sample` in the patch as well","commit_id":"537b62717d95fb96cd130471ce2259ba998f455e"},{"author":{"_account_id":14352,"name":"Antoni Segura Puimedon","email":"celebdor@gmail.com","username":"toni"},"change_message_id":"0b2eba98e53f70dad2c3582e5268a04019c319ef","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        usr/libexec/kuryr/unbound"},{"line_number":39,"context_line":"    /usr/lib/docker/plugins/kuryr \u003d etc/kuryr.spec"},{"line_number":40,"context_line":"    etc/kuryr \u003d"},{"line_number":41,"context_line":"        etc/kuryr.conf.sample"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"[build_sphinx]"},{"line_number":44,"context_line":"source-dir \u003d doc/source"}],"source_content_type":"text/x-ttcn-cfg","patch_set":3,"id":"dab17558_53ff4634","line":41,"in_reply_to":"dab17558_28884de7","updated":"2016-05-06 09:26:21.000000000","message":"No, the install instructions say that before running setup.py install you should use \n    oslo-config-generator --config-file\u003detc/kuryr-config-generator.conf\n\nTo generate it.","commit_id":"537b62717d95fb96cd130471ce2259ba998f455e"},{"author":{"_account_id":7505,"name":"Jaume Devesa","email":"devvesa@gmail.com","username":"devesa"},"change_message_id":"e41a7d777bdc6730cc18b99833c706afac9605ee","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        usr/libexec/kuryr/unbound"},{"line_number":39,"context_line":"    /usr/lib/docker/plugins/kuryr \u003d etc/kuryr.spec"},{"line_number":40,"context_line":"    etc/kuryr \u003d"},{"line_number":41,"context_line":"        etc/kuryr.conf.sample"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"[build_sphinx]"},{"line_number":44,"context_line":"source-dir \u003d doc/source"}],"source_content_type":"text/x-ttcn-cfg","patch_set":3,"id":"dab17558_2eca15a2","line":41,"in_reply_to":"dab17558_53ff4634","updated":"2016-05-06 09:38:08.000000000","message":"Then I don\u0027t think this declaration file should be here. I don\u0027t like to rely on a file that does not exist on source code. Neutron does not have its etc/neutron.conf there, and at the end, if you want to really applicate a custom configuration, you will need to copy the kuryr.conf.sample to kuryr.conf. \n\nThat won\u0027t force you to generate a sample file before run `python setup.py install` if you are ok with the default configuration.","commit_id":"537b62717d95fb96cd130471ce2259ba998f455e"},{"author":{"_account_id":11343,"name":"Gal Sagie","email":"gal.sagie@gmail.com","username":"GalSagie"},"change_message_id":"55d691ccf91e6039c18458e76b77db50d99e6b82","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[metadata]"},{"line_number":2,"context_line":"name \u003d kuryr"},{"line_number":3,"context_line":"version \u003d 0.1.0"},{"line_number":4,"context_line":"summary \u003d Docker libnetwork driver for OpenStack Neutron"},{"line_number":5,"context_line":"description-file \u003d"},{"line_number":6,"context_line":"    README.rst"},{"line_number":7,"context_line":"author \u003d OpenStack"}],"source_content_type":"text/x-ttcn-cfg","patch_set":4,"id":"dab17558_1d0f4d2f","line":4,"updated":"2016-05-09 06:04:28.000000000","message":"I think we can change this description as we adding the sub projects, but will do this in another patch","commit_id":"7aa20ff639e45c05a2441f7091c736f975ba14c8"}]}
