Skip to content
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

Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty failing intermittently #86532

Closed
jkoritzinsky opened this issue May 19, 2023 · 4 comments · Fixed by #93711 or #93786
Labels
area-Extensions-Configuration blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab
Milestone

Comments

@jkoritzinsky
Copy link
Member

jkoritzinsky commented May 19, 2023

Build Information

Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=280353
Build error leg or test failing: Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty
Pull request: #86519

Error Message

Fill the error message using known issues guidance.

{
  "ErrorMessage": "Expected: other stuff",
  "BuildRetry": false,
  "ErrorPattern": "",
  "ExcludeConsoleLog": true
}

Report

Build Definition Test Pull Request
436679 dotnet/runtime Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty
431376 dotnet/runtime Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty
430806 dotnet/runtime Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty #93066
429943 dotnet/runtime Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty #92057
425385 dotnet/runtime Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty
418980 dotnet/runtime Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty
417649 dotnet/runtime Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
0 0 7

Known issue validation

Build: 🔎
Result validation: ⚠️ Validation could not be done without an Azure DevOps build URL on the issue. Please add it to the "Build: 🔎" line.
Validation performed at: 6/28/2023 10:04:11 PM UTC

@jkoritzinsky jkoritzinsky added blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' area-Extensions-Configuration Known Build Error Use this to report build issues in the .NET Helix tab labels May 19, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label May 19, 2023
@ghost
Copy link

ghost commented May 19, 2023

Tagging subscribers to this area: @dotnet/area-extensions-configuration
See info in area-owners.md if you want to be subscribed.

Issue Details

Build Information

Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=280353
Build error leg or test failing: Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty
Pull request: #86519

Error Message

Fill the error message using known issues guidance.

{
  "ErrorMessage": "Expected: other stuff",
  "BuildRetry": false,
  "ErrorPattern": "",
  "ExcludeConsoleLog": true
}
Author: jkoritzinsky
Assignees: -
Labels:

blocking-clean-ci, area-Extensions-Configuration, Known Build Error

Milestone: -

@tarekgh
Copy link
Member

tarekgh commented May 19, 2023

The failing test is

The failure is https://dev.azure.com/dnceng-public/public/_build/results?buildId=280353&view=ms.vss-test-web.build-test-results-tab&runId=5559962&resultId=100636&paneView=debug

Assert.Equal() Failure
↓ (pos 0)
Expected: other stuff
Actual:   stuff
↑ (pos 0)


 at Microsoft.Extensions.Configuration.Binder.Tests.ConfigurationBinderTests.BindCanReadStaticProperty() in /_/src/libraries/Microsoft.Extensions.Configuration.Binder/tests/Common/ConfigurationBinderTests.cs:line 705
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr) in /_/src/libraries/System.Private.CoreLib/src/System/Reflection/MethodInvoker.cs:line 59

This is a very weird issue as it looks like the string somehow gets trimmed from the beginning. No idea how this can happen.

We may need to look if this issue can occur again.

@tarekgh tarekgh added this to the Future milestone May 19, 2023
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label May 19, 2023
@steveharter
Copy link
Member

This is likely due to ComplexOptions.StaticProperty being a static property and shared with other tests that could be running at the same time, including another test that sets it to "stuff".

@jkotas
Copy link
Member

jkotas commented Oct 20, 2023

@jkotas jkotas reopened this Oct 20, 2023
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Oct 20, 2023
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Oct 22, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Nov 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Extensions-Configuration blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants