move save_logs to ci_cleanup

Change-Id: Ibfaea6c3016a189855db981f55060afff8a6db87
Ok, I'll test it.

From: Ananth Suryanarayana <notifications@github.com<mailto:notifications@github.com>>
Reply-To: Juniper/contrail-infra-config <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Tuesday, August 26, 2014 at 12:54 PM
To: Juniper/contrail-infra-config <contrail-infra-config@noreply.github.com<mailto:contrail-infra-config@noreply.github.com>>
Cc: Kamlesh Parmar <kparmar@juniper.net<mailto:kparmar@juniper.net>>
Subject: Re: [contrail-infra-config] move save_logs to ci_cleanup (#30)


In modules/jenkins/files/slave_scripts/ci-infra/util.rb:

> @@ -263,6 +263,18 @@ def self.wait
>      def self.cleanup_on_exit() return @@cleanup_on_exit end
>      def self.cleanup_on_exit=(flag) @@cleanup_on_exit = flag end
>
> +    # Archive /var/log contents.
> +    def save_logs(build_id)
> +        filename = 'var_logs.tar.bz2'
> +        logfile = "/tmp/#{filename}"
> +        logdir = "/ci-admin/failed_systest_logs/#{build_id}"
> +        Sh.run("ssh #{@vms.first.vmname} tar --ignore-failed-read -cjf #{logfile} /var/log", true)


I think this should be @@vms<https://github.com/vms> as you moved this to a different class

-
Reply to this email directly or view it on GitHub<https://github.com/Juniper/contrail-infra-config/pull/30/files#r16737852>.