-
Notifications
You must be signed in to change notification settings - Fork 99
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 creation of manageiq.liveness-check topic #1101
Conversation
Need a cr migration to remove the |
46749fa
to
23007fd
Compare
@bdunne Added the CR migration, please review |
23007fd
to
73b87c5
Compare
@@ -59,6 +60,19 @@ func FindKafka(client client.Client, scheme *runtime.Scheme, namespace string, n | |||
return kafka | |||
} | |||
|
|||
func FindKafkaTopic(client client.Client, scheme *runtime.Scheme, namespace string, name string, group string) *unstructured.Unstructured { |
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.
Added group
as a parameter so that this function can be reused downstream
Checked commits nasark/manageiq-pods@905ebb3~...73b87c5 with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint |
Backported to
|
Remove creation of manageiq.liveness-check topic (cherry picked from commit 3ee8237)
We no longer use a push/pop approach as health check for kafka via the orchestrator and so this topic no longer needs to be created
Ref: ManageIQ/manageiq#23020
@miq-bot assign @bdunne
@miq-bot add_reviewers @bdunne, @Fryguy
@miq-bot add_labels bug, radjabov/yes?