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

[Backport 2.x] Initial implementation of context aware alert analysis #249

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 32888dd from #215.

* support context aware alert analysis

Signed-off-by: Hailong Cui <[email protected]>

* Render GeneratePopover IncontextInsight component as a button to generate summary

Signed-off-by: Songkan Tang <[email protected]>

* Remove hardcoded assistant role from the parameter payload

Signed-off-by: Songkan Tang <[email protected]>

* Make GeneratePopoverBody as independent component

Signed-off-by: Songkan Tang <[email protected]>

* Update change log

Signed-off-by: Songkan Tang <[email protected]>

* Add independent GeneratePopoverBody ut and reorgnize constants

Signed-off-by: Songkan Tang <[email protected]>

* Simplify states of loading to get summary process

Signed-off-by: Songkan Tang <[email protected]>

* Make IncontextInsight not shareable and each component has its own IncontextInsight

Signed-off-by: Songkan Tang <[email protected]>

* Enable context aware alert only if feature flag is enabled

Signed-off-by: Songkan Tang <[email protected]>

---------

Signed-off-by: Hailong Cui <[email protected]>
Signed-off-by: Songkan Tang <[email protected]>
Co-authored-by: Hailong Cui <[email protected]>
(cherry picked from commit 32888dd)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
@gaobinlong
Copy link
Collaborator

@songkant-aws please help to resolve the conflicts in this PR, thanks!

@songkant-aws songkant-aws mentioned this pull request Sep 5, 2024
4 tasks
@songkant-aws
Copy link
Contributor

Post a new PR to fix conflicts and it's merged now: #271

@gaobinlong gaobinlong closed this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants