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

Make distrosync as a targeted operation #242

Merged
merged 1 commit into from
Jan 20, 2023

Conversation

j-mracek
Copy link
Contributor

DNF provides all candidates for transaction to libsolv, therefore we have to force libsolv to use only those candidates for the job solution => to make job TARGETED. If distrosync or upgrade job is not marked as TARGETED explicitly, libsolv mark as TARGETED job without installed package in the set of candidates.

The patch unify upgrade and distrosync flags in DNF5 => using SOLVER_TARGETED.

Resolves: #222

@pkratoch pkratoch self-assigned this Jan 20, 2023
All candidates for distrosync transaction are provided by DNF, therefore
we have to force libsolv to use only those candidates for the job
solution => make job TARGETED. When job is not marked explicitly as
TARGETED, libsolv try to auto-detect TARGETED job. As a key it uses
presence or absence of installed packages (from @System repository)
in the provided set of candidates. If no installed package in the set,
it handles the job as TARGETED. TARGETED means that only packages in
the provided set can be use for a solution of the job.

The patch unify upgrade and distrosync flags in DNF5 => using
SOLVER_TARGETED.

Resolves: rpm-software-management#222
@pkratoch pkratoch merged commit 9d0ed3d into rpm-software-management:main Jan 20, 2023
@kontura
Copy link
Contributor

kontura commented Jan 23, 2023

It would be great to have a CI test for case.

@j-mracek
Copy link
Contributor Author

It would be great to have a CI test for case.

Good point

@j-mracek
Copy link
Contributor Author

What about a test - rpm-software-management/ci-dnf-stack#1211

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.

distro-sync shows a lot of unknown problems
3 participants