From 770e57ff3192c3828b2d3162ae1f132d3858997b Mon Sep 17 00:00:00 2001 From: Goutham Pacha Ravi Date: Mon, 27 Jul 2020 12:43:59 -0700 Subject: [PATCH] [ci] Remove explicit compression of log files Compression is not needed post [1], explicit compression renders them non-browsable. This commit applies to legacy style jobs that will soon be replaced with zuulv3 native jobs; however, fixing it isn't terrible, and we can use this fix for stable branches where zuulv3 native jobs may not come for a while. [1] http://lists.openstack.org/pipermail/openstack-discuss/2020-January/011875.html Co-Authored-By: yatinkarel Change-Id: Icd4f9dc435a807f6f729e4eb2d4df4b282ec79ac Signed-off-by: Goutham Pacha Ravi --- contrib/ci/common.sh | 19 +++---------------- 1 file changed, 3 insertions(+), 16 deletions(-) diff --git a/contrib/ci/common.sh b/contrib/ci/common.sh index b0117cbb54..2eb47cb096 100755 --- a/contrib/ci/common.sh +++ b/contrib/ci/common.sh @@ -60,15 +60,6 @@ function manila_wait_for_drivers_init { sleep 10 } -function archive_file { - # First argument is expected to be filename - local filename=$1 - - sudo gzip -9 $filename - sudo chown $USER:stack $filename.gz - sudo chmod a+r $filename.gz -} - function save_tempest_results { # First argument is expected to be number or tempest run @@ -89,8 +80,7 @@ function save_tempest_results { # 3. Save tempest log file cp $src_dirname/tempest.log $src_dirname/tempest.txt echo '' > $src_dirname/tempest.log - archive_file $src_dirname/tempest.txt - sudo mv $src_dirname/tempest.txt.gz $dst_dirname/tempest.txt.gz + sudo mv $src_dirname/tempest.txt $dst_dirname/tempest.txt # 4. Save tempest stestr results @@ -105,11 +95,8 @@ function save_tempest_results { if [ -f $src_dirname/tempest.subunit ]; then s2h=`type -p subunit2html` sudo $s2h $src_dirname/tempest.subunit $src_dirname/testr_results.html - archive_file $src_dirname/tempest.subunit - sudo mv $src_dirname/tempest.subunit.gz $dst_dirname/tempest.subunit.gz - - archive_file $src_dirname/testr_results.html - sudo mv $src_dirname/testr_results.html.gz $dst_dirname/testr_results.html.gz + sudo mv $src_dirname/tempest.subunit $dst_dirname/tempest.subunit + sudo mv $src_dirname/testr_results.html $dst_dirname/testr_results.html # 5. Cleanup sudo rm -rf $src_dirname/.stestr