Skip to content

[CI] Various sanitization fixes#35237

Merged
xiangyan99 merged 1 commit intoAzure:mainfrom
pvaneck:test-santizer-fixes
Apr 17, 2024
Merged

[CI] Various sanitization fixes#35237
xiangyan99 merged 1 commit intoAzure:mainfrom
pvaneck:test-santizer-fixes

Conversation

@pvaneck
Copy link
Member

@pvaneck pvaneck commented Apr 17, 2024

With the common sanitizers added to test-proxy, several tests among our SDKs are failing. This attempts to address a number of these failures.

  • Some recordings were manually updated with the new sanitized values.
  • Commented out some common sanitization patterns that were causing a large number of failures but aren't sensitive.
  • Marked some tests as live-only. Could potentially be fixed with a re-recording. I will open an issue for these.

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

With the common sanitizers added to test-proxy, several tests among our
SDKs are failing. This attempts to address a number of these failures.

Signed-off-by: Paul Van Eck <[email protected]>
@pvaneck pvaneck force-pushed the test-santizer-fixes branch from 4c63b30 to b4b5fdc Compare April 17, 2024 02:29
@pvaneck
Copy link
Member Author

pvaneck commented Apr 17, 2024

/check-enforcer override

@pvaneck pvaneck requested a review from scottaddie April 17, 2024 03:29
@xiangyan99 xiangyan99 merged commit 4b9f162 into Azure:main Apr 17, 2024
@pvaneck pvaneck deleted the test-santizer-fixes branch April 17, 2024 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants