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

Fix some Health checks on NuGet packages produced. #4406

Closed
joperezr opened this issue Sep 13, 2023 · 1 comment · Fixed by #4417
Closed

Fix some Health checks on NuGet packages produced. #4406

joperezr opened this issue Sep 13, 2023 · 1 comment · Fixed by #4417
Assignees

Comments

@joperezr
Copy link
Member

If I look at one of the RC1 packages from the dotnet/runtime repo that were shipped yesterday to NuGet, I see the following when opening it with the NuGet Package Explorer:

image

If you contrast that to one of the RC1 packages produced by dotnet/extensions, you'll see some differences:

image

I believe we are pushing symbols to Msft symbol servers as well as part of our build, so I'm not sure why this is what is stated here but we should figure out what the problem is and fix it. @RussKie can you please take a look so we make sure we correct this before RC2?

@RussKie
Copy link
Member

RussKie commented Sep 15, 2023

@mmitche was kind enough to help me root causing it, and we figured it boiled down to this line:

<SymbolPackageFormat>snupkg</SymbolPackageFormat>

The Arcade SDK doesn't support snupkg format, and the debug symbols must be in a form of *.symbols.nupkg.

RussKie added a commit to dotnet/arcade that referenced this issue Sep 15, 2023
To avoid the repeat of dotnet/extensions#4406,
warn if SymbolPackageFormat property is set to 'snupkg'.

If necessary, the warning can be suppressed with SupressSymbolPackageFormatValidation=true.
RussKie added a commit to RussKie/extensions that referenced this issue Sep 15, 2023
The Arcade SDK doesn't support "snupkg" format, which means the debug
symbols aren't getting published.

Fixes dotnet#4406
RussKie added a commit to dotnet/arcade that referenced this issue Sep 18, 2023
To avoid the repeat of dotnet/extensions#4406,
warn if SymbolPackageFormat property is set to 'snupkg'.

If necessary, the warning can be suppressed with SupressSymbolPackageFormatValidation=true.
RussKie added a commit to dotnet/arcade that referenced this issue Sep 18, 2023
To avoid the repeat of dotnet/extensions#4406,
warn if SymbolPackageFormat property is set to 'snupkg'.

If necessary, the warning can be suppressed with SuppressSymbolPackageFormatValidation=true.
RussKie added a commit that referenced this issue Sep 18, 2023
The Arcade SDK doesn't support "snupkg" format, which means the debug
symbols aren't getting published.

Fixes #4406
@ghost ghost removed the work in progress 🚧 label Sep 18, 2023
@RussKie RussKie closed this as completed Sep 18, 2023
RussKie added a commit to dotnet/arcade that referenced this issue Sep 19, 2023
To avoid the repeat of dotnet/extensions#4406,
warn if SymbolPackageFormat property is set to 'snupkg'.

If necessary, the warning can be suppressed with SuppressSymbolPackageFormatValidation=true.
RussKie added a commit to dotnet/arcade that referenced this issue Sep 20, 2023
To avoid the repeat of dotnet/extensions#4406,
warn if SymbolPackageFormat property is set to 'snupkg'.

If necessary, the warning can be suppressed with SuppressSymbolPackageFormatValidation=true.
@RussKie RussKie linked a pull request Oct 10, 2023 that will close this issue
@ghost ghost locked as resolved and limited conversation to collaborators Oct 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants