[Xen-devel] [PATCH OSSTEST v2] ts-logs-capture: ensure captured log files are readable.

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

At least /var/log/libvirt/libvirtd.log is 0600 on the target machine
and this gets preserved in the captured logfile, eventually meaning
that apache on the log server cannot read the file to serve it ("You
don't have permission to access...").

Set all logfiles to 0666, modified by umask.

Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
v2: Honour umask

Currently visible at
http://www.chiark.greenend.org.uk/~xensrcts/logs/34114/test-armhf-armhf-libvirt/marilith-n4---var-log-libvirt-libvirtd.log
but not mentioning in the commit log since it'll go away before long.
---
 ts-logs-capture | 1 +
 1 file changed, 1 insertion(+)

diff --git a/ts-logs-capture b/ts-logs-capture
index dbca13a..453b03d 100755
--- a/ts-logs-capture
+++ b/ts-logs-capture
@@ -59,6 +59,7 @@ END
             logm("fetching $logfile to $llogfile");
             if (!eval {
                 target_getfile_root($lho,60, $logfile,"$stash/$llogfile");
+               chmod(0666&~umask, "$stash/$llogfile") or die "chmod 
$stash/$llogfile $!";
                if ($logfile !~ m/\.gz$/) {
                    stat("$stash/$llogfile") or die "$stash/$llogfile $!";
                    compress_stashed($llogfile) if(stat _)[7] > 100000;
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel