Skip to content

[Failing test]: ParameterEndToEnd tests intermittently crash in the outerloop #9193

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.

Sign up for GitHub

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

Jump to bottom
Open
1 task done
RussKie opened this issue May 9, 2025 · 4 comments · May be fixed by #9269
Open
1 task done

[Failing test]: ParameterEndToEnd tests intermittently crash in the outerloop #9193

RussKie opened this issue May 9, 2025 · 4 comments · May be fixed by #9269
Assignees
@RussKie
Labels
area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication quarantined-test Quarantined tests that run only in the Outerloop Tests workflow testing ☑️

Comments

@RussKie
Copy link
Member

RussKie commented May 9, 2025 •
edited
Loading

Is there an existing issue for this?

  • I have searched the existing issues

Build information

Builds:

Fill in the error message template

I pulled a crashdump from my fork, and here's the problem:

System.InvalidOperationException
  HResult=0x80131509
  Message=Failed to convert configuration value at 'InsertionRows' to type 'System.Int32'.
  Source=<Cannot evaluate the exception source>
  StackTrace:
<Cannot evaluate the exception stack trace>

  This exception was originally thrown at this call stack:
    int.Parse(string, System.Globalization.NumberStyles, System.IFormatProvider)
    System.ComponentModel.Int32Converter.FromString(string, System.Globalization.NumberFormatInfo)
    System.ComponentModel.BaseNumberConverter.ConvertFrom(System.ComponentModel.ITypeDescriptorContext, System.Globalization.CultureInfo, object)

Inner Exception 1:
ArgumentException: 8C10549B is not a valid value for Int32.

Inner Exception 2:
FormatException: The input string '8C10549B' was not in a correct format.

Image

Image

Other info

The failure appears to be intermittent, e.g., one of the latest outerloop runs didn't crash.
@RussKie RussKie added the blocking-clean-ci Blocking a green CI label May 9, 2025
@github-actions github-actions bot added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label May 9, 2025
@RussKie RussKie changed the title [Failing test]: Playground tests intermittently crash in the outerloop [Failing test]: ParameterEndToEnd tests intermittently crash in the outerloop May 9, 2025
@RussKie
Copy link
Member Author

RussKie commented May 9, 2025

@mitchdenny looks like you added the test, do you think you could help?

@RussKie RussKie added testing ☑️ quarantined-test Quarantined tests that run only in the Outerloop Tests workflow and removed blocking-clean-ci Blocking a green CI labels May 9, 2025
@mitchdenny
Copy link
Member

That is very strange.

@davidfowl
Copy link
Member

I know what it is, so does @DamianEdwards. It’s our parameter randomization code in the test host.

I fixed it for one of our samples, but makes this one needs to be fixed too.

@RussKie
Copy link
Member Author

RussKie commented May 12, 2025 •
edited
Loading

It’s our parameter randomization code in the test host.

Could you please share more?

I fixed it for one of our samples

Could you please link to the change?

@RussKie RussKie self-assigned this May 12, 2025
RussKie added a commit to RussKie/aspire that referenced this issue May 12, 2025
@RussKie
Explicitly opt-in to parameter randomisation
3ac7e0a 
Resolves dotnet#9193

Also update outerloop definitions to match the innerloop
RussKie added a commit to RussKie/aspire that referenced this issue May 12, 2025
@RussKie
Explicitly opt-in to parameter randomisation
049c256 
Resolves dotnet#9193

Also update outerloop definitions to match the innerloop
RussKie added a commit to RussKie/aspire that referenced this issue May 12, 2025
@RussKie
Explicitly opt-in to parameter randomisation
3281e71 
Resolves dotnet#9193

Also update outerloop definitions to match the innerloop
RussKie added a commit to RussKie/aspire that referenced this issue May 12, 2025
@RussKie
Explicitly opt-in to parameter randomisation
faf0e82 
Resolves dotnet#9193
RussKie added a commit to RussKie/aspire that referenced this issue May 12, 2025
@RussKie
Explicitly opt-in to parameter randomisation
b03ac77 
Resolves dotnet#9193
@RussKie RussKie linked a pull request May 12, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RussKie RussKie

Labels
area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication quarantined-test Quarantined tests that run only in the Outerloop Tests workflow testing ☑️
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update outerloop workflow dotnet/aspire
3 participants
@davidfowl @mitchdenny @RussKie