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

build: use new imagebuilder setup.sh for appropriate versions #1104

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

efahl
Copy link
Contributor

@efahl efahl commented Dec 2, 2024

Properly detect versions for which the slim imagebuilder containers using setup.sh is being used. Set up the container accordingly.

Allows building the new 24.10 versions, including the RCs.

@efahl efahl force-pushed the prep-24.10 branch 2 times, most recently from 6c15ad8 to abbf8f0 Compare December 2, 2024 19:50
Copy link

codecov bot commented Dec 2, 2024

Codecov Report

Attention: Patch coverage is 76.92308% with 3 lines in your changes missing coverage. Please review.

Project coverage is 90.15%. Comparing base (5e65dec) to head (ae063cf).
Report is 167 commits behind head on main.

Files with missing lines Patch % Lines
asu/build.py 25.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1104      +/-   ##
==========================================
+ Coverage   80.75%   90.15%   +9.40%     
==========================================
  Files          15       14       -1     
  Lines         977     1189     +212     
==========================================
+ Hits          789     1072     +283     
+ Misses        188      117      -71     

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

@efahl
Copy link
Contributor Author

efahl commented Dec 3, 2024

@aparcar you might want to get this one in, as it allows the ASU server to build the 24.10 RCs, which are broken without it...

asu/util.py Outdated Show resolved Hide resolved
asu/util.py Outdated Show resolved Hide resolved
@efahl efahl force-pushed the prep-24.10 branch 2 times, most recently from 029edb9 to 62ffde7 Compare December 5, 2024 13:48
Properly detect versions for which the slim imagebuilder containers using
setup.sh is being used.  Set up the container accordingly.

Signed-off-by: Eric Fahlgren <[email protected]>
@efahl
Copy link
Contributor Author

efahl commented Dec 5, 2024

Rebased. Ends up not fixing anything, but the general cleanup and added tests are probably worth keeping.

@aparcar aparcar merged commit 28a1629 into openwrt:main Dec 5, 2024
3 of 4 checks passed
@efahl efahl deleted the prep-24.10 branch December 5, 2024 23:11
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.

2 participants