Skip to content

[release/9.0-staging] Disable the UTFStringConversionFailures test on CI runs #116460

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 10, 2025

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 10, 2025

Backport of #114747 to release/9.0-staging

/cc @jkoritzinsky

Customer Impact

  • Customer reported
  • Found internally

[Select one or both of the boxes. Describe how this issue impacts customers, citing the expected and actual behaviors and scope of the issue. If customer-reported, provide the issue number.]

Fixes test failures in the 9.0 branch (fixes #116456)

Regression

  • Yes
  • No

[If yes, specify when the regression was introduced. Provide the PR or commit if known.]

Testing

[How was the fix verified? How was the issue missed previously? What tests were added?]

Test fix

Risk

[High/Medium/Low. Justify the indication by mentioning how risks were measured and addressed.]

Low. Test only fix.

IMPORTANT: If this backport is for a servicing release, please verify that:

  • The PR target branch is release/X.0-staging, not release/X.0.

Package authoring no longer needed in .NET 9

IMPORTANT: Starting with .NET 9, you no longer need to edit a NuGet package's csproj to enable building and bump the version.
Keep in mind that we still need package authoring in .NET 8 and older versions.

jkoritzinsky and others added 2 commits June 10, 2025 04:18
…machines can't handle the load from allocating 2 2GB strings and the OOM killer was killing the process.
@jozkee jozkee added the Servicing-approved Approved for servicing release label Jun 10, 2025
@jozkee
Copy link
Member

jozkee commented Jun 10, 2025

@dotnet/dnceng build analysis contains only known test errors and yet it shows as red.

@jkotas
Copy link
Member

jkotas commented Jun 10, 2025

contains only known test errors and yet it shows as red.

I assume that it shows as red since there way too many known failures and the build analysis gave up. It would be nice if there is a clear message about that.

@jozkee
Copy link
Member

jozkee commented Jun 10, 2025

/ba-g this change is only disabling a noisy test for CI.

@jozkee jozkee merged commit 0ddb6b0 into release/9.0-staging Jun 10, 2025
90 of 94 checks passed
@akoeplinger akoeplinger deleted the backport/pr-114747-to-release/9.0-staging branch June 11, 2025 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants