Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[16.0] spec.sh json output fix for pkg/debug build#5509

Merged
rene merged 1 commit into
lf-edge:16.0from
rene:spec-bp-16.0
Dec 29, 2025
Merged

[16.0] spec.sh json output fix for pkg/debug build#5509
rene merged 1 commit into
lf-edge:16.0from
rene:spec-bp-16.0

Conversation

@rene

@rene rene commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

Description

Backport of #5506 , from @andrewd-zededa

How to test and validate this PR

cd pkg/debug
docker build -f Dockerfile.spec .

Changelog notes

Fix spec.sh script to not list bonding network interfaces

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
  • I've added a reference link to the original PR
  • PR's title follows the template
  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

pkg/debug build blocked by spec.sh json output

- Move COMMA variable out of output
- skip non-link files in /sys/class/net.  This
	may be only an issue when building pkg/debug on
	MacOS aarch64 building for ZARCH=amd64 due to the
	Docker desktop shim linux kernel.
- "docker build -f Dockerfile.spec ." to quickly test spec.sh

Signed-off-by: Andrew Durbin <[email protected]>
(cherry picked from commit 780f0c0)
@rene rene requested a review from rouming as a code owner December 29, 2025 11:08
@codecov

codecov Bot commented Dec 29, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 28.08%. Comparing base (a9c5321) to head (5ea20e8).
⚠️ Report is 14 commits behind head on 16.0.

Additional details and impacted files
@@            Coverage Diff             @@
##             16.0    #5509      +/-   ##
==========================================
+ Coverage   19.52%   28.08%   +8.55%     
==========================================
  Files          19       19              
  Lines        3021     2314     -707     
==========================================
+ Hits          590      650      +60     
+ Misses       2310     1520     -790     
- 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.

@eriknordmark eriknordmark left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@rene rene merged commit 4eadd52 into lf-edge:16.0 Dec 29, 2025
49 of 51 checks passed
@rene rene deleted the spec-bp-16.0 branch March 3, 2026 09:21
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