-
Notifications
You must be signed in to change notification settings - Fork 56
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
Uperf scale variables #250
Conversation
* added fields for node id, density, and node count * pass variables all the way * add data as part of datapoitns * remove typo and redundant field * add fields from the cluster resource file * remove duplicate variable * add node and density range
Also, can we squash? |
Usually you can squash merge, its a dropdown arrow inside the green commit button. |
yes! I have seen that glorious button! I actually have used it in the past too! for a simple change that doesn't seem to add a considerable amount of code, I thought a squash was simple... |
@jtaleric maybe I should rebase after making changes suggested here, hopefully the magic button will appear. |
* add fields from the cluster resource file
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.
Replied on the comment about why I would like to see the ranges as lists/arrays.
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.
Thank you for making the requested changes, LGTM
"networkpolicy": self.networkpolicy | ||
"networkpolicy": self.networkpolicy, | ||
"density": int(self.pod_density), | ||
"nodes_in_iter":int(self.nodes_in_iter), |
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 gives error as pod_density default value is ""
@@ -54,6 +54,18 @@ def __init__(self, parent_parser): | |||
self.args.num_pairs = os.getenv("num_pairs", "") | |||
self.args.multus_client = os.getenv("multus_client", "") | |||
self.args.networkpolicy = os.getenv("networkpolicy", "") | |||
self.args.nodes_in_iter = os.getenv("node_count", "") | |||
self.args.pod_density = os.getenv("pod_count", "") |
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.
initialize pod_count to default value 0 to avoid above type conversion error
These changes add the new variables from the uperf-scale enhancements to benchmark-operator. Since there are defaults provided in the python calls, I assume this change is not dependent on the forthcoming pr to benchmark operator. I am also planning on squash merging, as the intermediate commit messages are not always descriptive.