)]}'
{"lib/ironic":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"f69d4622ae70d4c7d90a451135747a277d6a7f08","unresolved":false,"context_lines":[{"line_number":597,"context_line":"                wget \"$IRONIC_AGENT_TARBALL\" -O ironic_agent_tarball.tar.gz"},{"line_number":598,"context_line":"                tar zxfv ironic_agent_tarball.tar.gz"},{"line_number":599,"context_line":"                mv imagebuild/coreos/UPLOAD/coreos_production_pxe.vmlinuz $IRONIC_DEPLOY_KERNEL_PATH"},{"line_number":600,"context_line":"                mv imagebuild/coreos/UPLOAD/coreos_production_pxe_image-oem.cpio.gz $IRONIC_DEPLOY_RAMDISK_PATH"},{"line_number":601,"context_line":"                sudo make clean \u0026\u0026 rm -rf imagebuild/coreos/UPLOAD/*"},{"line_number":602,"context_line":"                rm ironic_agent_tarball.tar.gz"},{"line_number":603,"context_line":"            else"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa98f980_1a49538b","line":600,"updated":"2014-09-02 16:55:41.000000000","message":"this continues to not address my key concern which is the fact that devstack is reaching deep into a foreign tarball and effectively making it\u0027s own installer for that code.\n\nConsuming a tarball for devstack should be:\n\n  wget \"$IRONIC_AGENT_TARBALL\"\n  tar xvzf $(basename $IRONIC_AGENT_TARBALL)\n  cd $(basename $IRONIC_AGENT_TARBALL) (minus the tar.gz)\n  make install PREFIX\u003d$IRONIC_DEPLOY_PATH\n\nanything more complicated than that really shouldn\u0027t be in devstack.","commit_id":"07ffeabc498c6ccf601627e385f3383761c1bdba"}]}