Skip to content

Conversation

@youssef-backport-bot
Copy link
Contributor

Backport of #5054 to rel/3.8

/cc @Evangelink @nohwnd

Evangelink
Evangelink previously approved these changes Feb 18, 2025
@Evangelink Evangelink enabled auto-merge (squash) February 18, 2025 09:26
@nohwnd
Copy link
Member

nohwnd commented Feb 18, 2025

Failed on output not mixed tests, looks like flakiness.

@Youssef1313
Copy link
Member

test\UnitTests\TestFramework.UnitTests\Exceptions\BinaryFormatterExceptionSerializationTests.cs(22,48): error CS0618: (NETCORE_ENGINEERING_TELEMETRY=Build) 'InternalTestFailureException' is obsolete: 'We will remove or hide this type starting with v4. If you are using this type, reach out to our team on ***.'

@nohwnd
Copy link
Member

nohwnd commented Feb 18, 2025

I was looking specifically for that in local run of unit tests and saw just the integration tests fail. Same as here in the failed tests:
image

dammit such complicated reporting (not your fault of course).

@nohwnd
Copy link
Member

nohwnd commented Feb 18, 2025

fixing on the original, where we are also discussing the approach.

auto-merge was automatically disabled February 18, 2025 15:02

Head branch was pushed to by a user without write access

@Evangelink Evangelink enabled auto-merge (squash) February 18, 2025 15:44
@Evangelink Evangelink merged commit ac22a6d into microsoft:rel/3.8 Feb 18, 2025
8 checks passed
@Youssef1313 Youssef1313 deleted the backport/5054/to/rel/3.8 branch February 18, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants