-
Notifications
You must be signed in to change notification settings - Fork 735
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
Track progress for Kubernetes 1.8 / kubernetes-kafka v3.0.0 #84
Comments
|
Found an explanation for why |
An update on testing, based on #79 (comment). I lack the Kafka experiment to interpret current results, so I think I want 3 tests that do essentially the same but with different clients. What we want to assert is basically "uptime" in the face of re-scheduled broker and zk pods, caused by things like node downtime, cluster upgrades or zone outages. Tests continuously "bootstrap" + consume from a topic with 2 replicas (b8bfda8) + regularly produce messages + assert that those messages get consumed. Measurements of throughput etc will have to wait until we have Prometheus monitoring up and running (#49 + Yolean/kubernetes-monitoring + ServiceMonitors + rules). The three tests are:
As a complement it'll be interesting to have kafkacat with new bootstrap for every assert run, i.e. https://github.com/Yolean/kubernetes-kafka/blob/master/test/basic-with-kafkacat.yml prior to #79) |
Killing pods and watching test readiness I tend to need a tab that does a human readable variant of #60. My new favorite oneliner 😄 (with alias from solsson/kubectx#1 (comment)): while :; do k get pods --all-namespaces -w; done | gawk '{ print strftime("%FT%T"), $0; fflush() }' |
Tests have been put into practice in #79 (comment) Huge improvement that we don't spin up 2 JVMs every 10s :) |
Closing this messy ticket in favor of https://github.com/Yolean/kubernetes-kafka/milestone/1, with scope reductions -> https://github.com/Yolean/kubernetes-kafka/milestone/2. |
Manifests:
Retain
(Use k8s 1.8 storage class syntax and set Retain by default #85)Test open PRs:
Test tests:
Structure:
The text was updated successfully, but these errors were encountered: