Code review comment for lp:~qzhang/lava-dispatcher/fix-861115

Revision history for this message
Spring Zhang (qzhang) wrote :

> 116 + self.android_result_dir =
> mkdtemp(dir=LavaClient.lava_result_dir)
> 117 + os.chmod(self.android_result_dir, 0755)
>
> LavaClient doesn't have property lava_result_dir, it should be
> self.context.lava_result_dir I think.
right
>
> btw, about the mode, default is as below, why need to change?
It's 0700 by default, I changed it to follow the routine in lava code

>
> The directory is readable, writable, and searchable only by the creating user.

« Back to merge proposal