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

Steve Linabery slinaber at redhat.com
Thu Jun 28 17:29:42 UTC 2012


On Wed, Jun 20, 2012 at 02:13:50PM -0400, Jason Guiditta wrote:
> 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
> 

ACK. This fix works for me. Running 'aeolus-debug -d ~/fubar' resulted in a tarball in that directory.

Pushing to 'master'.

Steve



More information about the aeolus-devel mailing list