-
Notifications
You must be signed in to change notification settings - Fork 1
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 second (and legacy) kafka consumer if env vars for it differ #30
Conversation
README.md
Outdated
| KAFKA_LEGACY_VERSION | `1.0.2` | The version of Kafka - non-TLS | ||
| KAFKA_ADDR | `localhost:9092` | The addresses of the kafka brokers (CSV) | ||
| KAFKA_VERSION | `1.0.2` | The version of Kafka | ||
| KAFKA_SEC_PROTO | _unset_ | if set to `TLS`, kafka connections will use TLS [1] |
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.
Maybe link [1]
to notes bullet point 1 - not necessarily obvious why this is here unless you review PR or look at code changes
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.
is this any better?
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.
You can use this as a reference (ONSdigital/dp-dimension-search-api#89). This was reviewed by Nathan and approved
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.
Thanks Justin - yes I was thinking of the same changes made here
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.
my markdown linter complains about inline-HTML for your method, so I prefer my method 😛
(frankly, I think if people don't understand the original footnote syntax, then 🤷 but here we are...)
9531a93
to
c00d57f
Compare
What
(currently feature-flagged off in secrets)
add a second kafka consumer where secrets determine a second (differing) kafka server address
add TLS to kafka consumer, if requested
How to review
test with features off
Who can review
!gedge