On Thu, 2010-09-16 at 16:33 -0400, Mohammed Morsi wrote:
> 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.

Pushed with change, thanks!

        Ian


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

Reply via email to