-
Notifications
You must be signed in to change notification settings - Fork 116
cloudapi,weldr: enable librepo by default (and update tests) #4911
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
base: main
Are you sure you want to change the base?
Conversation
AFAIK, it won't, because CloudAPI is not used on-prem by default. Weldr API needs to use it to help with on-prem issues. |
c625126 to
4b23ad3
Compare
You are right of course, silly me! I added support for librepo into weldr too, I leave it as a single PR but we could split and/or start with weldr only etc. |
4b23ad3 to
87ea3a6
Compare
thozza
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks
87ea3a6 to
bf5b3bd
Compare
|
The tests are currently failing with a variant of so something is not quite right yet :/ |
bf5b3bd to
23228d8
Compare
|
Rebasing on 8f6ecc4. |
Funny. I'm getting the same on #4904. The manifest is generated with image-builder, so it uses librepo, and then the build fails with the same problem. |
|
ddf74f5 to
16f66a3
Compare
16f66a3 to
1055da2
Compare
| "dependencies": { | ||
| "osbuild": { | ||
| "commit": "a52e256c481c00e61f785f3ad68e29e1ac8a09fb" | ||
| "commit": "b1e74a309d5c2c088959c6e0322739fe20969912" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also bump the mimimum osbuild version in the spec file, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, excellent catch! Updated.
This commit enables librepo by default in the cloudapi. We have librepo in osbuild/images/ibcli/bib for a while now, c.f. osbuild/image-builder-cli#51 osbuild/bootc-image-builder#786 (enable) osbuild/bootc-image-builder#840 (make default) It is not configurable right now, I am not sure there is a reason to keep libcurl around for manifest generation as its more fragile but if it is desired I can look into that too. Having it configurable would allow us to revert to the previous org.osbuild.curl in case anything is wrong with the new downloader (and arguably it was only excercised by a relatively small number of people compared to how much the service is building).
This commit enables librepo by default in the weldr api. This should help with on-prem out of sync mirror issues like: osbuild#4427
The BaseDepsolveResult was only returning the packages part of the DepsolveResult. With librepo we need the repos as well so that packages can be linked to repos. This commit returns the full DepsolveResult with the Packages/Repos linked now.
We need v166 of osbuild to ensure we have the "quoting" fix in the librepo source [0]. [0] osbuild/osbuild#2253
1055da2 to
b848719
Compare
We could start by only enabling on weldr, it seems
the service very rarely deals with issue about
out-of-sync mirrors.
mocks: make BaseDepsolveResult() return a DepsolveResult
The BaseDepsolveResult was only returning the packages part of
the DepsolveResult. With librepo we need the repos as well so
that packages can be linked to repos. This commit returns the
full DepsolveResult with the Packages/Repos linked now.
weldr: enable librepo by default (and update tests)
This commit enables librepo by default in the weldr api. This
should help with on-prem out of sync mirror issues like:
#4427
cloudapi: enable librepo by default (and update tests)
This commit enables librepo by default in the cloudapi.
We have librepo in osbuild/images/ibcli/bib for a while now, c.f.
osbuild/image-builder-cli#51
osbuild/bootc-image-builder#786 (enable)
osbuild/bootc-image-builder#840 (make default)
It is not configurable right now, I am not sure there is a reason
to keep libcurl around for manifest generation as its more fragile
but if it is desired I can look into that too. Having it configurable
would allow us to revert to the previous org.osbuild.curl in case
anything is wrong with the new downloader (and arguably it was
only excercised by a relatively small number of people compared
to how much the service is building).