osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/15760 )
Change subject: docker: chown -R build:build /usr/local ...................................................................... Patch Set 2: > Patch Set 2: > > Well it feels non-usual to have /usr/local as non-root I guess, which could > mask issues in the future, but fine, go on, let's not spend time on this > since I understand the other way is more time-consuming to get done correctly. Yes, I think it is slightly more time consuming. FWIW, we are doing the same thing in ansible/roles/osmocom-jenkins-slave/tasks/generic-slave.yml. -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/15760 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ie1de8a6d436de0b286ed3372166eceb408bab084 Gerrit-Change-Number: 15760 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osm...@sysmocom.de> Gerrit-Reviewer: laforge <lafo...@osmocom.org> Gerrit-Reviewer: osmith <osm...@sysmocom.de> Gerrit-Reviewer: pespin <pes...@sysmocom.de> Gerrit-Comment-Date: Fri, 11 Oct 2019 11:24:11 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment