Browse Source

[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 <ykarel@redhat.com>

Change-Id: Icd4f9dc435a807f6f729e4eb2d4df4b282ec79ac
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
(cherry picked from commit 770e57ff31)
(cherry picked from commit 531e810526)
(cherry picked from commit afb63e9db8)
(cherry picked from commit 2b16184da8)
changes/18/743618/2
Goutham Pacha Ravi 1 week ago
committed by Tom Barron
parent
commit
6be3eda9f5
1 changed files with 3 additions and 16 deletions
  1. +3
    -16
      contrib/ci/common.sh

+ 3
- 16
contrib/ci/common.sh View File

@@ -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


Loading…
Cancel
Save