Skip to content

[Disposable] Security tests verification#1663

Closed
jackiehanyang wants to merge 21 commits intoopensearch-project:mainfrom
jackiehanyang:securityTests
Closed

[Disposable] Security tests verification#1663
jackiehanyang wants to merge 21 commits intoopensearch-project:mainfrom
jackiehanyang:securityTests

Conversation

@jackiehanyang
Copy link
Copy Markdown
Collaborator

@jackiehanyang jackiehanyang commented Feb 3, 2026

Description

[Describe what this change achieves]

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

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.

@jackiehanyang
initial commit
64021b5 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
Merge remote-tracking branch 'origin/main' into tmp/merge-main-into-i…
45e795f 
…nsights
@jackiehanyang
Introduce Insights API
b14da81 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
spotless apply
ed68a6d 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
add change log
200d461 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
fix
671e103 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
make timestamp more user friendly when reading
77fcda9 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
fix forbidden api violation by avoiding default-locale usage
3d50179 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
add more tests
49018e0 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
use stashed context when starting insights job
816d69b 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
add one time insights job run
1356920 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
do immediate one time run job right after starting insights job
b731aef 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
address comments
1ebe2a8 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
Merge remote-tracking branch 'origin/main' into insights
be01bc9
@jackiehanyang
replace ml-commons metrics correlation algorithm with temporal overla…
7525e0d 
…p algorithm

Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
address comments
85db8bb 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
add more tests
bb524d1 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
address comments
83f21d7 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
Merge remote-tracking branch 'origin/main' into insights
92914b5
@jackiehanyang
fix security test
fd0f1d9 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@jackiehanyang
security test verification
a8a2762 
Signed-off-by: Jackie <jkhanjob@gmail.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 4, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 74.10636% with 297 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.85%. Comparing base (5fe42cc) to head (a8a2762).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
...n/java/org/opensearch/ad/InsightsJobProcessor.java 75.90% 87 Missing and 33 partials ⚠️
...arch/ad/rest/handler/InsightsJobActionHandler.java 49.31% 102 Missing and 9 partials ⚠️
...n/java/org/opensearch/ad/ml/InsightsGenerator.java 82.92% 6 Missing and 15 partials ⚠️
...opensearch/timeseries/indices/IndexManagement.java 61.76% 10 Missing and 3 partials ⚠️
...a/org/opensearch/ad/indices/ADIndexManagement.java 77.08% 9 Missing and 2 partials ⚠️
.../org/opensearch/ad/rest/RestInsightsJobAction.java 77.77% 3 Missing and 5 partials ⚠️
...earch/timeseries/transport/InsightsJobRequest.java 78.12% 4 Missing and 3 partials ⚠️
...earch/ad/transport/InsightsJobTransportAction.java 80.95% 3 Missing and 1 partial ⚠️
...g/opensearch/ad/transport/InsightsJobResponse.java 97.89% 1 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1663      +/-   ##
============================================
- Coverage     81.33%   80.85%   -0.49%     
- Complexity     6241     6410     +169     
============================================
  Files           544      554      +10     
  Lines         25323    26441    +1118     
  Branches       2621     2716      +95     
============================================
+ Hits          20596    21378     +782     
- Misses         3415     3674     +259     
- Partials       1312     1389      +77
Flag Coverage Δ
plugin 80.85% <74.10%> (-0.49%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
.../java/org/opensearch/ad/constant/ADCommonName.java 0.00% <ø> (ø)
...c/main/java/org/opensearch/ad/indices/ADIndex.java 100.00% <100.00%> (ø)
...g/opensearch/ad/model/AnomalyCorrelationInput.java 100.00% <100.00%> (ø)
...java/org/opensearch/ad/model/DetectorMetadata.java 100.00% <100.00%> (ø)
...pensearch/ad/settings/AnomalyDetectorSettings.java 100.00% <100.00%> (ø)
...org/opensearch/ad/transport/InsightsJobAction.java 100.00% <100.00%> (ø)
...main/java/org/opensearch/timeseries/JobRunner.java 81.81% <100.00%> (+13.39%) ⬆️
...ensearch/timeseries/TimeSeriesAnalyticsPlugin.java 99.11% <100.00%> (+0.03%) ⬆️
...timeseries/rest/handler/IndexJobActionHandler.java 89.27% <ø> (ø)
...g/opensearch/timeseries/util/RestHandlerUtils.java 67.46% <ø> (ø)
... and 9 more

... and 23 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmazanec15 jmazanec15 Awaiting requested review from jmazanec15 jmazanec15 is a code owner

@jngz-es jngz-es Awaiting requested review from jngz-es jngz-es is a code owner

@kaituo kaituo Awaiting requested review from kaituo kaituo is a code owner

@saratvemulapalli saratvemulapalli Awaiting requested review from saratvemulapalli saratvemulapalli is a code owner

@ohltyler ohltyler Awaiting requested review from ohltyler ohltyler is a code owner

@vamshin vamshin Awaiting requested review from vamshin vamshin is a code owner

@VijayanB VijayanB Awaiting requested review from VijayanB VijayanB is a code owner

@ylwu-amzn ylwu-amzn Awaiting requested review from ylwu-amzn ylwu-amzn is a code owner

@amitgalitz amitgalitz Awaiting requested review from amitgalitz amitgalitz is a code owner

@sean-zheng-amazon sean-zheng-amazon Awaiting requested review from sean-zheng-amazon sean-zheng-amazon is a code owner

@dbwiddis dbwiddis Awaiting requested review from dbwiddis dbwiddis is a code owner

@owaiskazi19 owaiskazi19 Awaiting requested review from owaiskazi19 owaiskazi19 is a code owner

@joshpalis joshpalis Awaiting requested review from joshpalis joshpalis is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jackiehanyang