pio 0.30.0 → 0.30.1

data/lib/pio/open_flow10/vendor_action.rb

@@ -2,32 +2,34 @@ require 'bindata'
 require 'forwardable'
 
 module Pio
-  # Vendor defined action
-  class VendorAction
-    # OpenFlow 1.0 OFPAT_VENDOR action format.
-    class Format < BinData::Record
-      endian :big
+  module OpenFlow10
+    # Vendor defined action
+    class VendorAction
+      # OpenFlow 1.0 OFPAT_VENDOR action format.
+      class Format < BinData::Record
+        endian :big
 
-      uint16 :action_type, value: 0xffff
-      uint16 :action_length, value: 8
-      uint32 :vendor
-    end
+        uint16 :action_type, value: 0xffff
+        uint16 :action_length, value: 8
+        uint32 :vendor
+      end
 
-    def self.read(raw_data)
-      allocate.tap do |strip_vlan|
-        strip_vlan.instance_variable_set :@format, Format.read(raw_data)
+      def self.read(raw_data)
+        allocate.tap do |strip_vlan|
+          strip_vlan.instance_variable_set :@format, Format.read(raw_data)
+        end
       end
-    end
 
-    extend Forwardable
+      extend Forwardable
 
-    def_delegators :@format, :action_type
-    def_delegator :@format, :action_length, :length
-    def_delegators :@format, :vendor
-    def_delegator :@format, :to_binary_s, :to_binary
+      def_delegators :@format, :action_type
+      def_delegator :@format, :action_length, :length
+      def_delegators :@format, :vendor
+      def_delegator :@format, :to_binary_s, :to_binary
 
-    def initialize(vendor)
-      @format = Format.new(vendor: vendor)
+      def initialize(vendor)
+        @format = Format.new(vendor: vendor)
+      end
     end
   end
 end

data/lib/pio/open_flow13.rb

@@ -15,14 +15,21 @@ require 'pio/open_flow13/stats_request'
 # Actions
 require 'pio/open_flow/nicira_resubmit'
 require 'pio/open_flow/nicira_resubmit_table'
+require 'pio/open_flow13/copy_ttl_inwards'
+require 'pio/open_flow13/copy_ttl_outwards'
+require 'pio/open_flow13/decrement_ip_ttl'
 require 'pio/open_flow13/nicira_reg_load'
 require 'pio/open_flow13/nicira_reg_move'
 require 'pio/open_flow13/nicira_send_out_port'
+require 'pio/open_flow13/nicira_stack_pop'
+require 'pio/open_flow13/nicira_stack_push'
+require 'pio/open_flow13/nicira_conjunction'
 require 'pio/open_flow13/send_out_port'
 require 'pio/open_flow13/set_arp_operation'
 require 'pio/open_flow13/set_arp_sender_hardware_address'
 require 'pio/open_flow13/set_arp_sender_protocol_address'
 require 'pio/open_flow13/set_destination_mac_address'
+require 'pio/open_flow13/set_ip_ttl'
 require 'pio/open_flow13/set_metadata'
 require 'pio/open_flow13/set_source_mac_address'
 

data/lib/pio/open_flow13/actions.rb

@@ -1,47 +1,49 @@
 require 'bindata'
 
 module Pio
-  # Actions not yet implemented.
-  class UnsupportedAction < BinData::Record
-    endian :big
+  module OpenFlow13
+    # Actions not yet implemented.
+    class UnsupportedAction < BinData::Record
+      endian :big
 
-    uint16 :action_type
-    uint16 :action_length
-    string :body, length: -> { action_length - 4 }
+      uint16 :action_type
+      uint16 :action_length
+      string :body, length: -> { action_length - 4 }
 
-    def to_binary
-      to_binary_s
+      def to_binary
+        to_binary_s
+      end
     end
-  end
 
-  # Actions list of actions-apply instruction.
-  class Actions < BinData::Primitive
-    mandatory_parameter :length
+    # Actions list of actions-apply instruction.
+    class Actions13 < BinData::Primitive
+      mandatory_parameter :length
 
-    endian :big
+      endian :big
 
-    string :binary, read_length: :length
+      string :binary, read_length: :length
 
-    def set(actions)
-      self.binary = Array(actions).map(&:to_binary).join
-    end
+      def set(actions)
+        self.binary = Array(actions).map(&:to_binary).join
+      end
 
-    # rubocop:disable MethodLength
-    def get
-      actions = []
-      tmp = binary
-      while tmp.length > 0
-        action = case BinData::Uint16be.read(tmp)
-                 when 0
-                   SendOutPort.read(tmp)
-                 else
-                   UnsupportedAction.read(tmp)
-                 end
-        tmp = tmp[action.action_length..-1]
-        actions << action
+      # rubocop:disable MethodLength
+      def get
+        actions = []
+        tmp = binary
+        until tmp.empty?
+          action = case BinData::Uint16be.read(tmp)
+                   when 0
+                     OpenFlow13::SendOutPort.read(tmp)
+                   else
+                     UnsupportedAction.read(tmp)
+                   end
+          tmp = tmp[action.action_length..-1]
+          actions << action
+        end
+        actions
       end
-      actions
+      # rubocop:enable MethodLength
     end
-    # rubocop:enable MethodLength
   end
 end

data/lib/pio/open_flow13/apply.rb

@@ -1,36 +1,39 @@
 require 'bindata'
 require 'forwardable'
+require 'pio/open_flow/instruction'
 require 'pio/open_flow13/actions'
 
 module Pio
-  # An instruction to apply a list of actions to a packet in-order.
-  class Apply
-    # OpenFlow 1.3.4 OFPIT_APPLY_ACTIONS instruction format.
-    class Format < BinData::Record
-      endian :big
+  module OpenFlow13
+    # An instruction to apply a list of actions to a packet in-order.
+    class Apply < OpenFlow::Instruction
+      # OpenFlow 1.3.4 OFPIT_APPLY_ACTIONS instruction format.
+      class Format < BinData::Record
+        endian :big
 
-      uint16 :instruction_type, value: 4
-      uint16 :instruction_length,
-             initial_value: -> { 8 + actions.binary.length }
-      string :padding, length: 4
-      actions :actions, length: -> { instruction_length - 8 }
-    end
+        uint16 :instruction_type, value: 4
+        uint16 :instruction_length,
+               initial_value: -> { 8 + actions.binary.length }
+        string :padding, length: 4
+        actions13 :actions, length: -> { instruction_length - 8 }
+      end
 
-    def self.read(raw_data)
-      allocate.tap do |apply|
-        apply.instance_variable_set :@format, Format.read(raw_data)
+      def self.read(raw_data)
+        allocate.tap do |apply|
+          apply.instance_variable_set :@format, Format.read(raw_data)
+        end
       end
-    end
 
-    extend Forwardable
+      extend Forwardable
 
-    def_delegators :@format, :instruction_type
-    def_delegators :@format, :instruction_length
-    def_delegators :@format, :actions
-    def_delegators :@format, :to_binary_s
+      def_delegators :@format, :instruction_type
+      def_delegators :@format, :instruction_length
+      def_delegators :@format, :actions
+      def_delegators :@format, :to_binary_s
 
-    def initialize(actions = [])
-      @format = Format.new(actions: actions)
+      def initialize(actions = [])
+        @format = Format.new(actions: actions)
+      end
     end
   end
 end

data/lib/pio/open_flow13/copy_ttl_inwards.rb

@@ -0,0 +1,15 @@
+require 'pio/open_flow/action'
+
+module Pio
+  module OpenFlow13
+    # Copies TTL "inwards" -- from outermost to next-to-outermost
+    class CopyTtlInwards < OpenFlow::Action
+      action_header action_type: 12, action_length: 8
+      string :padding, length: 4
+
+      def initialize
+        super({})
+      end
+    end
+  end
+end

data/lib/pio/open_flow13/copy_ttl_outwards.rb

@@ -0,0 +1,15 @@
+require 'pio/open_flow/action'
+
+module Pio
+  module OpenFlow13
+    # Copies TTL "outwards" -- from next-to-outermost to outermost
+    class CopyTtlOutwards < OpenFlow::Action
+      action_header action_type: 11, action_length: 8
+      string :padding, length: 4
+
+      def initialize
+        super({})
+      end
+    end
+  end
+end

data/lib/pio/open_flow13/decrement_ip_ttl.rb

@@ -0,0 +1,15 @@
+require 'pio/open_flow/action'
+
+module Pio
+  module OpenFlow13
+    # Decrements IP TTL
+    class DecrementIpTtl < OpenFlow::Action
+      action_header action_type: 24, action_length: 8
+      string :padding, length: 4
+
+      def initialize
+        super({})
+      end
+    end
+  end
+end

data/lib/pio/open_flow13/echo/reply.rb

@@ -5,10 +5,10 @@ module Pio
     module Echo
       # OpenFlow 1.3 Echo Reply message.
       class Reply < OpenFlow::Message
-        open_flow_header version: 4, message_type: 3
-        string :body, read_length: -> { message_length - 8 }
+        open_flow_header version: 4, type: 3
+        string :body, read_length: -> { length - 8 }
 
-        alias_method :user_data, :body
+        alias user_data body
       end
     end
   end

data/lib/pio/open_flow13/echo/request.rb

@@ -5,10 +5,10 @@ module Pio
     module Echo
       # OpenFlow 1.3 Echo Request message.
       class Request < OpenFlow::Message
-        open_flow_header version: 4, message_type: 2
-        string :body, read_length: -> { message_length - 8 }
+        open_flow_header version: 4, type: 2
+        string :body, read_length: -> { length - header_length }
 
-        alias_method :user_data, :body
+        alias user_data body
       end
     end
   end

data/lib/pio/open_flow13/error.rb

@@ -1,33 +1,22 @@
+require 'pio/open_flow/error_message'
+require 'pio/open_flow13/error/bad_request'
 require 'pio/open_flow13/error/error_type13'
+require 'pio/open_flow13/error/hello_failed'
 
 module Pio
   module OpenFlow13
     # Error message parser
     module Error
+      mattr_reader(:type) { 1 }
+
+      extend OpenFlow::ErrorMessage
+
       # Error message body parser.
       class BodyParser < BinData::Record
         endian :big
         error_type13 :error_type
         uint16 :error_code
       end
-
-      # rubocop:disable MethodLength
-      def self.read(binary)
-        body = OpenFlowHeaderParser.read(binary).body
-        error = BodyParser.read(body).snapshot
-        klass = case error.error_type
-                when :hello_failed
-                  HelloFailed
-                when :bad_request
-                  BadRequest
-                else
-                  # Not implemented yet
-                  fail 'Unknown error message '\
-                       "(type=#{error.error_type}, code=#{error.error_code})"
-                end
-        klass.read binary
-      end
-      # rubocop:enable MethodLength
     end
   end
 end

data/lib/pio/open_flow13/error/bad_request.rb

@@ -1,3 +1,4 @@
+require 'pio/open_flow/error_message'
 require 'pio/open_flow/message'
 require 'pio/open_flow13/error/error_type13'
 
@@ -23,7 +24,7 @@ module Pio
             bad_port: 11,
             bad_packet: 12,
             multipart_buffer_overflow: 13
-          }
+          }.freeze
 
           endian :big
           uint16 :error_code
@@ -38,8 +39,8 @@ module Pio
         end
 
         open_flow_header version: 4,
-                         message_type: 1,
-                         message_length: -> { 12 + raw_data.length }
+                         type: OpenFlow::ErrorMessage.type,
+                         length: -> { 12 + raw_data.length }
         error_type13 :error_type, value: -> { :bad_request }
         bad_request_code :error_code
         rest :raw_data

data/lib/pio/open_flow13/error/error_type13.rb

@@ -19,7 +19,7 @@ module Pio
           meter_mod_failed: 12,
           table_features_failed: 13,
           experimenter: 0xffff
-        }
+        }.freeze
 
         endian :big
         uint16 :error_type

data/lib/pio/open_flow13/error/hello_failed.rb

@@ -1,3 +1,4 @@
+require 'pio/open_flow/error_message'
 require 'pio/open_flow/hello_failed_code'
 require 'pio/open_flow/message'
 require 'pio/open_flow13/error/error_type13'
@@ -8,8 +9,8 @@ module Pio
       # Hello Failed error message
       class HelloFailed < OpenFlow::Message
         open_flow_header version: 4,
-                         message_type: 1,
-                         message_length: -> { 12 + description.length }
+                         type: OpenFlow::ErrorMessage.type,
+                         length: -> { 12 + description.length }
         error_type13 :error_type
         hello_failed_code :error_code
         rest :description

data/lib/pio/open_flow13/features/reply.rb

@@ -1,3 +1,4 @@
+require 'pio/open_flow/datapath_id'
 require 'pio/open_flow/message'
 
 module Pio
@@ -6,8 +7,14 @@ module Pio
     class Features
       # Features Reply message.
       class Reply < OpenFlow::Message
-        extend OpenFlow::Flags
-
+        open_flow_header version: 4, type: 6, length: 32
+        datapath_id :datapath_id
+        alias dpid datapath_id
+        uint32 :n_buffers
+        uint8 :n_tables
+        uint8 :auxiliary_id
+        uint16 :padding
+        hide :padding
         flags_32bit(:capabilities,
                     [:flow_stats,
                      :table_stats,
@@ -18,21 +25,7 @@ module Pio
                      :queue_stats,
                      :NOT_USED,
                      :port_blocked])
-
-        open_flow_header version: 4, message_type: 6, message_length: 32
-        datapath_id :datapath_id
-        uint32 :n_buffers
-        uint8 :n_tables
-        uint8 :auxiliary_id
-        uint16 :padding
-        hide :padding
-        capabilities :capabilities
         uint32 :reserved
-
-        def datapath_id
-          @format.datapath_id.to_i
-        end
-        alias_method :dpid, :datapath_id
       end
     end
   end

data/lib/pio/open_flow13/features/request.rb

@@ -6,7 +6,7 @@ module Pio
     class Features
       # Features Request message.
       class Request < OpenFlow::Message
-        open_flow_header version: 4, message_type: 5
+        open_flow_header version: 4, type: 5
         string :body, value: ''
       end
     end