Remove docs for managed agent on k8s#799
Conversation
5142ff5 to
03630b1
Compare
|
A documentation preview will be available soon: |
|
I wonder if we should fully remove it or instead mainly remove the links to the manifest files and mention that it is not fully supported yet. But we have already questions on discuss and users will run it on k8s, so some guidance would be good. |
|
@ruflin I think that providing guidance without an actual manifest would be incomplete. Also I have not run the case yet so I'm not sure what kind of guidance we could provide here. We can prioritise the work for elastic/beats#25237 even in a primitive stage so as to have something complete soon. We also need this for ourselves when we test k8s integration/package manually in managed mode. |
|
SGTM |
|
Can we just remove this from the documentation build rather than deleting the file completely? To do that, you just need to comment out lines 19 and 34 If you think it's better to remove the file completely, then make sure you also remove these lines ^^ to fix the doc build. |
Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
03630b1 to
245bd37
Compare
|
@dedemorton what you proposed sounds better so I applied your approach, thanks! |
|
@dedemorton can we have this in soon since the manifests have already been removed? |
dedemorton
left a comment
There was a problem hiding this comment.
LGTM! Sorry about the delay. I fell down a TLS rabbit hole.
|
I'm going to take the liberty of merging and backporting this. |
Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
This PR removes section for managed elastic-agent on k8s since we don't currently officially support it.
K8s manifests are also being removed: elastic/beats#26368