)]}'
{"imagebuild/tinyipa/build-tinyipa.sh":[{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"c80577d537cbb3800b5193be74ac2de18a4f886f","unresolved":false,"context_lines":[{"line_number":6,"context_line":"BUILD_AND_INSTALL_TINYIPA\u003d${BUILD_AND_INSTALL_TINYIPA:-false}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"CHROOT_PATH\u003d\"/usr/local/sbin:/usr/local/bin:/apps/bin:/usr/sbin:/usr/bin:/sbin:/bin\""},{"line_number":9,"context_line":"CHROOT_CMD\u003d\"sudo chroot $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH\""},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# Ensure we have an extended sudo to prevent the need to enter a password over"},{"line_number":12,"context_line":"# and over again."}],"source_content_type":"text/x-sh","patch_set":1,"id":"1af94dfe_50566cce","line":9,"updated":"2016-03-18 08:38:32.000000000","message":"Unfortunately adding -i will mean that the useful environment variables like http_proxy and https_proxy will not be passed into the build chroot leading to build failures in environments which require them. Which environment variables were causing problems?","commit_id":"1d3698d39704dbaff9ff318f2609124e6d50295f"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"a6f7bbeeb6b2f7021a5d4da8e2c53cc0439bbc77","unresolved":false,"context_lines":[{"line_number":6,"context_line":"BUILD_AND_INSTALL_TINYIPA\u003d${BUILD_AND_INSTALL_TINYIPA:-false}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"CHROOT_PATH\u003d\"/usr/local/sbin:/usr/local/bin:/apps/bin:/usr/sbin:/usr/bin:/sbin:/bin\""},{"line_number":9,"context_line":"CHROOT_CMD\u003d\"sudo chroot $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH\""},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# Ensure we have an extended sudo to prevent the need to enter a password over"},{"line_number":12,"context_line":"# and over again."}],"source_content_type":"text/x-sh","patch_set":1,"id":"1af94dfe_e3cd2f32","line":9,"in_reply_to":"1af94dfe_50566cce","updated":"2016-03-18 10:44:17.000000000","message":"locale.LC_ALL \nsee http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-03-17.log.html#t2016-03-17T15:17:57\nthe -i fix the issue \ncan\u0027t we export the http_proxy and https_proxy instead.\nI feel we should not be effected by use env variable when building","commit_id":"1d3698d39704dbaff9ff318f2609124e6d50295f"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"87b789cb4dbc8e8339e849bacb503cde35906b71","unresolved":false,"context_lines":[{"line_number":6,"context_line":"BUILD_AND_INSTALL_TINYIPA\u003d${BUILD_AND_INSTALL_TINYIPA:-false}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"CHROOT_PATH\u003d\"/usr/local/sbin:/usr/local/bin:/apps/bin:/usr/sbin:/usr/bin:/sbin:/bin\""},{"line_number":9,"context_line":"CHROOT_CMD\u003d\"sudo chroot $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH\""},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# Ensure we have an extended sudo to prevent the need to enter a password over"},{"line_number":12,"context_line":"# and over again."}],"source_content_type":"text/x-sh","patch_set":1,"id":"1af94dfe_c37aae23","line":9,"in_reply_to":"1af94dfe_cfef721c","updated":"2016-03-18 12:17:58.000000000","message":"Done","commit_id":"1d3698d39704dbaff9ff318f2609124e6d50295f"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"449e5af2ce4a5a8e6cf5e812ae13c69bd0494f05","unresolved":false,"context_lines":[{"line_number":6,"context_line":"BUILD_AND_INSTALL_TINYIPA\u003d${BUILD_AND_INSTALL_TINYIPA:-false}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"CHROOT_PATH\u003d\"/usr/local/sbin:/usr/local/bin:/apps/bin:/usr/sbin:/usr/bin:/sbin:/bin\""},{"line_number":9,"context_line":"CHROOT_CMD\u003d\"sudo chroot $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH\""},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# Ensure we have an extended sudo to prevent the need to enter a password over"},{"line_number":12,"context_line":"# and over again."}],"source_content_type":"text/x-sh","patch_set":1,"id":"1af94dfe_cfef721c","line":9,"in_reply_to":"1af94dfe_e3cd2f32","updated":"2016-03-18 11:44:49.000000000","message":"Confirmed I\u0027m seeing this error on my Debian machine too, it must be a change in upstream Tiny Core linux that\u0027s causing this, -i does fix, but please can we add http/s_proxy to the CHROOT_CMD here so that it doesn\u0027t break builds behind a proxy :)","commit_id":"1d3698d39704dbaff9ff318f2609124e6d50295f"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"c80577d537cbb3800b5193be74ac2de18a4f886f","unresolved":false,"context_lines":[{"line_number":61,"context_line":"$CHROOT_CMD chmod a+rwx /etc/sysconfig/tcuser"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"while read line; do"},{"line_number":64,"context_line":"    sudo chroot --userspec\u003d1001:50 $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH tce-load -wci $line"},{"line_number":65,"context_line":"done \u003c $WORKDIR/build_files/buildreqs.lst"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"sudo umount $BUILDDIR/proc"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1af94dfe_90cc746b","line":64,"range":{"start_line":64,"start_character":16,"end_line":64,"end_character":34},"updated":"2016-03-18 08:38:32.000000000","message":"Was there any error given here? The man page definitely says:   \n\n--userspec\u003dUSER:GROUP\n\t      specify user and group (ID or name) to use\n\nhttp://www.polarhome.com/service/man/?qf\u003dchroot\u0026tf\u003d2\u0026of\u003dFedora\u0026sf\u003d1\n\nIf we are going to keep it like this I would prefer if we added a couple of variables:\n\n    $TC\u003d1001\n    $STAFF\u003d50\n\nso it could be:\n\n    --userspec\u003d$TC:$STAFF\n\nMy concern is the potential for the ids to change when moving between versions of tiny core linux, whereas referencing them by name abstracts us from that.","commit_id":"1d3698d39704dbaff9ff318f2609124e6d50295f"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"87b789cb4dbc8e8339e849bacb503cde35906b71","unresolved":false,"context_lines":[{"line_number":61,"context_line":"$CHROOT_CMD chmod a+rwx /etc/sysconfig/tcuser"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"while read line; do"},{"line_number":64,"context_line":"    sudo chroot --userspec\u003d1001:50 $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH tce-load -wci $line"},{"line_number":65,"context_line":"done \u003c $WORKDIR/build_files/buildreqs.lst"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"sudo umount $BUILDDIR/proc"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1af94dfe_6392ba4d","line":64,"range":{"start_line":64,"start_character":16,"end_line":64,"end_character":34},"in_reply_to":"1af94dfe_63a2bf9a","updated":"2016-03-18 12:17:58.000000000","message":"Done","commit_id":"1d3698d39704dbaff9ff318f2609124e6d50295f"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"a6f7bbeeb6b2f7021a5d4da8e2c53cc0439bbc77","unresolved":false,"context_lines":[{"line_number":61,"context_line":"$CHROOT_CMD chmod a+rwx /etc/sysconfig/tcuser"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"while read line; do"},{"line_number":64,"context_line":"    sudo chroot --userspec\u003d1001:50 $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH tce-load -wci $line"},{"line_number":65,"context_line":"done \u003c $WORKDIR/build_files/buildreqs.lst"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"sudo umount $BUILDDIR/proc"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1af94dfe_63a2bf9a","line":64,"range":{"start_line":64,"start_character":16,"end_line":64,"end_character":34},"in_reply_to":"1af94dfe_90cc746b","updated":"2016-03-18 10:44:17.000000000","message":"this is the error:\n+ sudo chroot --userspec\u003dtc:staff /.autodirect/mtrswgwork/moshele/openstack/ironic-python-agent/imagebuild/tinyipa/tinyipabuild /usr/bin/env PATH\u003d/usr/local/sbin:/usr/local/bin:/apps/bin:/usr/sbin:/usr/bin:/sbin:/bin tce-load -wci bash.tcz\nchroot: invalid user\nMakefile:6: recipe for target \u0027build\u0027 failed\nmake: *** [build] Error 125\n[root@r-ufm231 tinyipa]# cat /etc/issue\nFedora release 21 (Twenty One)\nKernel \\r on an \\m (\\l)\nI don\u0027t know why it doesn\u0027t work with names\n\nI will update the code to use \n    $TC\u003d1001\n    $STAFF\u003d50\n    --userspec\u003d$TC:$STAFF","commit_id":"1d3698d39704dbaff9ff318f2609124e6d50295f"}]}