-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Need approval to remove EFK #3352
Comments
I approve ! :) |
In short, if it was not that complex I would be ok to leave it as a kubespray addons. Logs manager could be seen as a core components of a cluster. +1 to remove it. |
Maybe we should provide a list of helm charts / commands in some sort of contrib, to show people how to install EFK with helm instead? So we only maintain a list of helm charts if so ? |
i think documentation about helm charts would be sufficient |
A recommendation or a list of recommendations in the doc would be helpful. |
+1 |
Remove it. Thanks! |
+1 for removing efk |
Although I actually like the idea of focusing on core and removing all of this, I think it has been done too quickly. From the outside, it feels like kubespray does not care about its users. I know it is not true, but at least a few lines of documentation stating the breaking changes AND giving some clue about how to install an EFK would be nice. I personally don't care as I know how to do the switch, but I don't think newcomers would think kubespray as being "stable" seeing this. |
I could not disagree more. EFK is still available in the latest release. The discussion about removing EFK has been up for very long. Yesterday it was discussed in our Slack channel again. So the first step here was/is to remove EFK from upstream. Next step must be to include various docs / contrib for people that like to use EFK etc. |
Then we agree in this case. All I'm telling is "tell the users", but if it is a planned next step then I'm happy with it. |
Neeed approval to remove the EFK stack from Kubespray.
@ant31
@mattymo
@Atoms
@chadswen
@rsmitty
@bogdando
@bradbeam
@riverzhang
@holser
@Smana
@mirwan
The text was updated successfully, but these errors were encountered: