Skip to content

Add option to skip deleting latency pods or saturationn pods#1159

Merged
centaurus-cloud-bot merged 2 commits intoCentaurusInfra:masterfrom
sonyafenge:perftest-deletingpods
Aug 23, 2021
Merged

Add option to skip deleting latency pods or saturationn pods#1159
centaurus-cloud-bot merged 2 commits intoCentaurusInfra:masterfrom
sonyafenge:perftest-deletingpods

Conversation

@sonyafenge
Copy link
Collaborator

What type of PR is this?

/kind feature

What this PR does / why we need it:
Add constants to have an option skip deleting latency pods or saturationn pods to reduce density testing time for large scale testing when debugging purpose
Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:
Add this to perf test run command to skip deleting latency pods and saturation latency pods

--testoverrides=./testing/experiments/deleting_saturation_pods.yaml 
--testoverrides=./testing/experiments/deleting_latency_pods.yaml

using 100 node scale-out run verified

$ export KUBEMARK_NUM_NODES=100 NUM_NODES=2 SCALEOUT_TP_COUNT=1 SCALEOUT_RP_COUNT=1 RUN_PREFIX=deletingpods-081921-1x1x100
$ export MASTER_DISK_SIZE=500GB MASTER_ROOT_DISK_SIZE=500GB KUBE_GCE_ZONE=us-west2-b MASTER_SIZE=n1-highmem-32 NODE_SIZE=n1-highmem-16 NODE_DISK_SIZE=500GB GOPATH=$HOME/go KUBE_GCE_ENABLE_IP_ALIASES=true KUBE_GCE_PRIVATE_CLUSTER=true CREATE_CUSTOM_NETWORK=true KUBE_GCE_INSTANCE_PREFIX=${RUN_PREFIX} KUBE_GCE_NETWORK=${RUN_PREFIX} ENABLE_KCM_LEADER_ELECT=false ENABLE_SCHEDULER_LEADER_ELECT=false ETCD_QUOTA_BACKEND_BYTES=8589934592 SHARE_PARTITIONSERVER=false LOGROTATE_FILES_MAX_COUNT=200 LOGROTATE_MAX_SIZE=200M KUBE_ENABLE_APISERVER_INSECURE_PORT=true KUBE_ENABLE_PROMETHEUS_DEBUG=true KUBE_ENABLE_PPROF_DEBUG=true TEST_CLUSTER_LOG_LEVEL=--v=2 HOLLOW_KUBELET_TEST_LOG_LEVEL=--v=2 SCALEOUT_CLUSTER=true GCE_REGION=us-west2-b
$ ./cluster/kube-up.sh
$ ./test/kubemark/start-kubemark.sh
$ ./_output/dockerized/bin/linux/amd64/kubectl --kubeconfig=./test/kubemark/resources/kubeconfig.kubemark.tp-1 create tenant arktos
$ SCALEOUT_TEST_TENANT=arktos RUN_PREFIX=deletingpods-081921-1x1x100 PERF_LOG_DIR=/home/sonyali/logs/perf-test/gce-100/arktos/${RUN_PREFIX}/${SCALEOUT_TEST_TENANT}
$ SCALEOUT_TEST_TENANT=arktos RUN_PREFIX=deletingpods-081921-1x1x100 PERF_LOG_DIR=/home/sonyali/logs/perf-test/gce-100/arktos/${RUN_PREFIX}/${SCALEOUT_TEST_TENANT} nohup perf-tests/clusterloader2/run-e2e.sh --nodes=100 --provider=kubemark --kubeconfig=/home/sonyali/go/src/k8s.io/arktos/test/kubemark/resources/kubeconfig.kubemark-proxy --report-dir=${PERF_LOG_DIR} --testconfig=testing/density/config.yaml --testoverrides=./testing/experiments/deleting_saturation_pods.yaml --testoverrides=./testing/experiments/deleting_latency_pods.yaml > ${PERF_LOG_DIR}/perf-run.log  2>&1  &

Does this PR introduce a user-facing change?:


Copy link
Collaborator

@Sindica Sindica left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@zmn223
Copy link
Collaborator

zmn223 commented Aug 23, 2021

/approve

@centaurus-cloud-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Sindica, zmn223

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@centaurus-cloud-bot centaurus-cloud-bot merged commit 2e83ef8 into CentaurusInfra:master Aug 23, 2021
@sonyafenge sonyafenge deleted the perftest-deletingpods branch February 4, 2022 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants