)]}'
{"docker/centos/source/neutron/plugins/networking-arista/Dockerfile":[{"author":{"_account_id":10428,"name":"Harm Weites","email":"harm@weites.com","username":"harmw"},"change_message_id":"783cb3221ba611ebf5d60b1358702396c83dde84","unresolved":false,"context_lines":[{"line_number":11,"context_line":"        /etc/neutron/plugins/networking-arista \\"},{"line_number":12,"context_line":"    \u0026\u0026 rm -rf /networking-arista"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"VOLUME [ \"/usr/lib/python2.7/site-packages/neutron/plugins/networking-arista\" ]"},{"line_number":15,"context_line":"VOLUME [ \"/etc/neutron/plugins/networking-arista\" ]"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"CMD [\"/bin/true\"]"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"3a50d1a3_0f0bae7e","line":14,"updated":"2015-07-31 10:56:13.000000000","message":"you realy need *this* path exposed?","commit_id":"7b0bcd87e5305e40662d5d8dee7a4dfd0c09f1d1"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"16394ab2668b1d8ab1a28709d4b6c5513aaf60df","unresolved":false,"context_lines":[{"line_number":11,"context_line":"        /etc/neutron/plugins/networking-arista \\"},{"line_number":12,"context_line":"    \u0026\u0026 rm -rf /networking-arista"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"VOLUME [ \"/usr/lib/python2.7/site-packages/neutron/plugins/networking-arista\" ]"},{"line_number":15,"context_line":"VOLUME [ \"/etc/neutron/plugins/networking-arista\" ]"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"CMD [\"/bin/true\"]"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"1a4dcd0f_2325f207","line":14,"in_reply_to":"3a50d1a3_0f0bae7e","updated":"2015-08-06 15:11:21.000000000","message":"Yes, this code was originally part of the neutron source tree, but was moved out for the reasons described at https://wiki.openstack.org/wiki/Neutron_Plugins_and_Drivers\n\nInstalling the code here and exposing it as a volume enables us to plug back into Neutron in a modular way, simply by using --volumes-from on the neutron containers.","commit_id":"7b0bcd87e5305e40662d5d8dee7a4dfd0c09f1d1"}],"docker_templates/neutron/neutron-server/Dockerfile.j2":[{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"48930383617d472e1ec3a6dcf8da2b288bca0bee","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"# Install optional third party plugins"},{"line_number":16,"context_line":"RUN yum install -y \\"},{"line_number":17,"context_line":"        openstack-neutron-bigswitch.noarch \\"},{"line_number":18,"context_line":"        openstack-neutron-brocade.noarch \\"},{"line_number":19,"context_line":"        openstack-neutron-cisco.noarch \\"},{"line_number":20,"context_line":"        openstack-neutron-common.noarch \\"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"fa1b9901_1aed7225","line":17,"updated":"2015-08-21 19:56:12.000000000","message":"can you verify that these do not pull in lots of extra deps?","commit_id":"3e188942608ec1fd3e648b7f0d54ee85b509be84"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"48930383617d472e1ec3a6dcf8da2b288bca0bee","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    \u0026\u0026 yum clean all"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"# Install optional third party plugins"},{"line_number":51,"context_line":"curl http://tarballs.openstack.org/networking-bigswitch/networking-bigswitch-master.tar.gz \\"},{"line_number":52,"context_line":"        -o networking-bigswitch-master.tar.gz \\"},{"line_number":53,"context_line":"    \u0026\u0026 pip install /bsnstacklib-*/ \\"},{"line_number":54,"context_line":"    \u0026\u0026 rm -rf /networking-bigswitch-master.tar.gz /bsnstacklib-*/"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"fa1b9901_da7ffa64","line":51,"updated":"2015-08-21 19:56:12.000000000","message":"The issue with curl inside the container like this is it will not pull in changes from upstream if the remote source changes. Perhaps we can take the same approach as the source builds for adding this in? I am open to ideas.","commit_id":"3e188942608ec1fd3e648b7f0d54ee85b509be84"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"2cc72cd37ca3f1eb024200666e11cb73f571070e","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    \u0026\u0026 yum clean all"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"# Install optional third party plugins"},{"line_number":51,"context_line":"curl http://tarballs.openstack.org/networking-bigswitch/networking-bigswitch-master.tar.gz \\"},{"line_number":52,"context_line":"        -o networking-bigswitch-master.tar.gz \\"},{"line_number":53,"context_line":"    \u0026\u0026 pip install /bsnstacklib-*/ \\"},{"line_number":54,"context_line":"    \u0026\u0026 rm -rf /networking-bigswitch-master.tar.gz /bsnstacklib-*/"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"fa1b9901_e5d7a4eb","line":51,"in_reply_to":"fa1b9901_ca8f0f74","updated":"2015-08-23 07:43:33.000000000","message":"Thats not a bad idea, so have a tarball that wraps n number of other tarballs so there is a single ADD command. We will have to test if we can make that work with the docker cache, but if so I am all for that.\n\nThen we can simply loop through the directories to install like this pseudo code\n\nADD all_plugins.tar /plugins\nRUN for dir in /plugins:\n          pip install $dir","commit_id":"3e188942608ec1fd3e648b7f0d54ee85b509be84"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"c758de9d73648b8d09e80011b94ba5de06e9d2b4","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    \u0026\u0026 yum clean all"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"# Install optional third party plugins"},{"line_number":51,"context_line":"curl http://tarballs.openstack.org/networking-bigswitch/networking-bigswitch-master.tar.gz \\"},{"line_number":52,"context_line":"        -o networking-bigswitch-master.tar.gz \\"},{"line_number":53,"context_line":"    \u0026\u0026 pip install /bsnstacklib-*/ \\"},{"line_number":54,"context_line":"    \u0026\u0026 rm -rf /networking-bigswitch-master.tar.gz /bsnstacklib-*/"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"fa1b9901_ca8f0f74","line":51,"in_reply_to":"fa1b9901_da7ffa64","updated":"2015-08-23 05:11:32.000000000","message":"one option that just popped into my head was to make a special build.ini type that is for plugins only that lists all remote sources and all the tarballs get merged into one bigger tarball  before being copied into the container.  I\u0027d really prefer to keep the remote list in build.ini rather then hard coding in the dockerfile.","commit_id":"3e188942608ec1fd3e648b7f0d54ee85b509be84"}]}