Skip to content
This repository has been archived by the owner on Apr 22, 2024. It is now read-only.

Implement OF v0x04 (1.3.0) #158

Closed
diraol opened this issue Sep 6, 2016 · 11 comments
Closed

Implement OF v0x04 (1.3.0) #158

diraol opened this issue Sep 6, 2016 · 11 comments

Comments

@diraol
Copy link
Contributor

diraol commented Sep 6, 2016

For now, while we do not fix #157 , we will implement v0x04 (1.3.0) from the ground, by copying the v0x01 files and rewriting it.

The bellow list still need to be reviewed in comparison with the spec.

@diraol
Copy link
Contributor Author

diraol commented Sep 6, 2016

  • Asynchronous messages:
    • error_msg.py
    • flow_removed.py
    • packet_in.py
    • port_status.py

@diraol
Copy link
Contributor Author

diraol commented Sep 6, 2016

  • Common structures:
    • actions.py
    • constants.py
    • flow_match.py
    • flow_instructions.py
    • header.py
    • port.py
    • queue.py

@diraol
Copy link
Contributor Author

diraol commented Sep 6, 2016

  • Controller2Switch Messages
    • barrier_reply.py
    • barrier_request.py
    • common.py
      • Enums
        • ConfigFlags (Enum)
          • There are two different definitions of this enum on the spec.
        • ControllerRole (Enum)
        • MultipartTypes (Enum)
        • StatsTypes (Enum)
      • GenericStructs
        • AggregateStatsReply (GenericStruct)
        • AggregateStatsRequest (GenericStruct)
        • DescStats (GenericStruct)
        • FlowStats (GenericStruct)
        • FlowStatsRequest (GenericStruct)
        • ListOfActions (GenericStruct)
        • PortStats (GenericStruct)
        • PortStatsRequest (GenericStruct)
        • QueueStats (GenericStruct)
        • QueueStatsRequest (GenericStruct)
        • TableStats (GenericStruct)
      • GenericMessages (Base for other messages)
        • AsyncConfig (GenericMessage)
        • RoleBaseMessage (GenericMessage)
        • SwitchConfig (GenericMessage)
    • features_reply.py
      • Capabilities (Enum)
      • SwitchFeatures (GenericMessage)
      • FeaturesReply (GenericMessage)
    • features_request.py
    • flow_table.py (new)
    • flow_mod.py
      • FlowModCommand (Enum)
      • FlowModFlags (Enum)
      • FlowMod (GenericMessage)
    • flow_mod.py - flow_delete
    • flow_mod.py - flow_add
    • flow_stats.py
    • flow_stats_request.py
    • get_async_reply.py (new)
      • GetAsyncReply (GenericMessage)
    • get_async_request.py (new)
      • GetAsyncRequest (GenericMessage)
    • get_config_reply.py
      • GetConfigReply (GenericMessage)
    • get_config_request.py
      • GetConfigRequest (GenericMessage)
    • group_mod.py (new)
      • Bucket (Struct)
      • GroupModCommand (Enum)
      • GroupType (Enum)
      • ListOfBuckets (FixedTypeList)
      • GroupMod (GenericMessage)
    • meter_mod.py (new)
      • Meter (Enum)
      • MeterBandType (Enum)
      • MeterFlags (Enum)
      • MeterModCommand (Enum)
      • MeterBandDrop (GenericStruct)
      • MeterBandDscpRemark (GenericStruct)
      • MeterBandExperimenter (GenericStruct)
      • MeterBandHeader (GenericStruct)
      • MeterMod (GenericMessage)
    • multipart_request.py (new)
      • MultipartRequestFlags (Enum)
      • MultipartRequest (GenericMessage)
    • multipart_reply.py (new)
      • MultipartReplyFlags (Enum)
      • MultipartReply (GenericMessage)
    • packet_out.py
      • PacketOut (GenericMessage)
        • _review validations_
    • port_mod.py
      • PortMod (GenericMessage)
    • queue_get_config_reply.py
      • QueueGetConfigReply (GenericMessage)
    • queue_get_config_request.py
      • QueueGetConfigRequest (GenericMessage)
    • queue_stats.py
    • queue_stats_request.py
    • role_reply.py (new)
      • RoleReply (GenericMessage)
    • role_request.py (new)
      • RoleRequest (GenericMessage)
    • set_async.py (new)
      • SetAsync (GenericMessage)
    • set_config.py
    • stats_reply.py
    • stats_request.py
    • table_stats.py
    • table_mod.py (new)
      • Table (Enum)
      • TableMod (GenericMessage)

@diraol
Copy link
Contributor Author

diraol commented Sep 6, 2016

  • Symmetric messages
    • echo_reply.py
    • echo_request.py
    • hello.py
    • experimenter.py

@cemsbr
Copy link
Contributor

cemsbr commented Sep 6, 2016

Beginning Controller2Switch.

@diraol
Copy link
Contributor Author

diraol commented Sep 6, 2016

I've done the common/header.py and now I'm working on the symmetric messages.

@diraol
Copy link
Contributor Author

diraol commented Sep 6, 2016

Symmetric finished. Going to common. If any of you guys need to change a file on that module, just let me know. =)

@abaruchi
Copy link
Contributor

abaruchi commented Sep 6, 2016

Im doing the async messages.

@diraol
Copy link
Contributor Author

diraol commented Sep 12, 2016

Symmetric and Asynchronous modules done and reviews (for lint errors).

@abaruchi
Copy link
Contributor

GET_CONFIG_REPLY and GET_CONFIG_REQUEST messages type do not change in v0x04.

abaruchi added a commit that referenced this issue Sep 13, 2016
abaruchi added a commit that referenced this issue Sep 14, 2016
abaruchi added a commit that referenced this issue Sep 14, 2016
@abaruchi
Copy link
Contributor

set_config.py remains the same for v0x04, no changes required. Only enum flags must be updated (was already fixed).

diraol pushed a commit that referenced this issue Sep 14, 2016
beraldoleal added a commit that referenced this issue Sep 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants