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

Refactor RepoManager and add Service related pipelines #538

Merged
merged 14 commits into from
Jun 4, 2024

Conversation

bzeller
Copy link
Contributor

@bzeller bzeller commented Apr 12, 2024

Rather big patch that refactors RepoManager into a template usable for sync and async code execution and adds service related pipelines, as well as some changes related to them.

@bzeller bzeller force-pushed the service-refresh-pipeline branch 3 times, most recently from 42d9c8e to e06159a Compare April 17, 2024 17:26
@bzeller bzeller marked this pull request as ready for review April 17, 2024 18:08
@bzeller bzeller requested a review from mlandres April 17, 2024 18:08
@bzeller bzeller force-pushed the service-refresh-pipeline branch 6 times, most recently from daf5a39 to 0fa40e4 Compare April 25, 2024 14:08
bzeller added 14 commits May 6, 2024 07:52
This patchs adds the missing functionality of requesting a medium from
the user via the zypp::media::MediaChangeReport.

Also improves the logging with extra loggroups and improved
exception handling when returning the current exception in a expected.
This change was required because testcode depeneds on the fact that
KeyRingReport is started only once for a operation.
This test broke because of the fixes in the service refresh
workflow for dir:/ and file:/ service URLs that live in a
chroot.
This patch disables the use of the feature in probeRepoType that
fully downloads the files required to probe a repo into a cache for now.
Because files like repomd.xml do not have a known checksum we can not
reuse them from the cache. We need to revisit this later if we find a
better way.
@bzeller bzeller merged commit df3895f into openSUSE:master Jun 4, 2024
8 of 10 checks passed
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.

None yet

2 participants