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

test: Add stress tests for OCI and Helm #926

Merged

Conversation

karlkfi
Copy link
Contributor

@karlkfi karlkfi commented Oct 7, 2023

  • Add TestStressManyDeployments to pre-load stress test clusters,
    so that the control plane auto-scales before using it for other
    stress tests.
  • Add TestStressMemoryUsageHelm to ensure the helm-sync container
    works at scale.
  • Add TestStressMemoryUsageOCI to ensure the oci-sync container
    works at scale.

Dependencies:

Related Profiling: go/config-sync-reconciler-profiling-2

@karlkfi karlkfi force-pushed the karl-resource-optimize branch 4 times, most recently from 83c15d3 to f2aed4c Compare October 9, 2023 20:23
@karlkfi karlkfi requested review from nan-yu and removed request for tiffanny29631 October 9, 2023 20:24
@karlkfi
Copy link
Contributor Author

karlkfi commented Oct 9, 2023

/hold

Blocked on kptdev/kpt#4055 and need to push gke-e2e

@karlkfi karlkfi force-pushed the karl-resource-optimize branch 2 times, most recently from d8b26cf to 86dc215 Compare October 13, 2023 20:45
@karlkfi karlkfi force-pushed the karl-resource-optimize branch 8 times, most recently from 4b1ae87 to 9859cf4 Compare October 21, 2023 01:05
@karlkfi
Copy link
Contributor Author

karlkfi commented Oct 21, 2023

/retest

@karlkfi karlkfi force-pushed the karl-resource-optimize branch 3 times, most recently from 16e1aaa to 91fbc97 Compare October 23, 2023 21:02
@karlkfi karlkfi changed the title [WIP] Add stress tests for OCI and Helm test: Add stress tests for OCI and Helm Oct 23, 2023
@karlkfi
Copy link
Contributor Author

karlkfi commented Oct 23, 2023

/unhold

anvil-set.yaml Outdated Show resolved Hide resolved
e2e/nomostest/artifactregistry/crane.go Show resolved Hide resolved
e2e/nomostest/artifactregistry/helm.go Show resolved Hide resolved
e2e/nomostest/iam/account.go Show resolved Hide resolved
e2e/testcases/stress_test.go Show resolved Hide resolved
e2e/nomostest/artifactregistry/crane.go Show resolved Hide resolved
- Add TestStressManyDeployments to pre-load stress test clusters,
  so that the control plane auto-scales before using it for other
  stress tests.
- Add TestStressMemoryUsageHelm to ensure the helm-sync container
  works at scale.
- Add TestStressMemoryUsageOCI to ensure the oci-sync container
  works at scale.
Copy link
Contributor

@nan-yu nan-yu left a comment

Choose a reason for hiding this comment

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

/lgtm

e2e/nomostest/artifactregistry/crane.go Show resolved Hide resolved
e2e/nomostest/iam/account.go Show resolved Hide resolved
@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nan-yu

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

The pull request process is described here

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

@google-oss-prow google-oss-prow bot merged commit 610b3b6 into GoogleContainerTools:main Oct 23, 2023
6 checks passed
@karlkfi karlkfi deleted the karl-resource-optimize branch October 23, 2023 23:24
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.

2 participants