[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