Skip to content

Add Dockerfile for arm64 compilation - #973

Merged
martincostello merged 2 commits into
mainfrom
arm64
Nov 27, 2023
Merged

Add Dockerfile for arm64 compilation#973
martincostello merged 2 commits into
mainfrom
arm64

Conversation

@martincostello

Copy link
Copy Markdown
Owner

Add a Dockerfile to compile for arm64 AoT on GitHub Actions (until they have a native arm64 runner).

@martincostello martincostello added enhancement github_actions Pull requests that update GitHub Actions code .NET Pull requests that update .net code labels Nov 27, 2023
@codecov

codecov Bot commented Nov 27, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (09568ce) 96.72% compared to head (5d9e836) 96.72%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #973   +/-   ##
=======================================
  Coverage   96.72%   96.72%           
=======================================
  Files          45       45           
  Lines         641      641           
  Branches       71       71           
=======================================
  Hits          620      620           
  Misses          7        7           
  Partials       14       14           
Flag Coverage Δ
linux 95.94% <ø> (ø)
macos 96.72% <ø> (ø)
windows 96.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@martincostello

Copy link
Copy Markdown
Owner Author

/deploy dev

@github-actions

github-actions Bot commented Nov 27, 2023

Copy link
Copy Markdown

🧪 Tests for deployment to dev passed ✅

Add a Dockerfile to compile for arm64 AoT on GitHub Actions (until they have a native arm64 runner).
@martincostello

Copy link
Copy Markdown
Owner Author

/deploy dev

@github-actions

github-actions Bot commented Nov 27, 2023

Copy link
Copy Markdown

🧪 Tests for deployment to dev passed ✅

@martincostello

Copy link
Copy Markdown
Owner Author

AoT on x86_64:

Published size: 39.1MB; 4 files
Executable size: 13.4MB

REPORT
Duration: 219.74 ms
Billed Duration: 350 ms
Memory Size: 192 MB
Max Memory Used: 31 MB
Init Duration: 129.42 ms

AoT on arm64:

Published size: 39.2MB; 4 files
Executable size: 13.9MB

REPORT
Duration: 357.59 ms
Billed Duration: 471 ms
Memory Size: 192 MB
Max Memory Used: 32 MB
Init Duration: 112.44 ms

TL;DR:

Duration: +62%
Billed Duration: +34%
Memory Size: -
Max Memory Used: +3%
Init Duration: -13%

Published size: +4%

@martincostello
martincostello marked this pull request as ready for review November 27, 2023 11:40
Ensure that the .NET SDK in global.json is installed on the Docker image to account for lag between new release and Docker image publishing.
@martincostello
martincostello enabled auto-merge (squash) November 27, 2023 12:47
@martincostello
martincostello merged commit d9bf6a4 into main Nov 27, 2023
@martincostello
martincostello deleted the arm64 branch November 27, 2023 12:48
martincostello added a commit that referenced this pull request Nov 28, 2023
Cross-architecture compile for arm64 on GitHub Actions' ubuntu-latest hosted runner.
See https://learn.microsoft.com/dotnet/core/deploying/native-aot/cross-compile#linux.
Effectively a revert of #973.
martincostello added a commit that referenced this pull request Nov 28, 2023
Cross-architecture compile for arm64 on GitHub Actions' ubuntu-latest hosted runner without using Docker.

See https://learn.microsoft.com/dotnet/core/deploying/native-aot/cross-compile#linux.

Effectively a revert of #973.

Also explicitly specify using bash as the shell where appropriate.
@martincostello martincostello mentioned this pull request Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement github_actions Pull requests that update GitHub Actions code .NET Pull requests that update .net code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant