openstack/bifrost
Code Issues Proposed changes

Fix testing script permission settings for logs

Browse Source 
Somewhere along the way we lost the ability to read the
conductor, api, and baremetal log files that we retained
for storage as CI test results. This revision resets
the permissions so any user can read the file since
we can't make assumptions about the webserver user.

Change-Id: Idc36f736f9e54a69348045988483559b88e51980
This commit is contained in:
Julia Kreger 2016-06-27 09:01:19 -04:00
parent 226407dd9a
commit ddb3965d30
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
scripts/collect-test-info.sh
View File

@ -14,6 +14,7 @@ echo "Making logs directory and collecting logs."
[ -d ${LOG_LOCATION} ] || mkdir -p ${LOG_LOCATION}
sudo cp /var/log/libvirt/baremetal_logs/testvm[[:digit:]]_console.log ${LOG_LOCATION}
sudo chown $USER ${LOG_LOCATION}/testvm[[:digit:]]_console.log
sudo chmod o+r ${LOG_LOCATION}/testvm[[:digit:]]_console.log
dmesg &> ${LOG_LOCATION}/dmesg.log
# NOTE(TheJulia): Netstat exits with error code 5 when --version is used.
sudo netstat -apn &> ${LOG_LOCATION}/netstat.log
@ -32,3 +33,6 @@ else
fi
sudo chown $USER ${LOG_LOCATION}/ironic-api.log
sudo chown $USER ${LOG_LOCATION}/ironic-conductor.log
# In CI scenarios, we want other users to be able to read the logs.
sudo chmod o+r ${LOG_LOCATION}/ironic-api.log
sudo chmod o+r ${LOG_LOCATION}/ironic-conductor.log