Skip to content

cross-compilers: ignore pkg hash inconsistencies#5798

Merged
eriknordmark merged 1 commit into
lf-edge:masterfrom
christoph-zededa:ignore_cross_compiler_alpine
Apr 14, 2026
Merged

cross-compilers: ignore pkg hash inconsistencies#5798
eriknordmark merged 1 commit into
lf-edge:masterfrom
christoph-zededa:ignore_cross_compiler_alpine

Conversation

@christoph-zededa
Copy link
Copy Markdown
Contributor

@christoph-zededa christoph-zededa commented Apr 14, 2026

Description

as no files of this package end up in the final EVE image, it does not matter to have older versions of files (because squashfs can compress same files better)

but this relieves us from bumping cross-compilers everytime we change something in pkg/alpine

How to test and validate this PR

change something in pkg/alpine, run make check-docker-hashes-consistency, fix all other inconsistencies, and ensure that no inconsistency was reported for pkg/cross-compilers.

Changelog notes

Build improvements

PR Backports

For all current LTS branches, please state explicitly if this PR should be
backported or not. This section is used by our scripts to track the backports,
so, please, do not omit it.

Here is the list of current LTS branches (it should be always up to date):

  • 16.0-stable: no
  • 14.5-stable: no
  • 13.4-stable: no

Checklist

  • I've provided a proper description
  • I've added the proper documentation
  • I've tested my PR on amd64 device
  • I've tested my PR on arm64 device
  • I've written the test verification instructions
  • I've set the proper labels to this PR

And the last but not least:

  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

Please, check the boxes above after submitting the PR in interactive mode.

as no files of this package end up in the final EVE image,
it does not matter to have older versions of files
(because squashfs can compress same files better)

but this relieves us from bumping cross-compilers everytime
we change something in pkg/alpine

Signed-off-by: Christoph Ostarek <christoph@zededa.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 28.34%. Comparing base (2281599) to head (e5240ce).
⚠️ Report is 494 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5798      +/-   ##
==========================================
+ Coverage   19.52%   28.34%   +8.81%     
==========================================
  Files          19       18       -1     
  Lines        3021     2417     -604     
==========================================
+ Hits          590      685      +95     
+ Misses       2310     1588     -722     
- Partials      121      144      +23     

☔ 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.

Copy link
Copy Markdown
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.

LGTM

@eriknordmark eriknordmark merged commit e278c6c into lf-edge:master Apr 14, 2026
36 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.

3 participants