elastic / observability-docs

Elastic Observability Documentation
Other
34 stars 161 forks source link

[Request]: Consider removing or updating the reindex method on the Observability AI Assistant doc #4352

Open eedugon opened 2 weeks ago

eedugon commented 2 weeks ago

Description

The procedure described on the section === Use the {es} Reindex API of the document has the following issues:

1) It's too advanced. 2) The destination index proposed might not be valid if data is rolled over. 3) The first step about indexing data with the Index API is now useless considering they can do that with Search Connectors and also integrate the data directly with the AI Assistant knowledge base.

Resources

Background:

Now that we have added in https://github.com/elastic/observability-docs/pull/4326 the information about how Search Connectors can help to enrich the AI Assistant Knowledge base, the method of using Reindex API proposed in the doc on the section...

=== Use the {es} Reindex API

... is probably not needed (or at least not needed as it is).

I would suggest to:

If we want to keep the content, then the following updates should be needed (IMO):

cc: @lucabelluccini , if possible I'd love to hear your opinion on this. cc: @emma-raffenne , as this is a follow up of our previous work, could you check this with @dgieselaar or someone else from the team?

Which documentation set does this change impact?

Stateful and Serverless

Feature differences

Both docs are very similar so in case of moving forward with this we should update both documents.

What release is this request related to?

8.16

Collaboration model

The documentation team

Point of contact.

Main contact: @eedugon

Stakeholders: @emma-raffenne

emma-raffenne commented 1 week ago

@eedugon Thank you for creating this issue. As @dgieselaar mentioned elsewhere, we don't want to recommend this method for adding content to the Knowledge Base, so IMO it's safe to remove it entirely from the user documentation.