Skip to content

Conversation

@martincostello
Copy link
Member

@martincostello martincostello commented Jun 10, 2025

Changes

Fix running in the sdk image instead of aspnet.

On my laptop the container size goes down from 1.3GB to ~360MB.

Merge requirement checklist

  • Unit tests added/updated
  • CHANGELOG.md file updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

@martincostello martincostello added the enhancement New feature or request label Jun 10, 2025
@martincostello martincostello changed the title Use chiselled image in example/OATS Use aspnet image in example/OATS Jun 10, 2025
- Fix running in the sdk image instead of aspnet.
- Use chiselled container to reduce size.
SqlClient doesn't work with invariant mode.
Appears to need extra setup for SQL Server to work.
- Update to latest .NET 8 SDK and runtime.
- Pin dotnet/aspnet container by SHA.
Configure dependabot to update the .NET 8 SDK in global.json and in the Dockerfile.
@martincostello martincostello marked this pull request as ready for review June 12, 2025 14:20
@martincostello martincostello requested a review from a team as a code owner June 12, 2025 14:20
@@ -1,5 +1,25 @@
version: 2
updates:
- package-ecosystem: "docker"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to merge this now, then remove dependabot once renovate is setup.

@martincostello martincostello mentioned this pull request Jun 12, 2025
3 tasks
@martincostello martincostello merged commit fc4bf13 into main Jun 12, 2025
13 checks passed
@martincostello martincostello deleted the chiseled-images branch June 12, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants