Skip to content

Comments

Self contained dotnet + delete SDK to reduce slug size#65

Open
dstj wants to merge 2 commits intojincod:masterfrom
dstj:self_contained_dotnet
Open

Self contained dotnet + delete SDK to reduce slug size#65
dstj wants to merge 2 commits intojincod:masterfrom
dstj:self_contained_dotnet

Conversation

@dstj
Copy link
Contributor

@dstj dstj commented Sep 13, 2018

Hi @jincod.

This is essentially a redo of PR #20, but improved a bit and base on the current master branch.

With it, I passed from a 304.9M slug (which causes Warning: Your slug size exceeds our soft limit (304 MB) which may affect boot time.) to a 149M slug size!

@dstj dstj changed the title Self contained dotnet Self contained dotnet + delete SDK to reduce slug size Sep 13, 2018
@ghost
Copy link

ghost commented Oct 9, 2019

This is useful, we have been running locally with this change for over a year to reduce container memory usage. This is inline with .NET Core 3.0 work of optimizing for running in limited memory containers.

@kiran94
Copy link

kiran94 commented Oct 10, 2019

@jincod Any update here? This would also be very beneficial for us!

@jincod
Copy link
Owner

jincod commented Oct 10, 2019

Hi there,

Could you please share more info about why do you need self-contained? Is it about slug size?

Here my current state
image

@kiran94
Copy link

kiran94 commented Oct 17, 2019

@jincod Yes, this is for the slug size

@Zero3
Copy link

Zero3 commented Nov 27, 2020

@jincod do you mind sharing the status on merging this PR? It has been idling for a year now, and I was wondering whether it is going to be merged at all, and whether you already fixed it in another way (as your screenshot indicates?) :)

@dstj @WilliamWhispell @kiran94 do you know anything? Or do you use another publicly available buildpack that has a fix for this already? :)

@dstj
Copy link
Contributor Author

dstj commented Nov 27, 2020

@Zero3 I always use my fork, which I'm rebasing on master every time I need the new .NET Core version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants