Martin Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/46804 )
Change subject: util/docker: Update atime mount point options for jenkins ......................................................................
util/docker: Update atime mount point options for jenkins
- The ccache files don't need atime. - Enable strict atime for the git repos. This will help find unused files.
Signed-off-by: Martin Roth martin@coreboot.org Change-Id: I94bcc55ea5c5a74f3ad0292ca50b74874a0d920d --- M util/docker/coreboot-jenkins-node/Dockerfile 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/46804/1
diff --git a/util/docker/coreboot-jenkins-node/Dockerfile b/util/docker/coreboot-jenkins-node/Dockerfile index e9d866b..36b77d6 100644 --- a/util/docker/coreboot-jenkins-node/Dockerfile +++ b/util/docker/coreboot-jenkins-node/Dockerfile @@ -38,10 +38,10 @@ # Create tmpfs directories to build in RUN mkdir /cb-build && \ chown coreboot:coreboot /cb-build && \ - echo "tmpfs /cb-build tmpfs rw,mode=1777 0 0" > /etc/fstab && \ + echo "tmpfs /cb-build tmpfs rw,mode=1777,noatime 0 0" > /etc/fstab && \ mkdir -p /home/coreboot/node-root/workspace && \ chown -R coreboot:coreboot /home/coreboot/node-root && \ - echo "tmpfs /home/coreboot/node-root/workspace tmpfs rw,mode=1777 0 0" >> /etc/fstab && \ + echo "tmpfs /home/coreboot/node-root/workspace tmpfs rw,mode=1777,strictatime,atime 0 0" >> /etc/fstab && \ chown coreboot:coreboot /home/coreboot/.ccache && \ echo "tmpfs /home/coreboot/.ccache tmpfs rw,mode=1777 0 0" >> /etc/fstab
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46804 )
Change subject: util/docker: Update atime mount point options for jenkins ......................................................................
Patch Set 1: Code-Review+2
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/46804 )
Change subject: util/docker: Update atime mount point options for jenkins ......................................................................
util/docker: Update atime mount point options for jenkins
- The ccache files don't need atime. - Enable strict atime for the git repos. This will help find unused files.
Signed-off-by: Martin Roth martin@coreboot.org Change-Id: I94bcc55ea5c5a74f3ad0292ca50b74874a0d920d Reviewed-on: https://review.coreboot.org/c/coreboot/+/46804 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Patrick Georgi pgeorgi@google.com --- M util/docker/coreboot-jenkins-node/Dockerfile 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Patrick Georgi: Looks good to me, approved
diff --git a/util/docker/coreboot-jenkins-node/Dockerfile b/util/docker/coreboot-jenkins-node/Dockerfile index e9d866b..36b77d6 100644 --- a/util/docker/coreboot-jenkins-node/Dockerfile +++ b/util/docker/coreboot-jenkins-node/Dockerfile @@ -38,10 +38,10 @@ # Create tmpfs directories to build in RUN mkdir /cb-build && \ chown coreboot:coreboot /cb-build && \ - echo "tmpfs /cb-build tmpfs rw,mode=1777 0 0" > /etc/fstab && \ + echo "tmpfs /cb-build tmpfs rw,mode=1777,noatime 0 0" > /etc/fstab && \ mkdir -p /home/coreboot/node-root/workspace && \ chown -R coreboot:coreboot /home/coreboot/node-root && \ - echo "tmpfs /home/coreboot/node-root/workspace tmpfs rw,mode=1777 0 0" >> /etc/fstab && \ + echo "tmpfs /home/coreboot/node-root/workspace tmpfs rw,mode=1777,strictatime,atime 0 0" >> /etc/fstab && \ chown coreboot:coreboot /home/coreboot/.ccache && \ echo "tmpfs /home/coreboot/.ccache tmpfs rw,mode=1777 0 0" >> /etc/fstab
-
Martin Roth (Code Review) -
Patrick Georgi (Code Review)