Merge pull request #487 from renanrodrigo/fix_actionsetfield_length

Fix Actions get_size method

Author: diraol

pyof/v0x04/common/action.py

@@ -1,6 +1,7 @@
 """Defines actions that may be associated with flows packets."""
 # System imports
 from enum import IntEnum
+from math import ceil
 
 # Local source tree imports
 from pyof.foundation.base import GenericStruct
@@ -107,6 +108,15 @@ def __init__(self, action_type=None, length=None):
         self.action_type = action_type
         self.length = length
 
+    def get_size(self, value=None):
+        """Return the action length including the padding (multiple of 8)."""
+        if isinstance(value, ActionHeader):
+            return value.get_size()
+        elif value is None:
+            current_size = super().get_size()
+            return ceil(current_size / 8) * 8
+        raise ValueError(f'Invalid value "{value}" for Action*.get_size()')
+
     def unpack(self, buff, offset=0):
         """Unpack a binary message into this object's attributes.