Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pkg/fscrypt: support builds for other ARCHs #3824

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

rouming
Copy link
Contributor

@rouming rouming commented Mar 19, 2024

Pillar, which is crossbuildable, depends on fscrypt. Without this cludge any attempt to build pillar for any ARCH other than amd64 raises the following absolutely implicit and opague error:

   Error: error building "lfedge/eve-pillar:f6cd4c5f47f4900781840ea5b861ddecca52c35d": error building for arch arm64: failed to solve: no match for platform in manifest: not found

Which can mean anything. By trials and errors build problems with the fscrypt were determined.

cc: @deitch

Pillar, which is crossbuildable, depends on fscrypt. Without this
cludge any attempt to build pillar for any ARCH other than amd64
raises the following absolutely implicit and opague error:

   Error: error building "lfedge/eve-pillar:f6cd4c5f47f4900781840ea5b861ddecca52c35d": error building for arch arm64: failed to solve: no match for platform in manifest: not found

Which can mean anything. By trials and errors build problems with
the fscrypt were determined.

Signed-off-by: Roman Penyaev <r.peniaev@gmail.com>
Copy link

codecov bot commented Mar 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 17.51%. Comparing base (e4f2710) to head (ca6e76f).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3824   +/-   ##
=======================================
  Coverage   17.51%   17.51%           
=======================================
  Files           3        3           
  Lines         805      805           
=======================================
  Hits          141      141           
  Misses        629      629           
  Partials       35       35           

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

Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

Run tests

Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

This seems to be the pattern we use in other Dockerfiles.

@rouming
Copy link
Contributor Author

rouming commented Mar 19, 2024

Unfortunately nothing indicates that pillar build fails because of fscrypt, which misses another arch build. I assume once dnsmasq or gpt-tools updated (new sha generated) - similar failure will happen on attempt to rebuild pillar.

@eriknordmark
Copy link
Contributor

failure

I wonder if we changed to relying cross compile at some point in time @rene @deitch . In any case those haven't changed in probably 4 years.

@eriknordmark eriknordmark merged commit ff52273 into lf-edge:master Mar 20, 2024
38 of 49 checks passed
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.

None yet

2 participants