-
Notifications
You must be signed in to change notification settings - Fork 76
Flesh out README.md #21
Comments
Crap, I just saw that you assigned this to yourself after I nearly finished a README. Would you mind if I submit a PR, and we can choose our favorite? |
I pushed my work so far (a copy/modify of the pires/kubernetes-elasticsearch-cluster README) to the If you like it, you can feel free to adopt and modify it. If your work is further along or is otherwise better, even better! 😄 In any case, I don't want to step on or reproduce your efforts. Apologies for not checking for an assignee. |
No problem, haven't started anything yet. |
Not sure duplicating that much content from pires is a good idea. I would rather link to the other repository in some cases. One more thing that I would add: a table of configuration variables (as seen in many official charts) |
Yeah, you're probably right. I'll pare it down.
Good thinking. I'll add that too. I may get to make more progress on this today. I'd like to get it in place asap though. |
Any update on README? |
This chart supports both kinds as well. Check the values.yaml file. |
The README file is depressingly sparse. I haven't had the time or energy to update it, but since it's the face of the project, and the project is starting to get some eyes, I'd like to consider this a priority.
Really, even if we just copy and modify the pires/kubernetes-elasticsearch-cluster README I think that should be fine.
I'll try to take care of this this weekend if nobody beats me to it.
The text was updated successfully, but these errors were encountered: