Skip to content

Conversation

@austinc3030
Copy link

Summary

This PR adds a default value for the BUILDPLATFORM build argument in the Dockerfile to prevent build failures when the platform is not explicitly specified.

Changes

  • Added ARG BUILDPLATFORM=linux/amd64 before the FROM instruction to provide a sensible default

Why This Fix is Needed

The Dockerfile uses --platform=${BUILDPLATFORM} in the FROM instruction for cross-compilation, but the BUILDPLATFORM argument was not defined with a default value. This causes build failures in environments where the build platform is not automatically set.

By providing a default value of linux/amd64, the Dockerfile will build successfully even when BUILDPLATFORM is not explicitly passed as a build argument, while still allowing it to be overridden when needed.

Testing

  • Verified the Dockerfile builds successfully without specifying --build-arg BUILDPLATFORM
  • Confirmed the existing build behavior is preserved when BUILDPLATFORM is explicitly provided

Signed-off-by: Austin Tyler Conn <[email protected]>
@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.99%. Comparing base (61655d4) to head (18d7589).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1070   +/-   ##
=======================================
  Coverage   94.99%   94.99%           
=======================================
  Files          61       61           
  Lines        3397     3397           
=======================================
  Hits         3227     3227           
  Misses        158      158           
  Partials       12       12           
Flag Coverage Δ
unittests 94.99% <ø> (ø)

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant