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

Fixes #38231 - Provide fallback to simple deb publish #11321

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

quba42
Copy link
Contributor

@quba42 quba42 commented Feb 20, 2025

Co-authored-by: @m-bucher

We have encountered some rare corner cases where pulp_deb repos created using older Katello versions do not have any PRC structure content. In such cases we fall back to simple publishing. In the future we may replace this with a data repair migration on the pulp_deb side.

@quba42 quba42 force-pushed the structured_apt_fallback branch 4 times, most recently from fdcd46b to 28681e0 Compare February 20, 2025 17:46
Co-authored-by: Markus Bucher <bucher@atix.de>

We have encountered some rare corner cases where pulp_deb repos created
using older Katello versions do not have any PRC structure content. In
such cases we fall back to simple publishing. In the future we may
replace this with a data repair migration on the pulp_deb side.
@quba42 quba42 force-pushed the structured_apt_fallback branch from 28681e0 to 140ff95 Compare February 24, 2025 10:25
@quba42 quba42 marked this pull request as ready for review February 24, 2025 10:26
Copy link
Contributor

@m-bucher m-bucher left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@m-bucher m-bucher merged commit a94ad14 into Katello:master Feb 24, 2025
18 of 19 checks passed
@m-bucher m-bucher deleted the structured_apt_fallback branch February 24, 2025 16:00
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.

2 participants