Improve error-handling

[rails.git] / lib / daemons / gpx_import.rb

diff --git a/lib/daemons/gpx_import.rb b/lib/daemons/gpx_import.rb
index 1fc60858a3cebb423fa0aca1f2dc5ebd7e7576f4..698e858ba375c27cf1d592bdf8e84617d371af6c 100755 (executable)
--- a/lib/daemons/gpx_import.rb
+++ b/lib/daemons/gpx_import.rb
@@ -1,56 +1,59 @@
 #!/usr/bin/env ruby
 
 #You might want to change this
 #!/usr/bin/env ruby
 
 #You might want to change this

-ENV["RAILS_ENV"] ||= "development"
+#ENV["RAILS_ENV"] ||= "development"

 
 require File.dirname(__FILE__) + "/../../config/environment"
 
 
 require File.dirname(__FILE__) + "/../../config/environment"
 

-$running = true;
-Signal.trap("TERM") do 
-  $running = false
-end
+terminated = false

 
 logger = ActiveRecord::Base.logger
 
 
 logger = ActiveRecord::Base.logger
 

-while($running) do
-  
+while(true) do

   ActiveRecord::Base.logger.info("GPX Import daemon wake @ #{Time.now}.")
 
   ActiveRecord::Base.logger.info("GPX Import daemon wake @ #{Time.now}.")
 

-  traces = Trace.find(:all, :conditions => ['inserted = ?', false])
-
-  if traces and traces.length > 0
-    traces.each do |trace|
-      begin
-
-        logger.info("GPX Import importing #{trace.name} from #{trace.user.email}")
-
-        gzipped = `file -b /tmp/#{trace.id}.gpx`.chomp =~/^gzip/
-
-        if gzipped
-          logger.info("gzipped")
-        else
-          logger.info("not gzipped")
-        end
-        gpx = OSM::GPXImporter.new("/tmp/#{trace.id}.gpx")
-
-        gpx.points do |point|
-          tp = Tracepoint.new
-          tp.latitude = point['latitude']
-          tp.latitude = point['longitude']
-          tp.altitude = point['altitude']
-          tp.user_id = trace.user.id
-          tp.gpx_id = trace.id
-          tp.trackid = point['segment']
-        end 
-        trace.size = gpx.actual_points
-        trace.inserted = true
-        trace.save
-        Notifier::deliver_gpx_success(trace, gpx.possible_points)
-      rescue Exception => ex
+  Trace.find(:all, :conditions => { :inserted => false, :visible => true }, :order => "id").each do |trace|
+    Signal.trap("TERM") do 
+      terminated = true
+    end
+
+    begin
+      gpx = trace.import
+
+      if gpx.actual_points > 0
+        Notifier.gpx_success(trace, gpx.actual_points).deliver
+      else
+        Notifier.gpx_failure(trace, '0 points parsed ok. Do they all have lat,lng,alt,timestamp?').deliver

         trace.destroy
         trace.destroy

-        Notifier::deliver_gpx_failure(trace, ex.to_s)

       end
       end

+    rescue Exception => ex
+      logger.info ex.to_s
+      ex.backtrace.each {|l| logger.info l }
+      Notifier.gpx_failure(trace, ex.to_s + "\n" + ex.backtrace.join("\n")).deliver
+      trace.destroy
+    end
+
+    Signal.trap("TERM", "DEFAULT")
+
+    exit if terminated
+  end
+
+  Trace.find(:all, :conditions => { :visible => false }, :order => "id").each do |trace|
+    Signal.trap("TERM") do 
+      terminated = true
+    end
+
+    begin
+      trace.destroy
+    rescue Exception => ex
+      logger.info ex.to_s
+      ex.backtrace.each {|l| logger.info l }

     end
     end

+
+    Signal.trap("TERM", "DEFAULT")
+
+    exit if terminated

   end
   end

-  sleep 15.minutes
+
+  sleep 5.minutes.value

 end
 end