Closed davidor closed 5 days ago
/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 26m.
Use /merge -c
to cancel this operation!
Run ID: 88faa51c-0fda-4357-9eb4-ae9b1712dd42 Metrics dashboard Target profiles
Baseline: 1987bff89f063bf8bde58002196f5cd2f58245df Comparison: ea8e2c750b8c7b27d294eaa87d88ade81e965e3c
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.
What does this PR do?
Fixes a bug in the workloadmeta store. It was not working correctly for filters that match entities, like
workloadmeta.IsNodeMetadata
used by the metadata controller.I've set the no-changelog label because this doesn't affect any released versions.
Describe how to test/QA your changes
Skip. It's a bit difficult to test this case manually, but I added a unit test that should cover it.