-
Notifications
You must be signed in to change notification settings - Fork 17
RoCEv2 OTG model [latest] #415
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
As discussed, PFC Rx stats are important for RoCEv2 testing. Please add them in port stats. |
@kunkeysight : This should be added in a separate PR since it is an additional feature as far as I understand. I think might be better to first complete RoCEv2 control plane + traffic based on this PR with dev builds to allow merging of PR and initial delivery . And then separately implement PFC tx/rx stats in whichever order that is needed. Can happen in parallel as well . |
@satyamsinghKey -> Comments on the example and Introduction.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more round of review since it is probably getting closer to merge and dont see any new reviews/review-requests.
I think at least some of the documentation still needs significant improvement esp. for flow side and some typos / extra fields probably need a re-check before we can get to merge-able state.
x-field-uid: 2 | ||
Rocev2.TransmitType: | ||
description: |- | ||
RoCEv2 flows can be configured to run in continuous mode or fixed iteration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This description is wrong. Needs to be fixed . On the lines of something like :
Specification of the traffic profile for flows from QPs on this port.
Currently , only target_line_rate mode is supported. When this mode is selected, all the configured QPs on this port will send data packets as per configured message_size with the messages from different flows/QPs being interleaved ( this is JUST an example ... needs exact expected technical specification in terms of RoCEv2 which should be provided by someone with proper understanding of how to set expectations for this mode in a generic manner ) with each other such that the cumulative rate for all packets being transmitted across all flows never exceed the target_line_rate.value ( exact field must be specified , these are just examples of what sort of info I would expect in this text ) . The actual rate achieved will depend on congestion encountered by DUT which can lead to decrease of actual packet rate achieved due to congestion reduction mechanisms such as DCQCN ( triggered by ECN bits indicating congestions from the DUT ) or due to PFC congestion notifications from DUT.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some responses to comments received.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blocking comments handled. So removing blocking tag.
Some changes still expected in Descriptions.
Approved. |
Issue: #403
Redocly View:
https://redocly.github.io/redoc/?url=https://raw.githubusercontent.com/open-traffic-generator/models/dev2_rocev2/artifacts/openapi.yaml&nocors#tag/Configuration/operation/set_config
Objective: Add support for RoCEv2 protocol
Sample Code:
This sample snippet will be constantly updated.
`
`
NOTE:
A dummy field exists in options.per_port_options.port_settings.protocols.dummy_field. Please ignore this field for now, as it is present due to a compilation error we have identified and are currently investigating.
Some sections/fields do not have descriptions, and you may come across the placeholder text "Description here". If you are familiar with these fields, please suggest appropriate descriptions.
This PR is created new due to major modification required in the model.
Link to view previous PR : (RoCEv2 OTG Model #405)