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

CecilPackageVersionMatchesAssemblyVersion failing #103563

Closed
MichalStrehovsky opened this issue Jun 17, 2024 · 5 comments · Fixed by #103665
Closed

CecilPackageVersionMatchesAssemblyVersion failing #103563

MichalStrehovsky opened this issue Jun 17, 2024 · 5 comments · Fixed by #103665
Labels
area-Tools-ILLink .NET linker development as well as trimming analyzers blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms'

Comments

@MichalStrehovsky
Copy link
Member

MichalStrehovsky commented Jun 17, 2024

This seems to be randomly (?) failing in the dotnet-linker-tests pipeline: https://dev.azure.com/dnceng-public/public/_build?definitionId=139&_a=summary.

String lengths are both 6. Strings differ at index 5.
Expected: "0.11.4"
But was:  "0.11.5"
----------------^

https://dev.azure.com/dnceng-public/public/_build/results?buildId=709558&view=ms.vss-test-web.build-test-results-tab
https://dev.azure.com/dnceng-public/public/_build/results?buildId=709820&view=ms.vss-test-web.build-test-results-tab

Known Issue Error Message

Fill the error message using step by step known issues guidance.

{
  "ErrorMessage": "String lengths are both 6",
  "BuildRetry": false,
  "ExcludeConsoleLog": false
}

Report

Build Definition Test Pull Request
709558 dotnet/runtime Mono.Linker.Tests.CecilVersionCheck.CecilPackageVersionMatchesAssemblyVersion()

Summary

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

Known issue validation

Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=709558
Error message validated: [String lengths are both 6]
Result validation: ✅ Known issue matched with the provided build.
Validation performed at: 6/17/2024 1:28:58 PM UTC

@MichalStrehovsky MichalStrehovsky added blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' area-Tools-ILLink .NET linker development as well as trimming analyzers labels Jun 17, 2024
@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Jun 17, 2024
Copy link
Contributor

Tagging subscribers to this area: @agocke, @sbomer, @vitek-karas
See info in area-owners.md if you want to be subscribed.

@am11
Copy link
Member

am11 commented Jun 17, 2024

nit: to make stats working: "ErrorMessage": "String lengths are both 6" and delete "ErrorPattern" line.

@MichalStrehovsky
Copy link
Member Author

nit: to make stats working: "ErrorMessage": "String lengths are both 6" and delete "ErrorPattern" line.

I didn't think it would work since it's not in stdout but let's see.

@am11
Copy link
Member

am11 commented Jun 17, 2024

Counter seems to be working now.

@sbomer
Copy link
Member

sbomer commented Jun 17, 2024

Fixed in dotnet/cecil#192, flowing to runtime in #103574

@sbomer sbomer closed this as completed in 7172a7d Jun 18, 2024
@dotnet-policy-service dotnet-policy-service bot removed the untriaged New issue has not been triaged by the area owner label Jun 18, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jul 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Tools-ILLink .NET linker development as well as trimming analyzers blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms'
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants