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 option to use kafka instead of dds #32

Merged
merged 2 commits into from
Mar 6, 2024
Merged

Conversation

glennmorris
Copy link
Contributor

New class parameters:
(dds, kafka) boolean switches
(kafka_broker_address, kafka_registry_url, kafka_sasl_password) (kafka_sasl_username, kafka_templates_directory) strings with kafka values

(rpms) nothing to install when not using dds.
(services) no opensplice when not using dds.
(etc) add kafka setup file when relevant, skip dds when not. Add kafka line to .app files when relevant. Use a template for app files. Always use setup-sal as the pre-execute file.

(files/setup-sal) new file
(templates/ocs-app.epp, templates/setup-sal-kafka.epp) new templates

New class parameters:
(dds, kafka) boolean switches
(kafka_broker_address, kafka_registry_url, kafka_sasl_password)
(kafka_sasl_username, kafka_templates_directory) strings with kafka values

(rpms) nothing to install when not using dds.
(services) no opensplice when not using dds.
(etc) add kafka setup file when relevant, skip dds when not.
Add kafka line to .app files when relevant. Use a template for app files.
Always use setup-sal as the pre-execute file.

(files/setup-sal) new file
(templates/ocs-app.epp, templates/setup-sal-kafka.epp) new templates
@glennmorris glennmorris added the enhancement New feature or request label Mar 6, 2024
@glennmorris glennmorris marked this pull request as ready for review March 6, 2024 22:00
@glennmorris glennmorris merged commit 4f89220 into master Mar 6, 2024
16 checks passed
@glennmorris glennmorris deleted the IT-5146/kafka branch March 6, 2024 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant