LGTM, thanks
On Jan 9, 2014 5:52 PM, "Klaus Aehlig" <[email protected]> wrote:

> When running vlcuster QAs, most of the interesting logs
> end up in /srv/ganeti/vcluster. So this directory, in present,
> should also be included in the logs.
>
> Signed-off-by: Klaus Aehlig <[email protected]>
> ---
>  roles/buildmaster/files/srv/buildbot/lib/scripts/collect-logs | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/roles/buildmaster/files/srv/buildbot/lib/scripts/collect-logs
> b/roles/buildmaster/files/srv/buildbot/lib/scripts/collect-logs
> index 58cc677..591fd0b 100755
> --- a/roles/buildmaster/files/srv/buildbot/lib/scripts/collect-logs
> +++ b/roles/buildmaster/files/srv/buildbot/lib/scripts/collect-logs
> @@ -17,6 +17,13 @@ for host; do
>    rsync -av \
>      root@$host:/var/log/ganeti/ $hostdir/var/log/ganeti/
>
> +  # Also collect vcluster logs, if present
> +  if ssh root@$host test -d /srv/ganeti/vcluster; then
> +      mkdir -p $hostdir/srv/ganeti/vcluster
> +      rsync -av \
> +          root@$host:/srv/ganeti/vcluster/ $hostdir/srv/ganeti/vcluster
> +  fi
> +
>    ssh root@$host 'cat /proc/drbd || :' >$hostdir/proc-drbd.txt 2>&1
>    ssh root@$host 'dmsetup info || :' >$hostdir/dmsetup-info.txt 2>&1
>  done
> --
> 1.8.5.1
>
>

Reply via email to