Closed dkirov-dd closed 2 months ago
Use this command from test-infra-definitions to manually test this PR changes on a VM:
inv create-vm --pipeline-id=38227376 --os-family=ubuntu
Note: This applies to commit 0a2b3f45
Run ID: a8d90105-39ae-49b3-83ce-8521f70c5692 Metrics dashboard Target profiles
Baseline: d41ff6461c019fedd11af6b3b68cf699d8b45f85 Comparison: 0a2b3f4560575900cf4c1a5088f2bd152bad8cdf
Performance changes are noted in the perf column of each table:
Confidence level: 90.00% Effect size tolerance: |Δ mean %| ≥ 5.00%
There were no significant changes in experiment optimization goals at this confidence level and effect size tolerance.
/merge
:steam_locomotive: MergeQueue: waiting for PR to be ready
This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove
command.
Use /merge -c
to cancel this operation!
:steam_locomotive: MergeQueue: pull request added to the queue
The median merge time in main
is 25m.
Use /merge -c
to cancel this operation!
@dkirov-dd Was this self-QA'd? If not, do you have a set of QA instructions, perhaps with a specific integration to test?
What does this PR do?
Motivation
This is a follow-up to this PR, making the same addition for community python checks.
Additional Notes
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes
OriginSubProduct
andOriginProductDetail
can be viewed using scott's proxy-dumper. Run the proxy-dumper and change dd_url to http://localhost:8081/. Run the agent. See that the correctOriginSubProduct
andOriginProductDetail
values being populated for the checks