Baremetal #225
Closed
Baremetal #225
Conversation
This file contains 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
* Updated workloads to run on baremetal * first commit: skipping cleanup on baremetal * first commit: skipping cleanup on baremetal * fixed spacing * Checking if cloud is on bareMetal. If it is, we're no longer deleting the benchmark-operator namespace in between runs Co-authored-by: jdowni000 <[email protected]>
* first commit: adding calculations for allocatable CPU per MCP * Changing approach to creating new n number mcps depending on node count * removing old machineConfig_pool func * removing bash script testing code left in common.sh * fixed a couple naming convention issues * small fixes from testing in actual cluster * fixed applying mcp.yaml to use envsubst * more issues resolved from testing * more fixes in while loops while testing * completed deployment fix * changing function name, adding check for mcps and ns if exists * adding json creation and use of mb * adding logic to allow MCP_SIZE and MCP_NODE_COUNT to be set as a variable * changing env var MCP_SIZE to TOTAL_MCPS * fixed spacing in all of new function * adding mb_pod.yml and sending mb operations to pod instead of cli * removing response.csv file as it may be too large depending on how many mcps are genereated * added var to set to choose whether MCPs are created or not * just fix...its Friday...its fixed * fixed if statemnet with elif * creating logic to check env var inputs * adding if condition for sleep depending on size of node_count * adding resources to mb-pod * adding logic to check for sample app pods to be ready before mb-pod deployment * Update common.sh mb-pod and sample app
|
@mohit-sheth, Justin is on PTO till Oct, so do you mind closing this out so I will open a new PR without conflicts and few more changes to it. |
Merged
|
Closing in favor of #248 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Baremetal workload. PR for #208 . Has the ability to use new MCP's or not using supplied values or defaults. Calculates allocatable CPU per MCP and uses that number to deploy n number of sample-apps into each mcp. Than a mb-pod is created to hit all the replicas with http traffic for 1 days times (configurable). This will simulate http traffic and load in the cluster during an upgrade for testing.
Fixes