[Backport main] Correlation engine ux #537
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![opensearch-trigger-bot[bot]](https://avatars.githubusercontent.com/u/80134844?s=60&v=4){kind=small}
* wip Signed-off-by: Amardeepsingh Siglani <[email protected]> * completed rules page; added placeholder for findings flyout tabs Signed-off-by: Amardeepsingh Siglani <[email protected]> * reverted change to show empty table Signed-off-by: Amardeepsingh Siglani <[email protected]> * updated rule type Signed-off-by: Amardeepsingh Siglani <[email protected]> * updated create correlation rule page Signed-off-by: Amardeepsingh Siglani <[email protected]> * correlation page wip Signed-off-by: Amardeepsingh Siglani <[email protected]> * Correlation store and API services Signed-off-by: Jovan Cvetkovic <[email protected]> * Correlation store and API services Signed-off-by: Jovan Cvetkovic <[email protected]> * Correlation store and API services Signed-off-by: Jovan Cvetkovic <[email protected]> * Correlation store and API services Signed-off-by: Jovan Cvetkovic <[email protected]> * Correlation store and API services Signed-off-by: Jovan Cvetkovic <[email protected]> * updated correlations page Signed-off-by: Amardeepsingh Siglani <[email protected]> * updated store interfaces Signed-off-by: Amardeepsingh Siglani <[email protected]> * fixed correlation store Signed-off-by: Amardeepsingh Siglani <[email protected]> * added api for all correlations Signed-off-by: Amardeepsingh Siglani <[email protected]> * added filter support for correlations Signed-off-by: Amardeepsingh Siglani <[email protected]> * using apis; added validation Signed-off-by: Amardeepsingh Siglani <[email protected]> * Implemented logic to populate correlations tables data. Signed-off-by: AWSHurneyt <[email protected]> * restricted to AND Signed-off-by: Amardeepsingh Siglani <[email protected]> * log type clearable Signed-off-by: Amardeepsingh Siglani <[email protected]> * code cleanup Signed-off-by: Jovan Cvetkovic <[email protected]> * fixed API issue Signed-off-by: Amardeepsingh Siglani <[email protected]> * updated snapshots Signed-off-by: Amardeepsingh Siglani <[email protected]> * ux improvements Signed-off-by: Amardeepsingh Siglani <[email protected]> * code cleanup Signed-off-by: Jovan Cvetkovic <[email protected]> * removed redundant code Signed-off-by: Amardeepsingh Siglani <[email protected]> * removed unwanted param Signed-off-by: Amardeepsingh Siglani <[email protected]> * connected refresh Signed-off-by: Amardeepsingh Siglani <[email protected]> --------- Signed-off-by: Amardeepsingh Siglani <[email protected]> Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> (cherry picked from commit f565a92)
amsiglan
approved these changes
Apr 19, 2023
* removed duplicate import * updated snapshots --------- Signed-off-by: Amardeepsingh Siglani <[email protected]>
* pinned babel traverse and core * removed cyclic dependency * added types for vis * refactored code --------- Signed-off-by: Amardeepsingh Siglani <[email protected]>
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #537 +/- ##
==========================================
- Coverage 35.59% 32.65% -2.95%
==========================================
Files 121 135 +14
Lines 3444 4012 +568
Branches 602 694 +92
==========================================
+ Hits 1226 1310 +84
- Misses 2081 2562 +481
- Partials 137 140 +3 see 34 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
AWSHurneyt
approved these changes
Apr 19, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport f565a92 from #524