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.

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

Publish size regression on Linux #81374

Closed
sebastienros opened this issue Jan 30, 2023 · 4 comments · Fixed by #81379
Closed

Publish size regression on Linux #81374

sebastienros opened this issue Jan 30, 2023 · 4 comments · Fixed by #81379
Assignees
@kunalspathak
Labels
area-Infrastructure-coreclr blocking-official-build

Comments

@sebastienros
Copy link
Member

From 111MB to 284MB for a self-contained app. Only on Linux.

These files are the reason

image

Can be reproduced between versions 8.0.0-alpha.1.23076.9+007df05 and 8.0.0-alpha.1.23077.2+741fa51 which corresponds to this changeset 007df05...741fa51
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

@ghost ghost added the untriaged New issue has not been triaged by the area owner label Jan 30, 2023
@eerhardt
Copy link
Member

cc @kunalspathak - #77578 looks suspect.

@agocke agocke added area-Infrastructure-coreclr and removed untriaged New issue has not been triaged by the area owner labels Jan 30, 2023
@ghost
Copy link

ghost commented Jan 30, 2023

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

Issue Details

From 111MB to 284MB for a self-contained app. Only on Linux.

These files are the reason

image

Can be reproduced between versions 8.0.0-alpha.1.23076.9+007df05 and 8.0.0-alpha.1.23077.2+741fa51 which corresponds to this changeset 007df05...741fa51

Author: sebastienros
Assignees: -
Labels:

area-Infrastructure-coreclr, untriaged
Milestone: -

@agocke
Copy link
Member

agocke commented Jan 30, 2023

Yup, that change almost certainly also affects the official build -- @kunalspathak can you revert your change? We can bring it back when we come up with a way to only include symbols in PR, or add a step to strip them before publishing.

@kunalspathak kunalspathak mentioned this issue Jan 30, 2023
Merged
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jan 30, 2023
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Jan 30, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Mar 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kunalspathak kunalspathak

Labels
area-Infrastructure-coreclr blocking-official-build
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Revert "Produce crashreport.json and use llvm-symbolizer to create st… kunalspathak/runtime
4 participants
@agocke @sebastienros @eerhardt @kunalspathak