[PATCH conductor] BZ #807267: aeolus-debug is broken.

Jason Guiditta jguiditt at redhat.com
Wed Jun 20 18:13:50 UTC 2012


https://bugzilla.redhat.com/show_bug.cgi?id=807267

Alarmist subject aside, the issue was actually that specifying the
--dir option was causing an error, standard run of aeolus-debug was
still working fine.

The cause of this was a simple error in how we were contructing the
path being passed into the tar command.
---
 src/lib/aeolus/debug/aeolus-debug |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lib/aeolus/debug/aeolus-debug b/src/lib/aeolus/debug/aeolus-debug
index cc45788..45224c1 100755
--- a/src/lib/aeolus/debug/aeolus-debug
+++ b/src/lib/aeolus/debug/aeolus-debug
@@ -78,13 +78,13 @@ system("rpm -qa | egrep 'aeolus|iwhd|deltacloud|imagefactory|oz' >> #{File.join(
 system("virsh capabilities >> #{File.join(target_dir, 'virsh_capabilities')}")
 
 # Tar the output and log it
-tarfile = target_dir + ".tar.gz"
+tarfile = timename + ".tar.gz"
 pwd = Dir.pwd()
 Dir.chdir(options[:basedir])
 system("tar -czvf #{tarfile} #{timename}")
 Dir.chdir(pwd)
 
-puts "A debug file has been created at #{tarfile}."
+puts "A debug file has been created at #{File.join(options[:basedir],timename)}."
 
 # Clean up
 FileUtils.rm_r(target_dir)
-- 
1.7.7.6




More information about the aeolus-devel mailing list