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

[release/9.0][wasi/wasm] SimpleBuildDoesNotFail compares with the wrong expected value #108938

Open
carlossanlop opened this issue Oct 16, 2024 · 2 comments
Assignees
Labels
arch-wasm WebAssembly architecture area-Infrastructure-mono 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 os-browser Browser variant of arch-wasm os-wasi Related to WASI variant of arch-wasm
Milestone

Comments

@carlossanlop
Copy link
Member

carlossanlop commented Oct 16, 2024

Found this issue in the 9.0 stable branding PR for GA #108899

The test Wasi.Build.Tests.SdkMissingTests.SimpleBuildDoesNotFail is not passing because the expected value is wrong, while the actual value is right.

Build Information

Build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=844459
Build error leg or test failing: various wasi and wasm

Error Message

{
  "ErrorMessage" : [ "SimpleBuildDoesNotFail", "Runtime pack path doesn't match." ],
  "BuildRetry" : false,
  "ExcludeConsoleLog" : false
}
    Wasi.Build.Tests.SdkMissingTests.SimpleBuildDoesNotFail(config: "Release") [FAIL]
      Runtime pack path doesn't match.
      Expected: '/root/helix/work/workitem/e/dotnet-latest/packs/Microsoft.NETCore.App.Runtime.Mono.wasi-wasm/9.0.0-ci'
      Actual:   '/root/helix/work/workitem/e/dotnet-latest/packs/Microsoft.NETCore.App.Runtime.Mono.wasi-wasm/9.0.0'
      Stack Trace:
        /_/src/mono/wasi/Wasi.Build.Tests/BuildTestBase.cs(354,0): at Wasm.Build.Tests.BuildTestBase.AssertRuntimePackPath(String buildOutput, String targetFramework)
        /_/src/mono/wasi/Wasi.Build.Tests/BuildTestBase.cs(259,0): at Wasm.Build.Tests.BuildTestBase.BuildProject(BuildArgs buildArgs, String id, BuildProjectOptions options)
        /_/src/mono/wasi/Wasi.Build.Tests/SdkMissingTests.cs(70,0): at Wasi.Build.Tests.SdkMissingTests.BuildWithInvalidSdkPath(String config, String extraProperties, Boolean publish, Boolean expectSuccess)
        /_/src/mono/wasi/Wasi.Build.Tests/SdkMissingTests.cs(58,0): at Wasi.Build.Tests.SdkMissingTests.SimpleBuildDoesNotFail(String config)
           at InvokeStub_SdkMissingTests.SimpleBuildDoesNotFail(Object, Span`1)
           at System.Reflection.MethodBaseInvoker.InvokeWithOneArg(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)

Known issue validation

Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=844459
Error message validated: [SimpleBuildDoesNotFail Runtime pack path doesn't match.]
Result validation: ✅ Known issue matched with the provided build.
Validation performed at: 10/16/2024 6:44:01 PM UTC

Report

Build Definition Test Pull Request
848154 dotnet/runtime Wasi.Build.Tests.WasiTemplateTests.ConsoleBuildThenRunThenPublish #109048
848159 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish #109049
847977 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish
847880 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish
847627 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish
847485 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish
847042 dotnet/runtime Wasi.Build.Tests.WasiTemplateTests.ConsoleBuildThenRunThenPublish
846981 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish
846875 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish
846663 dotnet/runtime Workloads-ST-Wasm.Build.Tests.WorkItemExecution #108843
846660 dotnet/runtime Workloads-ST-Wasm.Build.Tests.WorkItemExecution #108751
846630 dotnet/runtime Wasi.Build.Tests.WasiTemplateTests.ConsoleBuildThenRunThenPublish
846615 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish #108516
846401 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish #108482
846304 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish #108910
846263 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish #108975
845438 dotnet/runtime Wasi.Build.Tests.WasiTemplateTests.ConsoleBuildThenRunThenPublish #108482
845291 dotnet/runtime Wasi.Build.Tests.HttpTests.HttpBuildThenRunThenPublish
844459 dotnet/runtime Wasi.Build.Tests.WasiTemplateTests.ConsoleBuildThenRunThenPublish #108899

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
6 19 19
@carlossanlop carlossanlop added arch-wasm WebAssembly architecture 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 os-browser Browser variant of arch-wasm os-wasi Related to WASI variant of arch-wasm labels Oct 16, 2024
@carlossanlop carlossanlop added this to the 9.0.0 milestone Oct 16, 2024
Copy link
Contributor

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Copy link
Contributor

Tagging subscribers to this area: @directhex, @matouskozak
See info in area-owners.md if you want to be subscribed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arch-wasm WebAssembly architecture area-Infrastructure-mono 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 os-browser Browser variant of arch-wasm os-wasi Related to WASI variant of arch-wasm
Projects
None yet
Development

No branches or pull requests

3 participants