Merge ~cjwatson/launchpad:kafka-producer into launchpad:master
Proposed by
Colin Watson
Status: | Needs review |
---|---|
Proposed branch: | ~cjwatson/launchpad:kafka-producer |
Merge into: | launchpad:master |
Diff against target: |
324 lines (+246/-0) 10 files modified
lib/lp/services/config/schema-lazr.conf (+22/-0) lib/lp/services/configure.zcml (+1/-0) lib/lp/services/kafka/__init__.py (+0/-0) lib/lp/services/kafka/client.py (+59/-0) lib/lp/services/kafka/configure.zcml (+16/-0) lib/lp/services/kafka/interfaces.py (+41/-0) lib/lp/services/kafka/tests/__init__.py (+0/-0) lib/lp/services/kafka/tests/test_client.py (+105/-0) requirements/launchpad.txt (+1/-0) setup.cfg (+1/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Guruprasad | Approve | ||
Review via email: mp+439414@code.launchpad.net |
Commit message
Add a simple Kafka producer abstraction
Description of the change
This isn't used anywhere as yet, but in an earlier experiment I was able to use this to send events to Kafka on `git push`.
I want the abstraction layer in place for ease of testing, and in order that we can switch to different bindings without too much trouble, since we aren't very familiar with the operational properties of these bindings as yet.
Dependencies MP: https:/
To post a comment you must log in.
Unmerged commits
- da53c25... by Colin Watson
-
docs:0 (build) lint:0 (build) mypy:0 (build) 1 → 3 of 3 results First • Previous • Next • Last
LGTM 👍