)]}'
{"nodepool/elements/cache-devstack/extra-data.d/55-cache-devstack-repos":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"f1c335c75c5fa963e0def2c38390ab15e33db51b","unresolved":false,"context_lines":[{"line_number":208,"context_line":"                args \u003d dict("},{"line_number":209,"context_line":"                    name\u003dfname,"},{"line_number":210,"context_line":"                    location\u003dos.path.join(\u0027/home/zuul/cache/files\u0027, fname),"},{"line_number":211,"context_line":"                    url\u003durl)"},{"line_number":212,"context_line":"                images_list.write(line_template % args)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"1aa78d24_2ed47145","line":211,"updated":"2016-07-05 18:48:45.000000000","message":"We should check the total size of this and its affect on smaller root filesystems. May want to consider hardlinking these cache locations together.","commit_id":"16d4b900ac0e7c2861c21680fc74f84e7e1dfcf8"}],"nodepool/elements/cache-devstack/install.d/99-cache-testrepository-db":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"f1c335c75c5fa963e0def2c38390ab15e33db51b","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"# TODO(pabelanger): Delete jenkins specific code once we have migrated to the"},{"line_number":33,"context_line":"# zuul user."},{"line_number":34,"context_line":"sudo chown -R jenkins:jenkins $TEMPEST_DIR/preseed-streams"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# Delete the venv after the script"},{"line_number":37,"context_line":"sudo rm -rf /opt/git/subunit2sql-env"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"1aa78d24_8ef485f2","line":34,"updated":"2016-07-05 18:48:45.000000000","message":"We should add a chmod here to make the files world readable so that zuul user can get at them.","commit_id":"16d4b900ac0e7c2861c21680fc74f84e7e1dfcf8"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6210baf3538846d555e9758b0405259c97d48dba","unresolved":false,"context_lines":[{"line_number":33,"context_line":"# zuul user."},{"line_number":34,"context_line":"sudo chown -R jenkins:jenkins $TEMPEST_DIR/preseed-streams"},{"line_number":35,"context_line":"# TODO(pabelanger): Remove this once we\u0027ve migrated to zuul user."},{"line_number":36,"context_line":"sudo chmod -R 0777 $TEMPEST_DIR/preseed-streams"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"# Delete the venv after the script"},{"line_number":39,"context_line":"sudo rm -rf /opt/git/subunit2sql-env"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"1aa78d24_be04054a","line":36,"updated":"2016-07-05 22:51:27.000000000","message":"Two things here. First if we open this to the world then either jenkins or zuul should be able use it which means we could change the chown above to be zuul:zuul and get slightly ahead in the transition.\n\nBut also would be nice to restrict this to just the perms that are necessary. Looking at devstack-gate the $testuser needs to be able to [ -d ] this dir and to read the files in the dir.\n\nThat would look like:\n\n  chmod 0555 $TEMPEST_DIR/preseed-streams\n  find $TEMPEST_DIR/preseed-streams -execdir chmod 0444 {}\\;\n\n(Double check my bash its probably not entirely correct but should illustrate what is required).","commit_id":"0bf8bd19f0007ae6092277847f9f55fd0aceb1ef"}]}