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.

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

[8.x](backport #4326) AI Assistant docs updates for 8.16 #4383

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Oct 16, 2024

Description

AI Assistant doc updates for 8.16, applying changes requested in #4273.

Preview link: https://observability-docs_bk_4326.docs-preview.app.elstc.co/guide/en/observability/master/obs-ai-assistant.html

The PR includes:

  • Added Enterprise Search requirement in case Search Connectors are used.
  • Included Search Connectors as a way to add data to the AI Assistant KB (Use Search Connectors section)
  • Updated names for the other methods (knowledge base UI and Index API)
  • Updated the icon related screenshot.
  • Added AI Assistant Settings section with screenshot.

Pending:

  • Determine if we should update the rest of the screenshots and gif --> This will be handled in a separate issue / separate PR, and we will avoid using screenshots if not needed (thanks @bmorelli25 ).
  • Determine if we can remove the method about using the Index and Reindex API calls to add data to the AI Assistant KB, as it's too advanced and also doesn't appear completely accurate (maybe we could move that to another type of document) --> Moved to a new issue ([Request]: Consider removing or updating the reindex method on the Observability AI Assistant doc #4352) for a later stage and decision.
  • Should we add a clarification about the different nature of Search Connectors (used to populate the KB) and Stack Connectors (used to connect to LLM). Users could mix them up. --> Agreed to add a TIP in the doc.
  • I still need to add some details about Search Connectors but I was unable because of technical issues in the lab environment.
  • Backporting checks: Are the changes in this PR valid for other releases? If Search Connectors were also integrated with the AI Assistant in other releases I'd say yes, but I'm not sure about it.
  • Port to serveless decision (depends on the final content of this PR) --> Decided to not port to serverless as Search Conectors cannot be used there to populate the AI Assistant knowledge base.

Other concerns:

  • Search Connectors wording: Should we use Connectors or Search Connectors? I think Connectors is more ambiguous considering we have other type of connectors.

Documentation sets edited in this PR

Check all that apply.

  • Stateful (docs/en/observability/*)
  • Serverless (docs/en/serverless/*)
  • Integrations Developer Guide (docs/en/integrations/*)
  • None of the above

Related issue

Closes #4273

Checklist

  • Product/Engineering Review
  • Writer Review

Select one.

  • This PR does not need to be ported to another doc set because:
    • The concepts in this PR only apply to one doc set (serverless or stateful)
    • The PR contains edits to both doc sets (serverless and stateful)
  • This PR needs to be ported to another doc set:
    • Port to stateful docs: <link to PR or tracking issue>
    • Port to serverless docs: <link to PR or tracking issue>

Serverless docs not updated due to #4273 (comment)


This is an automatic backport of pull request #4326 done by [Mergify](https://mergify.com).

* initial structure change to add settings and search connectors

* enterprise search requirement added

* search connectors explained

* fixed image link

* extra comma removed from json example

* ai settings moved to the end

* tbd content added

* screenshots deleted

* some you-cans removed

* connectors distinction included

* minor updates

* AI Assistant icon added

* AI Assistant icon added

* AI Assistant icon added

* search connectors setup added

* passive voice and to_do_this update

* reindex method changes cancelled

* override search connector indices list

* reindex method removed

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

* missing link added

* Update docs/en/observability/observability-ai-assistant.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

---------

Co-authored-by: Arianna Laudazzi <[email protected]>
(cherry picked from commit 20cc3dc)
@mergify mergify bot requested a review from a team as a code owner October 16, 2024 16:20
@mergify mergify bot added the backport label Oct 16, 2024
Copy link
Contributor

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@bmorelli25 bmorelli25 merged commit 6863d4c into 8.x Oct 18, 2024
8 checks passed
@bmorelli25 bmorelli25 deleted the mergify/bp/8.x/pr-4326 branch October 18, 2024 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants