Initialize Kafka topics

ManageIQ / manageiq-appliance

System files for the ManageIQ appliance.

Apache License 2.0

7 stars 50 forks source link

Initialize Kafka topics #378

Closed nasark closed 11 months ago

nasark commented 1 year ago

Initialize Kafka topics that will be used in MIQ once messaging is ready

@miq-bot assign @agrare @miq-bot add_label enhancement

nasark commented 1 year ago

I need to test this on an appliance, marking as WIP for now

nasark commented 11 months ago

@agrare Tested and ready for review

nasark commented 11 months ago

@miq-bot add_label quinteros/yes?

nasark commented 11 months ago

Is this installed on the appliances?

@Fryguy /opt/kafka seems to be the location of kafka in downstream appliances. I haven't tested on upstream appliances but I'm assuming it would be the same?

miq-bot commented 11 months ago

Checked commits https://github.com/nasark/manageiq-appliance/compare/887f64690e99503590bc9432a0a3ac8715f643fd~...72ad999b2c932f8ca872d13731139ea35c67a5e4 with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 0 files checked, 0 offenses detected Everything looks fine. :cookie:

miq-bot commented 11 months ago

Some comments on commits https://github.com/nasark/manageiq-appliance/compare/887f64690e99503590bc9432a0a3ac8715f643fd~...72ad999b2c932f8ca872d13731139ea35c67a5e4

COPY/usr/bin/manageiq-messaging-ready

[ ] :warning: - 57 - Detected puts. Remove all debugging statements.
[ ] :warning: - 58 - Detected puts. Remove all debugging statements.
[ ] :warning: - 60 - Detected puts. Remove all debugging statements.

Fryguy commented 10 months ago

Backported to quinteros in commit adc9fd04a26435ae086869161b86e784df11330d.

commit adc9fd04a26435ae086869161b86e784df11330d
Author: Jason Frey <fryguy9@gmail.com>
Date:   Tue Dec 19 16:47:56 2023 -0500

    Merge pull request #378 from nasark/initialize_kafka_topics

    Initialize Kafka topics

    (cherry picked from commit 0ad54683c5938535c5388fbf4677043d8f11c85e)