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

do not set empty mirror architectures for flat mirrors #1388

Merged
merged 9 commits into from
Nov 16, 2024

Conversation

neolynx
Copy link
Member

@neolynx neolynx commented Nov 8, 2024

Fixes #1385

Description of the Change

when creating a mirror of a flat repository, aptly sets the architectures to nil. This prevents filtering with dependencies, because this is done by looping through the architectures.

This change fixes this by forcing all mirrors to have an architecture (and those which don't require the architectures specified when creating the mirror). If available, the architectures will be set from the Release file.

Checklist

  • functional test added/updated (if change is functional)

@neolynx neolynx self-assigned this Nov 8, 2024
Copy link

codecov bot commented Nov 8, 2024

Codecov Report

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

Project coverage is 74.82%. Comparing base (2f06b06) to head (c86b888).
Report is 13 commits behind head on master.

Files with missing lines Patch % Lines
deb/list.go 0.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1388      +/-   ##
==========================================
- Coverage   74.83%   74.82%   -0.01%     
==========================================
  Files         157      157              
  Lines       18094    18100       +6     
==========================================
+ Hits        13540    13543       +3     
- Misses       3456     3458       +2     
- Partials     1098     1099       +1     

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

@neolynx neolynx force-pushed the fix/flat-mirror-filtering branch 4 times, most recently from f696399 to 23a4d6e Compare November 8, 2024 14:56
@neolynx neolynx added the increase coverage The PR lacks test coverage label Nov 8, 2024
note: 'Architecture' is not official, but used by nvidia mirrors for no debian arch 'x86_64'. shold this be supported ?
flat repos may have architecture which is needed for filtering dependencies
@neolynx neolynx force-pushed the fix/flat-mirror-filtering branch 4 times, most recently from fc7e083 to aad12f8 Compare November 8, 2024 17:19
@neolynx neolynx requested a review from a team November 8, 2024 17:22
@neolynx neolynx added needs review Ready for review & merge and removed increase coverage The PR lacks test coverage labels Nov 8, 2024
@neolynx neolynx force-pushed the fix/flat-mirror-filtering branch from aad12f8 to 8fb7f21 Compare November 8, 2024 17:51
@neolynx neolynx force-pushed the fix/flat-mirror-filtering branch from 8fb7f21 to c86b888 Compare November 8, 2024 18:00
Copy link
Contributor

@iofq iofq left a comment

Choose a reason for hiding this comment

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

LGTM!

@neolynx neolynx merged commit 31ccce1 into master Nov 16, 2024
45 of 46 checks passed
@neolynx neolynx deleted the fix/flat-mirror-filtering branch November 16, 2024 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs approval needs review Ready for review & merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

-filter-with-deps flag isn't including dependencies
2 participants