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

fix import packages after core #8157 #499

Closed
wants to merge 1 commit into from

Conversation

sbcd90
Copy link
Collaborator

@sbcd90 sbcd90 commented Jul 25, 2023

Description

fix import packages after core #8157

Issues Resolved

#508

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 Jul 25, 2023

Codecov Report

Merging #499 (3038d27) into main (9a3213b) will not change coverage.
The diff coverage is n/a.

❗ Current head 3038d27 differs from pull request most recent head 2693948. Consider uploading reports for the commit 2693948 to get more accurate results

@@            Coverage Diff            @@
##               main     #499   +/-   ##
=========================================
  Coverage     28.20%   28.20%           
  Complexity      941      941           
=========================================
  Files           236      236           
  Lines          9864     9864           
  Branches       1118     1118           
=========================================
  Hits           2782     2782           
  Misses         6841     6841           
  Partials        241      241           
Files Changed Coverage Δ
...rch/securityanalytics/SecurityAnalyticsPlugin.java 2.56% <ø> (ø)
...rch/securityanalytics/action/AckAlertsRequest.java 75.00% <ø> (ø)
...ch/securityanalytics/action/AckAlertsResponse.java 77.27% <ø> (ø)
.../opensearch/securityanalytics/action/AlertDto.java 50.45% <ø> (ø)
...rityanalytics/action/CorrelatedFindingRequest.java 0.00% <ø> (ø)
...ityanalytics/action/CorrelatedFindingResponse.java 0.00% <ø> (ø)
...tyanalytics/action/CreateIndexMappingsRequest.java 46.93% <ø> (ø)
...analytics/action/DeleteCorrelationRuleRequest.java 0.00% <ø> (ø)
...ecurityanalytics/action/DeleteDetectorRequest.java 0.00% <ø> (ø)
...curityanalytics/action/DeleteDetectorResponse.java 0.00% <ø> (ø)
... and 84 more

Signed-off-by: Subhobrata Dey <sbcd90@gmail.com>
@sbcd90 sbcd90 closed this Aug 8, 2023
riysaxen-amzn pushed a commit to riysaxen-amzn/security-analytics that referenced this pull request Feb 20, 2024
* [BUG] Edit details Save button should be blue opensearch-project#499

Signed-off-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com>

* [BUG] Detector details has some bad text handling opensearch-project#496

Signed-off-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com>

* [BUG] Detector details has some bad text handling opensearch-project#496

Signed-off-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com>

* [BUG] Detector details has some bad text handling opensearch-project#496

Signed-off-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com>

---------

Signed-off-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com>
riysaxen-amzn pushed a commit to riysaxen-amzn/security-analytics that referenced this pull request Mar 25, 2024
…rch-project#499)

As per gradle [docs] add support to remove timestamps and package with same order which is required from
[reproducible] builds

[docs]: https://docs.gradle.org/current/userguide/working_with_files.html#sec:archives
[reproducible]: https://reproducible-builds.org/

Signed-off-by: Leonidas Spyropoulos <artafinde@archlinux.org>
(cherry picked from commit a27d5a214f2bcd9101e7602d715d17592ccd5e0f)
Signed-off-by: prudhvigodithi <pgodithi@amazon.com>

Co-authored-by: Leonidas Spyropoulos <artafinde@gmail.com>
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.

None yet

1 participant