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

Add documentation for Kafka keystores #1728

Closed
wants to merge 1 commit into from

Conversation

nasark
Copy link
Member

@nasark nasark commented Mar 31, 2023

  • added documentation outlining steps to create keystores for Kafka and how to integrate them with internal-certificates-secret in order to use SSL with Kafka

Depends on:

Needed for:

Ref:

@miq-bot assign @bdunne
@miq-bot add_reviewer @Fryguy
@miq-bot add_label enhancement

@agrare
Copy link
Member

agrare commented Apr 4, 2023

@bdunne how is this done for other services like postgres/memcached? I don't think there are extra manual steps for those?

@nasark
Copy link
Member Author

nasark commented Apr 4, 2023

@agrare they use the cert-generator script https://github.com/ManageIQ/manageiq-pods/blob/master/tools/cert_generator.rb. Kafka uses keystores for SSL which require the use of keytool and so I provided manual steps. We could add a script that does this instead perhaps?

Kafka does support PEM format but the keys generated by the cert-generator script are in PKSC#1 whereas Kafka uses PKSC#8 and I was not able to make the conversion

@nasark
Copy link
Member Author

nasark commented Apr 10, 2023

@miq-bot
Copy link
Member

miq-bot commented Apr 24, 2023

Checked commit nasark@38ddb52 with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 👍

@miq-bot miq-bot added the stale label Jul 31, 2023
@miq-bot
Copy link
Member

miq-bot commented Jul 31, 2023

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s)

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@nasark
Copy link
Member Author

nasark commented Jul 31, 2023

@miq-bot remove_label stale

@miq-bot miq-bot removed the stale label Jul 31, 2023
@miq-bot miq-bot added the stale label Nov 6, 2023
@miq-bot
Copy link
Member

miq-bot commented Nov 6, 2023

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s)

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@nasark
Copy link
Member Author

nasark commented Dec 22, 2023

This is no longer applicable as we have moved to using strimzi

@nasark nasark closed this Dec 22, 2023
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.

4 participants