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

Updates demo certs used in integ tests #544

Closed
wants to merge 2 commits into from

Conversation

jowg-amazon
Copy link
Collaborator

Description

Fixes build breaks when running integ tests for 2.10.0 tar distribution

Issues Resolved

#525

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@codecov
Copy link

codecov bot commented Sep 7, 2023

Codecov Report

Merging #544 (ba5d5c0) into main (1003936) will increase coverage by 0.01%.
The diff coverage is n/a.

❗ Current head ba5d5c0 differs from pull request most recent head 20ef1f7. Consider uploading reports for the commit 20ef1f7 to get more accurate results

@@             Coverage Diff              @@
##               main     #544      +/-   ##
============================================
+ Coverage     25.12%   25.14%   +0.01%     
- Complexity      942      943       +1     
============================================
  Files           255      255              
  Lines         11064    11060       -4     
  Branches       1231     1231              
============================================
+ Hits           2780     2781       +1     
+ Misses         8032     8028       -4     
+ Partials        252      251       -1     

see 2 files with indirect coverage changes

📢 Have feedback on the report? Share it here.

@sbcd90 sbcd90 closed this Sep 7, 2023
@sbcd90
Copy link
Collaborator

sbcd90 commented Sep 7, 2023

closing in favor of #543

riysaxen-amzn pushed a commit to riysaxen-amzn/security-analytics that referenced this pull request Feb 20, 2024
* Correlation engine ux (opensearch-project#524)

* 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 f565a923c6bfec1b8a69128d21e5d823a50cb142)

* Removed duplicate import for EuiSpacer (opensearch-project#538) (opensearch-project#540)

* removed duplicate import



* updated snapshots



---------

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

* Pinned babel traverse and core (opensearch-project#539) (opensearch-project#544)

* pinned babel traverse and core



* removed cyclic dependency



* added types for vis



* refactored code



---------

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

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[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.

2 participants