Skip to content
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

Add support for random modifiers on fields #324

Open
rmurthy716 opened this issue Feb 9, 2022 · 4 comments
Open

Add support for random modifiers on fields #324

rmurthy716 opened this issue Feb 9, 2022 · 4 comments
Labels
enhancement New feature or request spec Impacts API Spec
Milestone

Comments

@rmurthy716
Copy link
Collaborator

Can we support choice: random for all the different modifiers?

@rmurthy716 rmurthy716 added the enhancement New feature or request label Feb 9, 2022
@rmurthy716
Copy link
Collaborator Author

Changes are required in artifacts/openapi.yml from my understanding. I can make the necessary changes if we are ok with adding this to the model.

@ajbalogh
Copy link
Contributor

ajbalogh commented Feb 9, 2022

Hi @rmurthy716 - to add a choice:random will require changes in the openapiart repo under x-field-pattern extension.
Random was initially proposed for the x-field-pattern but was subsequently dropped as it was decided to leave it up to the tester to supply values that were random.
If the extension is to be extended then random should be qualified - ie repeatable random, random etc.

@ajbalogh ajbalogh transferred this issue from open-traffic-generator/models Feb 9, 2022
@ankur-sheth
Copy link
Contributor

Right. The other reason was that we needed all OTG implementations to generate the same stream of packets. So 'random' for Ixia-c would have to generate the same 'random' for say TRex. What are our use-cases for random?

@rmurthy716
Copy link
Collaborator Author

The use case is for load balancing hashing algorithms with random TCP ports. This is coming from a scan of a test plan shared with us.

@ashutshkumr ashutshkumr added the help wanted Extra attention is needed label Nov 11, 2022
@ashutshkumr ashutshkumr transferred this issue from open-traffic-generator/openapiart Jul 18, 2023
@ashutshkumr ashutshkumr added spec Impacts API Spec and removed help wanted Extra attention is needed labels Jul 26, 2023
@ashutshkumr ashutshkumr added this to the v1.0.0 milestone Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request spec Impacts API Spec
Projects
Status: Backlog
Development

No branches or pull requests

4 participants