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

Update python-pulp-deb to 2.20.2 #648

Merged
merged 1 commit into from May 10, 2023

Conversation

quba42
Copy link
Contributor

@quba42 quba42 commented May 3, 2023

No description provided.

@quba42
Copy link
Contributor Author

quba42 commented May 3, 2023

Looks like foreman-packaging is already using the corresponding client for rpm/develop branch:
https://github.com/theforeman/foreman-packaging/tree/rpm/develop/packages/katello/rubygem-pulp_deb_client

Question: If this is merged now, will this be part of the upcoming Katello 4.9 release? (That would be good)
https://community.theforeman.org/t/katello-4-9-branching-process/33357

@ekohl
Copy link
Member

ekohl commented May 3, 2023

@quba42 Katello 4.8 already uses the 3.22 repos, so it would also land there.

@quba42
Copy link
Contributor Author

quba42 commented May 3, 2023

@ekohl Then it might make sense to also update the client gem there: https://github.com/theforeman/foreman-packaging/tree/rpm/3.6/packages/katello/rubygem-pulp_deb_client

I believe this would not require a change to the Katello RPM, since it allows client gems up to the next Y release.

I do not know for certain if updating to this version of pulp_deb requires the newer client, but there is reason to believe that it might do so. (which is admittedly awkward, since this is a Z-release).

Should I go ahead and open a cherry-pick PR against foreman-packaging rpm/3.6?

@quba42
Copy link
Contributor Author

quba42 commented May 5, 2023

I went ahead and opened theforeman/foreman-packaging#9336 to go with this.

Ideally both this and the foreman-packaging PR should be merged (or rather packaged) together.

@evgeni evgeni merged commit c4012e9 into theforeman:rpm/3.22 May 10, 2023
4 checks passed
@quba42 quba42 deleted the update_pulp_deb branch May 10, 2023 12:33
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

3 participants