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

Kafka - configurable javax.net.debug setting #1891

Merged
merged 2 commits into from
Dec 8, 2020

Conversation

toszo
Copy link
Contributor

@toszo toszo commented Dec 6, 2020

Modified kafka.service.j2 to handle configurable javax_net_debug which by default will be disabled (commented).

Tested on ubuntu onprem.

@toszo toszo linked an issue Dec 6, 2020 that may be closed by this pull request
plirglo
plirglo previously approved these changes Dec 8, 2020
sk4zuzu
sk4zuzu previously approved these changes Dec 8, 2020
Copy link
Contributor

@sk4zuzu sk4zuzu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -2,6 +2,12 @@
Description=Kafka Daemon
After=zookeeper.service

{% if specification.kafka_var.javax_net_debug is defined %}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kafka_var ? maybe we should refactor this sometime in the future ? 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sk4zuzu have you seen "Gopher's day" movie? ;)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know... 😞

toszo added 2 commits December 8, 2020 15:36
javax.net.debug option is disabled by default and configurable from kafka yaml (kafka_var.javax_net_debug)
@toszo toszo dismissed stale reviews from sk4zuzu and plirglo via ebd17a7 December 8, 2020 14:36
@toszo toszo force-pushed the fix/kafka-disable-debug branch from 216d368 to ebd17a7 Compare December 8, 2020 14:36
@toszo toszo merged commit 3970070 into hitachienergy:develop Dec 8, 2020
@atsikham atsikham mentioned this pull request Dec 20, 2021
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Kafka - Disable debug logging and make this option configurable
4 participants