[k8sattributes] Add extended load test#47825
Open
ChrsMark wants to merge 1 commit intoopen-telemetry:mainfrom
Open
[k8sattributes] Add extended load test#47825ChrsMark wants to merge 1 commit intoopen-telemetry:mainfrom
ChrsMark wants to merge 1 commit intoopen-telemetry:mainfrom
Conversation
Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
22f45ff to
f1e0402
Compare
dmitryax
approved these changes
Apr 21, 2026
Contributor
MovieStoreGuy
left a comment
There was a problem hiding this comment.
There is a few lint issues to address, but otherwise his looks reasonable?
Comment on lines
-28
to
-35
| func skipIfKwokUnavailable(t *testing.T) { | ||
| if os.Getenv("SKIP_KWOK_TESTS") == "1" { | ||
| t.Skip("Skipping KWOK test: SKIP_KWOK_TESTS=1") | ||
| } | ||
| if _, err := exec.LookPath("kwokctl"); err != nil { | ||
| t.Skipf("Skipping KWOK test: kwokctl not found in PATH (install from https://kwok.sigs.k8s.io/)") | ||
| } | ||
| } |
Contributor
There was a problem hiding this comment.
Is this expected be removed from the changes?
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
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
As described at #47389 (comment), we need an extended benchmark for the
k8sattributesprocessor which will stress test the component not just against the default configuration but against an extended configuration which enables all the available workload type attributes (daemonset, statefulset etc).This PR adds this extended test using the following configuration:
and for the following Workload types:
The scenarios test against 110, 1K and 5K distinct workloads. This means the 1K scenario tests a kwok cluster with 1K Deployment, 1K Daemonsets, 1K Cronjobs and 1K StatefulSets so as to simulate a full scale K8s cluster with various types of workloads. This will allow us understand what is the impact in the memory footprint for the additional workload types that we add. Below the scenarios details:
Link to tracking issue
Related to #47389
Testing
Running the benchmarks locally I get the following results:
Results align with the current behaviour: https://open-telemetry.github.io/opentelemetry-collector-contrib/benchmarks/loadtests/#metrick8sattributesprocessor-5k-workload-cluster-ram-mib