)]}'
{"nodepool/elements/jenkins-slave/install.d/20-jenkins-slave":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"e9f02ad2f704edabd57abfaeb121e10762527bd1","unresolved":false,"context_lines":[{"line_number":22,"context_line":"if [ -d /opt/cache/files ] ; then"},{"line_number":23,"context_line":"    mkdir -p /home/jenkins/cache"},{"line_number":24,"context_line":"    ln -sf /opt/cache/files /home/jenkins/cache/files"},{"line_number":25,"context_line":"    chown -R jenkins:jenkins /opt/cache/files"},{"line_number":26,"context_line":"fi"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# this was copied from outside the chroot by extras.d"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7f515b1d_26e576dd","side":"PARENT","line":25,"updated":"2017-10-03 13:02:22.000000000","message":"Was the reason for this that the jenkins user may need write access? Seems like writing to the file cache is of questionable value anyway and we should just let it all be root-owned (but that can be another patch since I\u0027m approving this one for now).","commit_id":"e158a4087c2f1f86758f449da9989e58a6f02084"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a0bc8db9e481a7f87d590ec9ed76aeb2ca3a714d","unresolved":false,"context_lines":[{"line_number":20,"context_line":"# devstack-cache element. This allows you to build an image without"},{"line_number":21,"context_line":"# incurring the cost of caching all the things."},{"line_number":22,"context_line":"if [ -d /opt/cache/files ] ; then"},{"line_number":23,"context_line":"    mkdir -p /home/jenkins/cache"},{"line_number":24,"context_line":"    ln -sf /opt/cache/files /home/jenkins/cache/files"},{"line_number":25,"context_line":"    chown -R jenkins:jenkins /opt/cache/files"},{"line_number":26,"context_line":"    # but make sure the cache is readable by everyone"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7f515b1d_a65cc918","line":23,"range":{"start_line":23,"start_character":19,"end_line":23,"end_character":26},"updated":"2017-10-03 09:29:45.000000000","message":"is it still jenkins?","commit_id":"5ec31e6c46051df15d4c5280849139ecd4147971"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"a0bc8db9e481a7f87d590ec9ed76aeb2ca3a714d","unresolved":false,"context_lines":[{"line_number":21,"context_line":"# incurring the cost of caching all the things."},{"line_number":22,"context_line":"if [ -d /opt/cache/files ] ; then"},{"line_number":23,"context_line":"    mkdir -p /home/jenkins/cache"},{"line_number":24,"context_line":"    ln -sf /opt/cache/files /home/jenkins/cache/files"},{"line_number":25,"context_line":"    chown -R jenkins:jenkins /opt/cache/files"},{"line_number":26,"context_line":"    # but make sure the cache is readable by everyone"},{"line_number":27,"context_line":"    chmod -R a+rX /opt/cache/files/*"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7f515b1d_66a011f5","line":24,"range":{"start_line":24,"start_character":34,"end_line":24,"end_character":41},"updated":"2017-10-03 09:29:45.000000000","message":"ditto","commit_id":"5ec31e6c46051df15d4c5280849139ecd4147971"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"ba18bececb3eb1b0f2efd950c570dba617bd7cf5","unresolved":false,"context_lines":[{"line_number":22,"context_line":"if [ -d /opt/cache/files ] ; then"},{"line_number":23,"context_line":"    mkdir -p /home/jenkins/cache"},{"line_number":24,"context_line":"    ln -sf /opt/cache/files /home/jenkins/cache/files"},{"line_number":25,"context_line":"    chown -R jenkins:jenkins /opt/cache/files"},{"line_number":26,"context_line":"    # but make sure the cache is readable by everyone"},{"line_number":27,"context_line":"    chmod -R a+rX /opt/cache/files/*"},{"line_number":28,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7f515b1d_2bb9c477","line":25,"range":{"start_line":25,"start_character":13,"end_line":25,"end_character":20},"updated":"2017-10-03 09:12:23.000000000","message":"shouldn\u0027t this be zuul:zuul nowadays?","commit_id":"5ec31e6c46051df15d4c5280849139ecd4147971"}]}