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

Remove all qpid related certificate handling #414

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Sep 20, 2023

When all references to Qpid are dropped, this can be merged. As Katello 4.10 should be handling that, we ought to be able to remove the code references in puppet-katello and puppet-foreman_proxy_content and then merge this.

@ekohl
Copy link
Member

ekohl commented Sep 20, 2023

With a downstream hat on, won't we still need to ensure things are absent in Katello 4.11?

Copy link
Member

@evgeni evgeni left a comment

Choose a reason for hiding this comment

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

The README still refers to qpid certs, otherwise LGTM

Verified

This commit was signed with the committer’s verified signature.
agliga Andrew Gliga
@ehelms ehelms merged commit 5cb07de into theforeman:master Mar 20, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants