tree 4f0c3e28da699fb659c1c46b6d74fe6f1fe71c5e
parent c092b383de3856fd28c104a510106b0c88e7fc1b
author xixuan <xixuan@chromium.org> 1478019071 -0700
committer chrome-bot <chrome-bot@chromium.org> 1478602420 -0800

devserver: ignore time stamps when extracting files with tar

Current 'tar' func in common_util will unzip files with timestamp when the
files' modified time is faster than devserver itself:
command:  tar xf tmp.tar.gz --directory ./ --verbose --gzip
output:   control.npo_delta_tmp
          tar: control.npo_delta_tmp: time stamp 2016-11-11 00:00:00 is
          832281.031926535 s in the future

During devserver staging, such output will be written to a marker_file, which
is used by devserver to check whether files are successfully downloaded and
unzipped. If the output includes such timestamp, it's actually not a real file
name, which leads to devserver failing to check whether this file exists, and
claim a false negative stage failure.

This CL changes the tar command to set all of the modified times on the
unpacked files to the current system time.

Also fix lint errors.

BUG=chromium:659850
TEST=Create a future file, zip it, and then call common.util.ExtractTarball()
to verify.

Change-Id: I55c7446ecd1ffe4cde350abc33009edbc6a8ec70
Reviewed-on: https://chromium-review.googlesource.com/406267
Commit-Ready: Xixuan Wu <xixuan@chromium.org>
Tested-by: Xixuan Wu <xixuan@chromium.org>
Reviewed-by: Luigi Semenzato <semenzato@chromium.org>