On 09/15/2010 07:01 PM, Ian Main wrote: > Log files were being created in a relative path to the WUI so I'm just > going to remove them as we no longer need them. We just use the event > log for events and can query condor with tools. > > Signed-off-by: Ian Main<[email protected]> > --- > src/app/util/condormatic.rb | 3 --- > 1 files changed, 0 insertions(+), 3 deletions(-) > > diff --git a/src/app/util/condormatic.rb b/src/app/util/condormatic.rb > index 3491f7f..e766231 100644 > --- a/src/app/util/condormatic.rb > +++ b/src/app/util/condormatic.rb > @@ -38,9 +38,6 @@ 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_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 > requirements += "\n" ACK. Looks good, though would you make one additional change. 'job_log' is still being defined on condormatic.rb line 29 but it is not being used anywhere. I verified that if you remove that line everything still works, so please remove it before you push. Thanks.
-Mo _______________________________________________ deltacloud-devel mailing list [email protected] https://fedorahosted.org/mailman/listinfo/deltacloud-devel
