-
Notifications
You must be signed in to change notification settings - Fork 576
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
NMS-15859: Inherit basic SSL config props from the producer to the stream client. #6412
Conversation
…ream client. Also document the behavior and existence of the stream config.
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.
minor doc changes. should it be merged to foundation?
docs/modules/operation/pages/deep-dive/kafka-producer/configure-kafka.adoc
Outdated
Show resolved
Hide resolved
docs/modules/operation/pages/deep-dive/kafka-producer/configure-kafka.adoc
Outdated
Show resolved
Hide resolved
docs/modules/operation/pages/deep-dive/kafka-producer/configure-kafka.adoc
Outdated
Show resolved
Hide resolved
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.
alarm-datasync-4ff962a7-05a9-47a0-9467-9bfba0afc4a5-GlobalStreamThread | GlobalStreamThread | 209 - org.apache.servicemix.bundles.kafka-clients - 3.3.1.1 | global-stream-thread [alarm-datasync-4ff962a7-05a9-47a0-9467-9bfba0afc4a5-GlobalStreamThread] State transition from CREATED to RUNNING
alarm-datasync-4ff962a7-05a9-47a0-9467-9bfba0afc4a5-GlobalStreamThread | KafkaStreams | 209 - org.apache.servicemix.bundles.kafka-clients - 3.3.1.1 | stream-client [alarm-datasync-4ff962a7-05a9-47a0-9467-9bfba0afc4a5] State transition from REBALANCING to RUNNING
kafka-producer-alarm-datasync-0 | KafkaStreams | 209 - org.apache.servicemix.bundles.kafka-clients - 3.3.1.1 | stream-client [alarm-datasync-4ff962a7-05a9-47a0-9467-9bfba0afc4a5] Started 0 stream threads
kafka-producer-alarm-datasync-0 | KafkaAlarmDataSync | 509 - org.opennms.features.kafka.producer - 31.0.6 | Starting alarm datasync started.
kafka-producer-alarm-datasync-0 | KafkaAlarmDataSync | 509 - org.opennms.features.kafka.producer - 31.0.6 | Waiting for alarm data store to be ready.
kafka-producer-alarm-datasync-0 | KafkaAlarmDataSync | 509 - org.opennms.features.kafka.producer - 31.0.6 | Alarm data store is ready!
Rad.
Co-authored-by: Bonrob2 <[email protected]>
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.
docs LGTM!
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.
LGTM.
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.
LGTM!
docs/modules/operation/pages/deep-dive/kafka-producer/configure-kafka.adoc
Outdated
Show resolved
Hide resolved
|
||
When producing alarms to a topic, we also automatically enable a Kafka Stream client to help synchronize the contents of the topic and ensure eventual consistency with the database. | ||
|
||
The streams client takes different properties than the producer and requires a separate configuration map. |
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.
"streams client" or "stream client"? it's singular when mentioned in the previous sentence. (and captialized)
Co-authored-by: mmahacek <[email protected]>
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.
LGTM.
JIRA: https://opennms.atlassian.net/browse/NMS-15859
Also document the behavior and existence of the stream config.