complex uperf net_policy with IP and ports#4
Open
mukrishn wants to merge 41 commits intoHughNhan:hnhan-uperf-scalefrom
Open
complex uperf net_policy with IP and ports#4mukrishn wants to merge 41 commits intoHughNhan:hnhan-uperf-scalefrom
mukrishn wants to merge 41 commits intoHughNhan:hnhan-uperf-scalefrom
Conversation
Model-S supports "node_range: [n, m]" and "density_range: [x, y]" to specify enumeration by both dimension.
…rk context. Fix last commit which introduced node_range and density_range but inadvertedly still used min_node and max_node for condition check i.e "when: max_node is defined"
…a correlation. While at it beefup a few place with "when: xxx is defined" for robustness
Valid step_size values are: addN or log2. N can be any decimal number.
enter main loop (due to start-True) then acquire pod_idx and node_idx. They miss out the restart state, and run prematurely
to reduce redis load.
This fits in with the idea that we are viewing worker nodes as just a pool of resources and not by individual hostnames. This gives the flexibility for users to isolate the tests to only one hardware model of workers as well (since each model can be labelled with its model name). Also there was a bug in previous code where only the first node in the list was technically being excluded in `workload_args.excluded_node[0]`. Signed-off-by: Sai Sindhur Malleni <smalleni@redhat.com>
Increase redis-server to 2 CPU. Add sleep's to workload/uperf clients Point operator resource spec to compatible image.
Signed-off-by: Sai Sindhur Malleni <smalleni@redhat.com>
***Important*** pin=true/false "pin=true" will run Pin mode "pin=false" will run Scale mode In Pin mode, "pair=n" is now obsolete. Use denstiy_range[] instead Scale mode The default values are: node_range=[1,1], density_range=1,1], step_size=add1 and colcate=false
Integrated "serviceip" into scale framework. Was not working until this commit. Cosmetic cleanup i.e debug TASK tags, and doublle "when" guards.
a. Keep 'pair' for pin mode b. Remove custom image URLs 2. Rebase to upstream
…several cosmetics
mukrishn
force-pushed
the
net_policy
branch
2 times, most recently
from
2be8702 to
1932f6f
Compare
HughNhan
reviewed
Mar 19, 2021
build/Dockerfile
Outdated
| FROM quay.io/operator-framework/ansible-operator:v0.17.1 | ||
| USER root | ||
|
|
||
| RUN pip3 install netaddr |
Author
There was a problem hiding this comment.
Hugh, still we are not finalized with the policy so this PR is just a draft until we test & freeze it.
For now this package is not required.
HughNhan
force-pushed
the
hnhan-uperf-scale
branch
from
28a7350 to
741f3d7
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Need extra arguments,
will add below policy
... ... ingress: - from: - podSelector: matchLabels: type: uperf-benchmark-bench-client-3090cbf0 - namespaceSelector: matchLabels: project: my-ripsaw - ipBlock: cidr: 10.128.0.0/14 except: - 10.129.2.0/23 - 10.128.2.0/23 ...