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.

Sign up for GitHub

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

Jump to bottom

Addon: storage classes for dynamically provisioned volumes #50

Open
wants to merge 36 commits into
base: kafka-011
Choose a base branch
from
Open

Addon: storage classes for dynamically provisioned volumes #50

wants to merge 36 commits into from

Conversation

solsson
Copy link
Contributor

@solsson solsson commented Jul 28, 2017

Lets you chose storage type

@solsson solsson changed the base branch from master to kafka-011 July 28, 2017 06:25
@solsson
Merge pull request #30 from Yolean/kafka-011
1aac57a 
Upgrade to Kafka 0.11 and test for open issues
@solsson solsson force-pushed the addon-storage-classes branch from 3ec89df to bed63b3 Compare July 29, 2017 02:56
@solsson solsson force-pushed the addon-storage-classes branch from bed63b3 to 008d522 Compare July 29, 2017 12:18
@solsson
Copy link
Contributor Author

solsson commented Aug 4, 2017

With Kubernets 1.8 we should set Retain as default. See #57.

@solsson solsson mentioned this pull request Aug 4, 2017
Merged
solsson and others added 16 commits August 5, 2017 05:47
@solsson
RBAC rights are purely additive so ...
7cf2a5d 
a project like kubernetes-kafka should keep them minimal.
To access nodes we do need ClusterRole instead of Role.
@solsson
I don't really care, but which yaml indentation is winning?
05107fd
@solsson
With default service account curl works again
a8ee55b 
fixes #39
@solsson
Got the feeling from kubectl get clusterrole ...
3597426 
that having access control rules, in particular cluster scoped,
lying around without knowing where they come from
will be unmaintainable over time. Labels show up nicely in describe.
@solsson
Recommends that you create rbac
8f637b7
@solsson
Shows how to see that you need rbac, but makes readme heavier
27421fb
@solsson
Addons can maintain their policies, so moving this to
1c6b7bb 
#39
@solsson
Details will live in the respective policies
79d65fd
@solsson
Moved to its PR, multizone-rack-awareness
13520a6
@solsson
Merge pull request #59 from Yolean/rbac-namespace-default
57c343b 
RBAC policies through kubectl apply -f rbac-namespace-default/
@solsson
Temp commit so that github allows a PR to be created, where I can kee…
23e1d9e 
…p notes
@solsson
Starts scripting, but the API call gets 403 for anonymous user
c0645ee
@solsson
Looks up zone if kubectl is found, tries to not break config otherwise
8fe76ff
@solsson
Uses an image with kubectl based on the same debian tag as kafka
7e7b342
@solsson
I suppose most init scripts will do fine with curl+kubectl+bash
3678ad5
@solsson
Adds RBAC policy for kubectl to look up node's zone
ff972b9
solsson and others added 11 commits August 5, 2017 07:05
@solsson
Merge pull request #41 from Yolean/multizone-rack-awareness
09e733b 
Multizone rack awareness
@solsson
got merged
723c514
@solsson
The default is Delete, which violates our no-surpises ambition
2ca4453
@solsson
Shorter, possibly clearer
7e83bac
@solsson
We prefer Ready:False status instead of restarted pods,
1f4321e 
at least for now, as it allows exec into the pods to investigate.
We've been having frequent restarts that are not due to OOMKilled (i.e. not #49).
Now failed probes will lead to unready pods, which we can monitor for using #60.
@solsson
Don't look like env var interpolation, when it's not
0295ac5
@solsson
I think Schema Registry + REST Proxy is ok for production now
197eaf3
@solsson
Still short
4d3754d
@solsson
Merge branch 'persistent-volumes-retain', the readme ...
11496d2 
part that recommends manual intervention
@solsson
Adds storage class for Zookeeper, with sample manifests
c3969a8 
This reverts commit efb1019.
@solsson
Adds storage class for kafka brokers
ebcff38
@solsson solsson force-pushed the addon-storage-classes branch from 008d522 to ebcff38 Compare August 8, 2017 11:02
@solsson solsson mentioned this pull request Sep 5, 2017
Closed
@solsson solsson mentioned this pull request Nov 9, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@solsson