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.

Sign up for GitHub

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

Jump to bottom

[Security Solution] [Bug] Wrong successful message with error notification pop up on adding a wrong format file under bulk import for knowledge base #196096

Closed
muskangulati-qasource opened this issue Oct 14, 2024 · 6 comments · Fixed by #205075
Closed

[Security Solution] [Bug] Wrong successful message with error notification pop up on adding a wrong format file under bulk import for knowledge base #196096

muskangulati-qasource opened this issue Oct 14, 2024 · 6 comments · Fixed by #205075
Assignees
@arturoliduena
Labels
bug Fixes for quality problems that affect the customer experience impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. Team:Obs AI Assistant Observability AI Assistant Team:Observability Team label for Observability Team (for things that are handled across all of observability) triage_needed v8.16.0

Comments

@muskangulati-qasource
Copy link

Describe the bug
Wrong successful message with error notification pop up on adding a wrong format file under bulk import for knowledge base

Kibana/Elasticsearch Stack version

VERSION: 8.16.0
BUILD: 78825
COMMIT: a805375758e4bc931cf13dfdcac89b8d877a15d2

Steps

  1. Kibana version 8.16.0 or above should exist without endpoints
  2. Navigate to Stack management under Management section
  3. Navigate to Kibana >> AI Assistants
  4. Click on Manage settings for Elastic AI Assistant for Observability
  5. Navigate to the Knowledge base tab
  6. Click on the icon for new entry
  7. Add a wrong extension file
  8. Observe the error notification pops up and the successful message is also shown to the user

Expected Result
There should be a notification message for successful deletion

Screen Recording

AIAssistantImportError.mp4
@muskangulati-qasource muskangulati-qasource added bug Fixes for quality problems that affect the customer experience impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. triage_needed v8.16.0 labels Oct 14, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@muskangulati-qasource
Copy link
Author

@amolnater-qasource please review!!

@amolnater-qasource
Copy link

Reviewed & assigned to @MadameSheema

@MadameSheema MadameSheema added the Team:Security Generative AI Security Generative AI label Oct 14, 2024
@peluja1012 peluja1012 added Team:Observability Team label for Observability Team (for things that are handled across all of observability) and removed Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. labels Dec 12, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/unified-observability (Team:Observability)

@peluja1012 peluja1012 added Team:obs-knowledge Observability Experience Knowledge team and removed Team:Security Generative AI Security Generative AI labels Dec 12, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-knowledge-team (Team:obs-knowledge)

@emma-raffenne emma-raffenne added Team:Obs AI Assistant Observability AI Assistant and removed Team:obs-knowledge Observability Experience Knowledge team labels Dec 16, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant)

@emma-raffenne emma-raffenne removed their assignment Dec 16, 2024
@viduni94 viduni94 self-assigned this Dec 22, 2024
@arturoliduena arturoliduena mentioned this issue Dec 23, 2024
Merged
arturoliduena added a commit that referenced this issue Jan 7, 2025
@arturoliduena @sorenlouv @kibanamachine
[Obs ai assistant] - bug bulk import for knowledge base (#205075)
2c97ed3 
Closes #196096 

## Summary

Wrong successful message with error notification pop up on adding a
wrong format file under bulk import for knowledge base
[#196096](#196096)

---------

Co-authored-by: Søren Louv-Jansen <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Jan 7, 2025
@arturoliduena
[Obs ai assistant] - bug bulk import for knowledge base (elastic#205075)
23587a1 
Closes elastic#196096

## Summary

Wrong successful message with error notification pop up on adding a
wrong format file under bulk import for knowledge base
[elastic#196096](elastic#196096)

---------

Co-authored-by: Søren Louv-Jansen <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 2c97ed3)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Jan 7, 2025
@arturoliduena
[Obs ai assistant] - bug bulk import for knowledge base (elastic#205075)
3a601f0 
Closes elastic#196096

## Summary

Wrong successful message with error notification pop up on adding a
wrong format file under bulk import for knowledge base
[elastic#196096](elastic#196096)

---------

Co-authored-by: Søren Louv-Jansen <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 2c97ed3)
kibanamachine added a commit that referenced this issue Jan 7, 2025
@kibanamachine @arturoliduena
[8.17] [Obs ai assistant] - bug bulk import for knowledge base (#205075…
b56ed4c 
…) (#205687)

# Backport

This will backport the following commits from `main` to `8.17`:
- [[Obs ai assistant] - bug bulk import for knowledge base
(#205075)](#205075)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Arturo
Lidueña","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-07T09:44:16Z","message":"[Obs
ai assistant] - bug bulk import for knowledge base (#205075)\n\nCloses
#196096 \r\n\r\n## Summary\r\n\r\nWrong successful message with error
notification pop up on adding a\r\nwrong format file under bulk import
for knowledge
base\r\n[#196096](https://github.com/elastic/kibana/issues/196096)\r\n\r\n---------\r\n\r\nCo-authored-by:
Søren Louv-Jansen <[email protected]>\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"2c97ed3bfec8a99de931d85f50b1e940835bde47","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","v9.0.0","Team:Obs
AI
Assistant","v8.16.0","backport:version","v8.17.0","v8.18.0"],"title":"[Obs
ai assistant] - bug bulk import for knowledge
base","number":205075,"url":"https://github.com/elastic/kibana/pull/205075","mergeCommit":{"message":"[Obs
ai assistant] - bug bulk import for knowledge base (#205075)\n\nCloses
#196096 \r\n\r\n## Summary\r\n\r\nWrong successful message with error
notification pop up on adding a\r\nwrong format file under bulk import
for knowledge
base\r\n[#196096](https://github.com/elastic/kibana/issues/196096)\r\n\r\n---------\r\n\r\nCo-authored-by:
Søren Louv-Jansen <[email protected]>\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"2c97ed3bfec8a99de931d85f50b1e940835bde47"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.17","8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205075","number":205075,"mergeCommit":{"message":"[Obs
ai assistant] - bug bulk import for knowledge base (#205075)\n\nCloses
#196096 \r\n\r\n## Summary\r\n\r\nWrong successful message with error
notification pop up on adding a\r\nwrong format file under bulk import
for knowledge
base\r\n[#196096](https://github.com/elastic/kibana/issues/196096)\r\n\r\n---------\r\n\r\nCo-authored-by:
Søren Louv-Jansen <[email protected]>\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"2c97ed3bfec8a99de931d85f50b1e940835bde47"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Arturo Lidueña <[email protected]>
kibanamachine added a commit that referenced this issue Jan 7, 2025
@kibanamachine @arturoliduena
[8.x] [Obs ai assistant] - bug bulk import for knowledge base (#205075)…
def21fa 
… (#205688)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Obs ai assistant] - bug bulk import for knowledge base
(#205075)](#205075)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Arturo
Lidueña","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-07T09:44:16Z","message":"[Obs
ai assistant] - bug bulk import for knowledge base (#205075)\n\nCloses
#196096 \r\n\r\n## Summary\r\n\r\nWrong successful message with error
notification pop up on adding a\r\nwrong format file under bulk import
for knowledge
base\r\n[#196096](https://github.com/elastic/kibana/issues/196096)\r\n\r\n---------\r\n\r\nCo-authored-by:
Søren Louv-Jansen <[email protected]>\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"2c97ed3bfec8a99de931d85f50b1e940835bde47","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","v9.0.0","Team:Obs
AI
Assistant","v8.16.0","backport:version","v8.17.0","v8.18.0"],"title":"[Obs
ai assistant] - bug bulk import for knowledge
base","number":205075,"url":"https://github.com/elastic/kibana/pull/205075","mergeCommit":{"message":"[Obs
ai assistant] - bug bulk import for knowledge base (#205075)\n\nCloses
#196096 \r\n\r\n## Summary\r\n\r\nWrong successful message with error
notification pop up on adding a\r\nwrong format file under bulk import
for knowledge
base\r\n[#196096](https://github.com/elastic/kibana/issues/196096)\r\n\r\n---------\r\n\r\nCo-authored-by:
Søren Louv-Jansen <[email protected]>\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"2c97ed3bfec8a99de931d85f50b1e940835bde47"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.17","8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205075","number":205075,"mergeCommit":{"message":"[Obs
ai assistant] - bug bulk import for knowledge base (#205075)\n\nCloses
#196096 \r\n\r\n## Summary\r\n\r\nWrong successful message with error
notification pop up on adding a\r\nwrong format file under bulk import
for knowledge
base\r\n[#196096](https://github.com/elastic/kibana/issues/196096)\r\n\r\n---------\r\n\r\nCo-authored-by:
Søren Louv-Jansen <[email protected]>\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"2c97ed3bfec8a99de931d85f50b1e940835bde47"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Arturo Lidueña <[email protected]>
kowalczyk-krzysztof pushed a commit to kowalczyk-krzysztof/kibana that referenced this issue Jan 7, 2025
@arturoliduena @sorenlouv
@kibanamachine @kowalczyk-krzysztof
[Obs ai assistant] - bug bulk import for knowledge base (elastic#205075)
024f7ef 
Closes elastic#196096 

## Summary

Wrong successful message with error notification pop up on adding a
wrong format file under bulk import for knowledge base
[elastic#196096](elastic#196096)

---------

Co-authored-by: Søren Louv-Jansen <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Jan 13, 2025
@arturoliduena @sorenlouv
@kibanamachine @CAWilson94
[Obs ai assistant] - bug bulk import for knowledge base (elastic#205075)
57b1b31 
Closes elastic#196096 

## Summary

Wrong successful message with error notification pop up on adding a
wrong format file under bulk import for knowledge base
[elastic#196096](elastic#196096)

---------

Co-authored-by: Søren Louv-Jansen <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
viduni94 pushed a commit to viduni94/kibana that referenced this issue Jan 23, 2025
@arturoliduena @sorenlouv
@kibanamachine @viduni94
[Obs ai assistant] - bug bulk import for knowledge base (elastic#205075)
18d1c51 
Closes elastic#196096 

## Summary

Wrong successful message with error notification pop up on adding a
wrong format file under bulk import for knowledge base
[elastic#196096](elastic#196096)

---------

Co-authored-by: Søren Louv-Jansen <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arturoliduena arturoliduena

Labels
bug Fixes for quality problems that affect the customer experience impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. Team:Obs AI Assistant Observability AI Assistant Team:Observability Team label for Observability Team (for things that are handled across all of observability) triage_needed v8.16.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Obs ai assistant] - bug bulk import for knowledge base arturoliduena/kibana
9 participants
@peluja1012 @elasticmachine @MadameSheema @viduni94 @jamesspi @arturoliduena @muskangulati-qasource @amolnater-qasource @emma-raffenne