rsmp 0.1.21 → 0.1.27

data/lib/rsmp/site.rb

@@ -26,9 +26,9 @@ module RSMP
         'supervisors' => [
           { 'ip' => '127.0.0.1', 'port' => 12111 }
         ],
-        'rsmp_versions' => ['3.1.1','3.1.2','3.1.3','3.1.4'],
-        'sxl' => 'traffic_light_controller',
-        'sxl_version' => '1.0.7',
+        'rsmp_versions' => ['3.1.1','3.1.2','3.1.3','3.1.4','3.1.5'],
+        'sxl' => 'tlc',
+        'sxl_version' => '1.0.15',
         'timer_interval' => 0.1,
         'watchdog_interval' => 1,
         'watchdog_timeout' => 2,
@@ -54,9 +54,17 @@ module RSMP
                   :acknowledgement_timeout,:command_response_timeout]
       check_required_settings @site_settings, required
 
+      check_sxl_version
+
       setup_components @site_settings['components']
     end
 
+    def check_sxl_version
+      sxl = @site_settings['sxl']
+      version = @site_settings['sxl_version']
+      RSMP::Schemer::find_schema! sxl, version
+    end
+
     def reconnect
       @sleep_condition.signal
     end
@@ -64,8 +72,10 @@ module RSMP
     def start_action
       @site_settings["supervisors"].each do |supervisor_settings|
         @task.async do |task|
-          task.annotate "site_proxy"
+          task.annotate "site proxy"
           connect_to_supervisor task, supervisor_settings
+        rescue StandardError => e
+          notify_error e, level: :internal
         end
       end
     end
@@ -101,10 +111,8 @@ module RSMP
         proxy.run       # run until disconnected
       rescue IOError => e
         log "Stream error: #{e}", level: :warning
-      rescue SystemCallError => e # all ERRNO errors
-        log "Reader exception: #{e.to_s}", level: :error
       rescue StandardError => e
-        log ["Reader exception: #{e}",e.backtrace].flatten.join("\n"), level: :error
+        notify_error e, level: :internal
       ensure
         begin
           if @site_settings["reconnect_interval"] != :no
@@ -135,7 +143,7 @@ module RSMP
     def starting
       log "Starting site #{@site_settings["site_id"]}",
           level: :info,
-          timestamp: RSMP.now_object
+          timestamp: @clock.now
     end
 
     def alarm

data/lib/rsmp/site_proxy.rb

@@ -15,6 +15,11 @@ module RSMP
       @site_id = nil
     end
 
+    def inspect
+      "#<#{self.class.name}:#{self.object_id}, #{inspector(
+        :@acknowledgements,:@settings,:@site_settings,:@components
+        )}>"
+    end
     def node
       supervisor
     end
@@ -31,7 +36,7 @@ module RSMP
 
     def connection_complete
       super
-      log "Connection to site #{@site_id} established", level: :info
+      log "Connection to site #{@site_id} established, using core #{@rsmp_version}, sxl #{@sxl} #{@site_sxl_version}", level: :info
     end
 
     def process_message message
@@ -43,6 +48,8 @@ module RSMP
           will_not_handle message
         when AggregatedStatus
           process_aggregated_status message
+        when AggregatedStatusRequest
+          will_not_handle message
         when Alarm
           process_alarm message
         when CommandResponse
@@ -66,18 +73,45 @@ module RSMP
     end
 
     def version_accepted message
-      log "Received Version message for site #{@site_id} using RSMP #{@rsmp_version}", message: message, level: :log
+      if @settings['sites']
+        @site_settings = @settings['sites'][@site_id]
+        @site_settings = @settings['sites'][:any] unless @site_settings
+        if @site_settings
+          @sxl = @site_settings['sxl']
+          setup_components @site_settings['components']
+        else
+          dont_acknowledge message, 'Rejected', "No config found for site #{@site_id}"
+        end
+      end
+
+      log "Received Version message for site #{@site_id}", message: message, level: :log
       start_timer
       acknowledge message
       send_version @site_id, @settings['rsmp_versions']
       @version_determined = true
 
-      if @settings['sites']
-        @site_settings = @settings['sites'][@site_id]
-        @site_settings =@settings['sites'][:any] unless @site_settings
-        if @site_settings
-          setup_components @site_settings['components']
+    end
+
+    def request_aggregated_status component, options={}
+      raise NotReady unless ready?
+      m_id = options[:m_id] || RSMP::Message.make_m_id
+
+      message = RSMP::AggregatedStatusRequest.new({
+          "ntsOId" => '',
+          "xNId" => '',
+          "cId" => component,
+          "mId" => m_id
+      })
+      if options[:collect]
+        result = nil
+        task = @task.async do |task|
+          wait_for_aggregated_status task, options[:collect]
         end
+        send_message message, validate: options[:validate]
+        return message, task.wait
+      else
+        send_message message, validate: options[:validate]
+        message
       end
     end
 
@@ -159,10 +193,16 @@ module RSMP
           collect_options = options[:collect].merge status_list: status_list
           collect_status_responses task, collect_options, m_id
         end
-        send_message message
-        return message, task.wait
+        send_message message, validate: options[:validate]
+
+        # task.wait return the result of the task. if the task raised an exception
+        # it will be reraised. but that mechanish does not work if multiple values
+        # are returned. so manually raise if first element is an exception
+        result = task.wait
+        raise result.first if result.first.is_a? Exception
+        return message, *result
       else
-        send_message message
+        send_message message, validate: options[:validate]
         message
       end
     end
@@ -193,15 +233,21 @@ module RSMP
           collect_options = options[:collect].merge status_list: status_list
           collect_status_updates task, collect_options, m_id
         end
-        send_message message
-        return message, task.wait
+        send_message message, validate: options[:validate]
+
+        # task.wait return the result of the task. if the task raised an exception
+        # it will be reraised. but that mechanish does not work if multiple values
+        # are returned. so manually raise if first element is an exception
+        result = task.wait
+        raise result.first if result.first.is_a? Exception
+        return message, *result
       else
-        send_message message
+        send_message message, validate: options[:validate]
         message
       end
     end
 
-    def unsubscribe_to_status component, status_list
+    def unsubscribe_to_status component, status_list, options={}
       raise NotReady unless ready?
       message = RSMP::StatusUnsubscribe.new({
           "ntsOId" => '',
@@ -209,7 +255,7 @@ module RSMP
           "cId" => component,
           "sS" => status_list
       })
-      send_message message
+      send_message message, validate: options[:validate]
       message
     end
 
@@ -218,7 +264,7 @@ module RSMP
       acknowledge message
     end
 
-    def send_alarm_acknowledgement component, alarm_code
+    def send_alarm_acknowledgement component, alarm_code, options={}
       message = RSMP::AlarmAcknowledged.new({
           "ntsOId" => '',
           "xNId" => '',
@@ -228,7 +274,7 @@ module RSMP
           "xNACId" => '',
           "aSp" => 'Acknowledge'
       })
-      send_message message
+      send_message message, validate: options[:validate]
       message
     end
 
@@ -248,10 +294,16 @@ module RSMP
           collect_options = options[:collect].merge command_list: command_list
           collect_command_responses task, collect_options, m_id
         end
-        send_message message
-        return message, task.wait
+        send_message message, validate: options[:validate]
+
+        # task.wait return the result of the task. if the task raised an exception
+        # it will be reraised. but that mechanish does not work if multiple values
+        # are returned. so manually raise if first element is an exception
+        result = task.wait
+        raise result.first if result.first.is_a? Exception
+        return message, *result
       else
-        send_message message
+        send_message message, validate: options[:validate]
         message
       end
     end
@@ -261,6 +313,14 @@ module RSMP
     end
 
     def check_sxl_version message
+
+      # check that we have a schema for specified sxl type and version
+      # note that the type comes from the site config, while the version
+      # comes from the Version message send by the site
+      type = 'tlc'
+      version = message.attribute 'SXL'
+      RSMP::Schemer::find_schema! type, version 
+
       # store sxl version requested by site
       # TODO should check agaist site settings
       @site_sxl_version = message.attribute 'SXL'
@@ -278,6 +338,8 @@ module RSMP
       check_rsmp_version message
       check_sxl_version message
       version_accepted message
+    rescue RSMP::Schemer::UnknownSchemaError => e
+      dont_acknowledge message, "Rejected #{message.type} message,", "#{e}"
     end
 
     def check_site_ids message
@@ -288,5 +350,9 @@ module RSMP
       site_ids_changed
     end
 
+    def notify_error e, options={}
+      @supervisor.notify_error e, options if @supervisor
+    end
+
   end
 end

data/lib/rsmp/site_proxy_wait.rb

@@ -20,12 +20,12 @@ module RSMP
       end
     end
 
-    def wait_for_alarm options={}
+    def wait_for_alarm parent_task, options={}
       matching_alarm = nil
-      item = collect(@task,options.merge(type: "Alarm", with_message: true, num: 1)) do |item|
+      message = collect(parent_task,options.merge(type: "Alarm", with_message: true, num: 1)) do |message|
         # TODO check components
         matching_alarm = nil
-        alarm = item[:message]
+        alarm = message
         next if options[:aCId] && options[:aCId] != alarm.attribute("aCId")
         next if options[:aSp] && options[:aSp] != alarm.attribute("aSp")
         next if options[:aS] && options[:aS] != alarm.attribute("aS")
@@ -33,7 +33,7 @@ module RSMP
         break
       end
       if item
-        { message: item[:message], status: matching_alarm }
+        { message: message, status: matching_alarm }
       end
     end
 
@@ -49,11 +49,11 @@ module RSMP
       task.annotate "wait for command response"
       want = options[:command_list].clone
       result = {}
-      item = collect(parent_task,options.merge({
+      messages = []
+      collect(parent_task,options.merge({
         type: ['CommandResponse','MessageNotAck'],
         num: 1
-      })) do |item|
-        message = item[:message]
+      })) do |message|
         if message.is_a?(MessageNotAck)
           if message.attribute('oMId') == m_id
             # set result to an exception, but don't raise it.
@@ -68,39 +68,38 @@ module RSMP
             false
           end
         else
-          found = []
+          add = false
           # look through querues
           want.each_with_index do |query,i|
             # look through items in message
-            item[:message].attributes['rvs'].each do |input|
-              ok = command_match? query, input
-              if ok
+            message.attributes['rvs'].each do |input|
+              matching = command_match? query, input
+              if matching == true
                 result[query] = input
-                found << i   # record which queries where matched succesfully
+                add = true
+              elsif matching == false
+                result.delete query
               end
             end
           end
-          # remove queries that where matched
-          found.sort.reverse.each do |i|
-            want.delete_at i
-          end
-          want.empty? # any queries left to match?
+          messages << message if add
+          result.size == want.size # any queries left to match?
         end
       end
-      result
+      return result, messages
     rescue Async::TimeoutError
-      raise RSMP::TimeoutError.new "Did not receive command response to #{m_id} within #{options[:timeout]}s"
+      raise RSMP::TimeoutError.new "Did not receive correct command response to #{m_id} within #{options[:timeout]}s"
     end
 
     def collect_status_updates_or_responses task, type, options, m_id
-      want = options[:status_list]
+      want = options[:status_list].clone
       result = {}
+      messages = []
       # wait for a status update
-      item = collect(task,options.merge({
+      collect(task,options.merge({
         type: [type,'MessageNotAck'],
         num: 1
-      })) do |item|
-        message = item[:message]
+      })) do |message|
         if message.is_a?(MessageNotAck)
           if message.attribute('oMId') == m_id
             # set result to an exception, but don't raise it.
@@ -115,33 +114,33 @@ module RSMP
           false
         else
           found = []
+          add = false
           # look through querues
           want.each_with_index do |query,i|
             # look through status items in message
-            item[:message].attributes['sS'].each do |input|
-              ok = status_match? query, input
-              if ok
+            message.attributes['sS'].each do |input|
+              matching = status_match? query, input
+              if matching == true
                 result[query] = input
-                found << i   # record which queries where matched succesfully
+                add = true
+              elsif matching == false
+                result.delete query
               end
             end
           end
-          # remove queries that where matched
-          found.sort.reverse.each do |i|
-            want.delete_at i
-          end
-          want.empty? # any queries left to match?
+          messages << message if add
+          result.size == want.size # any queries left to match?
         end
       end
-      result
+      return result, messages
     rescue Async::TimeoutError
       type_str = {'StatusUpdate'=>'update', 'StatusResponse'=>'response'}[type]
-      raise RSMP::TimeoutError.new "Did not received status #{type_str} in reply to #{m_id} within #{options[:timeout]}s"
+      raise RSMP::TimeoutError.new "Did not received correct status #{type_str} in reply to #{m_id} within #{options[:timeout]}s"
     end
 
     def status_match? query, item
-      return false if query['sCI'] && query['sCI'] != item['sCI']
-      return false if query['n'] && query['n'] != item['n']
+      return nil if query['sCI'] && query['sCI'] != item['sCI']
+      return nil if query['n'] && query['n'] != item['n']
       return false if query['q'] && query['q'] != item['q']
       if query['s'].is_a? Regexp
         return false if query['s'] && item['s'] !~ query['s']
@@ -152,8 +151,8 @@ module RSMP
     end
 
     def command_match? query, item
-      return false if query['cCI'] && query['cCI'] != item['cCI']
-      return false if query['n'] && query['n'] != item['n']
+      return nil if query['cCI'] && query['cCI'] != item['cCI']
+      return nil if query['n'] && query['n'] != item['n']
       if query['v'].is_a? Regexp
         return false if query['v'] && item['v'] !~ query['v']
       else
@@ -168,6 +167,8 @@ module RSMP
       # wait for command responses in an async task
       task = parent_task.async do |task|
         collect_block.call task, m_id
+      rescue StandardError => e
+        notify_error e, level: :internal
       end
 
        # call block, it should send command request using the given m_id
@@ -177,5 +178,29 @@ module RSMP
       task.wait
     end
 
+    def wait_for_aggregated_status parent_task, options={}
+      collect(parent_task,options.merge({
+        type: ['AggregatedStatus','MessageNotAck'],
+        num: 1
+      })) do |message|
+        if message.is_a?(MessageNotAck)
+          if message.attribute('oMId') == m_id
+            # set result to an exception, but don't raise it.
+            # this will be returned by the task and stored as the task result
+            # when the parent task call wait() on the task, the exception
+            # will be raised in the parent task, and caught by rspec.
+            # rspec will then show the error and record the test as failed
+            m_id_short = RSMP::Message.shorten_m_id m_id, 8
+            result = RSMP::MessageRejected.new "Aggregated status request #{m_id_short} was rejected: #{message.attribute('rea')}"
+            next true   # done, no more messages wanted
+          else
+            false
+          end
+        else
+          true
+        end
+      end
+    end
+
   end
 end