ACK. This is much better solution than my patch.

Thomas


----- Original Message -----
From: [email protected]
To: [email protected]
Sent: Tuesday, July 13, 2010 11:02:57 AM GMT +01:00 Amsterdam / Berlin / Bern / 
Rome / Stockholm / Vienna
Subject: [deltacloud-devel] [PATCH aggregator] Changed condor logfile location.

From: Jan Provaznik <[email protected]>

---
 src/app/util/condormatic.rb |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/app/util/condormatic.rb b/src/app/util/condormatic.rb
index e994504..81cf2b9 100644
--- a/src/app/util/condormatic.rb
+++ b/src/app/util/condormatic.rb
@@ -24,6 +24,7 @@ def condormatic_instance_create(task)
     realm = instance.realm rescue nil
 
     job_name = "job_#{instance.name}_#{instance.id}"
+    job_log = File.join('log', "condor-#{job_name}.log")
 
 
     # I use the 2>&1 to get stderr and stdout together because popen3 does not 
support
@@ -35,8 +36,8 @@ def condormatic_instance_create(task)
     Rails.logger.info "executable = #{job_name}\n"
     pipe.puts "grid_resource = dcloud $$(provider_url) $$(username) 
$$(password) $$(image_key) #{instance.name} $$(realm_key) 
$$(hardwareprofile_key)\n"
     Rails.logger.info "grid_resource = dcloud $$(provider_url) $$(username) 
$$(password) $$(image_key) #{instance.name} $$(realm_key) 
$$(hardwareprofile_key)\n"
-    pipe.puts "log = #{job_name}.log\n"
-    Rails.logger.info "log = #{job_name}.log\n"
+    pipe.puts "log = #{job_log}\n"
+    Rails.logger.info "log = #{job_log}\n"
 
     requirements = "requirements = hardwareprofile == 
\"#{instance.hardware_profile.id}\" && image == \"#{instance.image.id}\""
     requirements += " && realm == \"#{realm.name}\"" if realm != nil
-- 
1.7.0.1

_______________________________________________
deltacloud-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/deltacloud-devel
_______________________________________________
deltacloud-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/deltacloud-devel

Reply via email to