gleono commented on code in PR #88: URL: https://github.com/apache/openwhisk-runtime-dotnet/pull/88#discussion_r1777835929
########## core/net6.0/Dockerfile: ########## @@ -26,9 +26,9 @@ RUN dotnet restore COPY proxy/Apache.OpenWhisk.Runtime.Common/. ./Apache.OpenWhisk.Runtime.Common/ COPY proxy/Apache.OpenWhisk.Runtime.Dotnet.Minimal/. ./Apache.OpenWhisk.Runtime.Dotnet.Minimal/ WORKDIR /app/Apache.OpenWhisk.Runtime.Dotnet.Minimal -RUN dotnet publish -c Release -r linux-musl-x64 -o out +RUN dotnet publish -c Release -o out -FROM mcr.microsoft.com/dotnet/runtime:6.0-alpine AS runtime +FROM mcr.microsoft.com/dotnet/aspnet:6.0-alpine AS runtime Review Comment: Apologies, I needed to be more clear about the images that I was referring to. I am comparing the current [openwhisk/action-dotnet-v6.0:1.18.0](https://hub.docker.com/layers/openwhisk/action-dotnet-v6.0/1.18.0/images/sha256-7b6275f129d2aeea3856bb7b769ff3e2f24220b23affa52d473b8f8f33967a6b?context=explore) (76.12MB) and the one [from this change](https://hub.docker.com/layers/gleonoliva/action-dotnet-v6.0/slim/images/sha256-9de9c2a172c4bfa0935b993bd997b063105f1d70fdf379d71f79b13104656abc?context=explore) (42.94 MB). I know that these are different architectures but I expect similar results for amd64. This is because the step that copies the output from `dotnet publish -r` adds a bunch of libraries that are already in the `aspnet` image, under the `/usr/share/dotnet/shared/` directory to be specific. Here is a screenshot of the files bundled in the current image (1.18.0) under the `/app` directory.  Once I saw the dlls with "AspNetCore" in the name, I figured that maybe it would be better to switch to aspnet as the runtime image. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
