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
Pulpcore upgrade #10949
Pulpcore upgrade #10949
Conversation
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.
Looks good! Just need to wait to merge until the packaging PRs (like theforeman/foreman-packaging#10658) are ready.
Added teardown code to some of our tests which didn't have those. Also added code path to look up dangling distributions during orphan cleanup and version deletion to resolve this: https://projects.theforeman.org/issues/37314 |
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.
I found one bug where un-orphaned distributions were deleted, details below:
distribution_deletion_tasks = [] | ||
distro_hrefs = [] | ||
orphan_repository_versions.each do |api, version_hrefs| | ||
next if api.repository_type.id == :deb |
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.
Is there no workaround for Debian?
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.
Looking good! Just one question:
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.
Acking again! Still needs to wait for the other packaging ones to be good to go to merge.
(cherry picked from commit 4b306986bb493e77882b386984abc2850c030463)
Related pulpcore upgrade PRs:
theforeman/foreman-packaging#10658
theforeman/pulpcore-packaging#931
theforeman/foreman-packaging#10667
What are the changes introduced in this pull request?
Update pulp bindings and VCRs
Considerations taken when implementing this change?
What are the testing steps for this pull request?
On a box with upgraded pulp packages, check this out and run through some smoke testing.