Skip to content

[8.19] [Obs AI Assistant] Display warning banner when re-indexing is in progress (#222593)#223906

Merged
SrdjanLL merged 2 commits intoelastic:8.19from
SrdjanLL:backport/8.19/pr-222593
Jun 13, 2025
Merged

[8.19] [Obs AI Assistant] Display warning banner when re-indexing is in progress (#222593)#223906
SrdjanLL merged 2 commits intoelastic:8.19from
SrdjanLL:backport/8.19/pr-222593

Conversation

@SrdjanLL
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…ress (elastic#222593)

Closes elastic#221837

## Summary

This PR adds a warning callout to the Assistant Page, Flyout and
Knowledge Base tab, to make users aware when knowledge base is
re-indexing.

This will help avoid users' worry that their data is lost since it's not
always shown during re-indexing.

The banner relies on the `isReIndexing` flag from the knowledge base
status. Once the state changes to `isReIndexing=false`, the banner
disappears.

### Screenshots

<img width="1481" alt="image"
src="https://github.com/user-attachments/assets/b56f4d7a-ce97-4c33-a80f-4ce342f3e8b7"
/>

<img width="1725" alt="image"
src="https://github.com/user-attachments/assets/38d63042-df9c-4d8b-9963-606cd076a67a"
/>

<img width="1481" alt="image"
src="https://github.com/user-attachments/assets/32908997-dfcc-4162-9103-f07ed21c27eb"
/>

#### When shown simultaneously with Elastic Managed LLM Callout

<img width="1481" alt="image"
src="https://github.com/user-attachments/assets/45bb49ec-149f-446a-ad1b-6773a08d8c95"
/>

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] This was checked for breaking HTTP API changes, and any breaking
changes have been approved by the breaking-change committee. The
`release_note:breaking` label should be applied in these situations.
- [] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed -**I'm not sure about this one - let me know
if there are any related tests I should execute here**.
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit 1d2ab08)

# Conflicts:
#	x-pack/platform/packages/shared/kbn-ai-assistant/src/chat/chat_timeline.tsx
@SrdjanLL SrdjanLL requested a review from kibanamachine as a code owner June 13, 2025 13:06
@SrdjanLL SrdjanLL added the backport This PR is a backport of another PR label Jun 13, 2025
@SrdjanLL SrdjanLL enabled auto-merge (squash) June 13, 2025 13:06
onActionClick,
chatState,
}: ChatTimelineProps) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

@SrdjanLL SrdjanLL merged commit 478e965 into elastic:8.19 Jun 13, 2025
8 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #1 / ColumnsPopover renders correctly a list of selected columns
  • [job] [logs] Jest Tests #4 / licensing plugin #start #createLicensePoller only fetch the license if ES is available avoid fetching the license too often if ES status comes and goes

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
observabilityAIAssistantApp 452 453 +1
observabilityAiAssistantManagement 410 411 +1
searchAssistant 283 284 +1
total +3

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
observabilityAIAssistantApp 273.4KB 274.4KB +1.0KB
observabilityAiAssistantManagement 120.6KB 121.2KB +660.0B
searchAssistant 174.7KB 175.8KB +1.0KB
total +2.7KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
searchAssistant 5.5KB 5.5KB +1.0B

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants