-
Notifications
You must be signed in to change notification settings - Fork 359
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
ci: add versions fixture generator for Red Hat #1356
base: main
Are you sure you want to change the base?
Conversation
b6e974a
to
44a250c
Compare
nice, that's a very reasonable "worst case with no cache" time given the size (as a reminder, Debian is also pretty big typically taking 5+ minutes, but with caching it only takes ~20 seconds 😄)
double nice - way to go caching 🥳 |
5d7f6b8
to
add759d
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1356 +/- ##
==========================================
+ Coverage 68.52% 68.74% +0.21%
==========================================
Files 183 184 +1
Lines 17574 17714 +140
==========================================
+ Hits 12043 12177 +134
- Misses 4873 4877 +4
- Partials 658 660 +2 ☔ View full report in Codecov by Sentry. |
4ed1974
to
a08e2b8
Compare
a08e2b8
to
502c632
Compare
Branched off #1355
This adds a
semantic
fixture generator for the Red Hat ecosystem - as this ends up being just under 2mb, I've not committed the file like the others, instead making the test suite only use it if it's actually present.It works by leveraging
rpm
s built-in Lua interpreter, which includes exposing thevercmp
function meaning we can haverpm
tell us what the result of comparing two versions are so long as there's a version of that available.In terms of actually getting a version of
rpm
, that turns out to be a lot harder with Docker I assume just because CentOS & co is a less open-source-y distro vs the others; there are images, but they're all generally very big. In an interesting twist though it turns out Debian actually ships a version ofrpm
for building and the like - it has had it's package management abilities neutered, but that's not something we need so a simpleapt install rpm
gives us the dependency we need.