-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[CSPM] Compliance module: improve err report and fix tests #21150
Conversation
Bloop Bleep... Dogbot HereRegression Detector ResultsRun ID: 2a06656f-6e23-4e4b-a4c8-8c9f109ed301 Performance changes are noted in the perf column of each table:
|
perf | experiment | goal | Δ mean % | Δ mean % CI | confidence |
---|---|---|---|---|---|
❌ | trace_agent_msgpack | ingress throughput | +0.00 | [-0.00, +0.00] | 0.00% |
❌ | dogstatsd_string_interner_8MiB_100k | ingress throughput | -0.00 | [-0.00, +0.00] | 6.56% |
❌ | file_to_blackhole | egress throughput | -0.13 | [-6.29, +6.03] | 2.70% |
209f382
to
d97a9c4
Compare
d97a9c4
to
43bb944
Compare
43bb944
to
6c4afea
Compare
27fa1ac
to
41804d5
Compare
What does this PR do?
Also fixes and improve the tests:
Motivation
Additional Notes
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes
Reviewer's Checklist
Triage
milestone is set.major_change
label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.changelog/no-changelog
label has been applied.qa/skip-qa
label is not applied.team/..
label has been applied, indicating the team(s) that should QA this change.need-change/operator
andneed-change/helm
labels have been applied.k8s/<min-version>
label, indicating the lowest Kubernetes version compatible with this feature.