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.9] [Custom log types] Support custom log types in detection rule creation and detector creation #682

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 5388596 from #676

…n and detector creation (#676)

* log types table; details page with basic editing

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* implemented create, delete of log type

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* detection rules; delete modals added

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated snapshots

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* addressed PR comments

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated cypress workflow

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* use log types from API

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated detector cypress test

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* using api provided log types in rule creation

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* addressed PR comments

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated cypress test

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated nav panel

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed log type labels; cypress tests

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 5388596)
@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Merging #682 (e7c1b91) into 2.9 (55ba874) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              2.9     #682      +/-   ##
==========================================
- Coverage   31.37%   31.35%   -0.03%     
==========================================
  Files         151      150       -1     
  Lines        4771     4759      -12     
  Branches      781      783       +2     
==========================================
- Hits         1497     1492       -5     
+ Misses       3096     3089       -7     
  Partials      178      178              

see 16 files with indirect coverage changes

@amsiglan amsiglan merged commit b8aa5d3 into 2.9 Aug 2, 2023
AWSHurneyt pushed a commit to AWSHurneyt/security-analytics-dashboards-plugin that referenced this pull request Oct 12, 2023
…n and detector creation (opensearch-project#676) (opensearch-project#682)

* log types table; details page with basic editing

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* implemented create, delete of log type

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* detection rules; delete modals added

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated snapshots

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* addressed PR comments

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated cypress workflow

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* use log types from API

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated detector cypress test

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* using api provided log types in rule creation

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* addressed PR comments

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated cypress test

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated nav panel

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed log type labels; cypress tests

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 5388596)

Co-authored-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant