wu-sheng opened a new pull request, #13812: URL: https://github.com/apache/skywalking/pull/13812
### Fix duplicate expected entry incompatible with updated infra-e2e - [x] Explain briefly why the bug exists and how to fix it. **Changes:** 1. Bump `skywalking-infra-e2e` from `e26033e` to `b2a6639` (apache/skywalking-infra-e2e#143), which enforces distinct matching in `contains` — each actual item can only satisfy one expected entry. 2. Remove duplicate `ratings.default` entry in `test/e2e-v2/cases/rover/process/istio/expected/service.yml` — a copy-paste bug where the same service appeared twice. With the new distinct matching, this would require two actual `ratings.default` services (only one exists), causing the test to fail. **Verified locally** with the locally-built infra-e2e binary: - All other `contains` usage across ~491 expected files is unaffected - Alarm files with intentional duplicate entries (e.g., `silence-after-webhook.yml`) still pass because actual data has enough distinct messages with different `startTime` values - [ ] Update the [`CHANGES` log](https://github.com/apache/skywalking/blob/master/docs/en/changes/changes.md). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
