Skip to content

fixing security integ test #3646

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

Merged
merged 1 commit into from
Mar 13, 2025
Merged

Conversation

dhrubo-os
Copy link
Collaborator

Description

[fixing security integ test]

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.

Signed-off-by: Dhrubo Saha <dhrubo@amazon.com>
@mingshl
Copy link
Collaborator

mingshl commented Mar 13, 2025

it passed!!!! verifying it fix [BUG] SecureMLRestIT.testPredictWithReadOnlyMLAccess Failed

./gradlew integTest --tests "org.opensearch.ml.rest.SecureMLRestIT.testTrainWithReadOnlyMLAccess"  -Dtests.rest.cluster=localhost:9200 -Dtests.cluster=localhost:9200 -Dtests.clustername="opensearch" -Dhttps=true -Duser=admin -Dpassword=myStrongPassword123!
Starting a Gradle Daemon (subsequent builds will be faster)
=======================================
OpenSearch Build Hamster says Hello!
  Gradle Version        : 8.11.1
  OS Info               : Mac OS X 15.3.1 (x86_64)
  JDK Version           : 21 (Amazon Corretto JDK)
  JAVA_HOME             : /Users/mingshl/Library/Java/JavaVirtualMachines/corretto-21.0.4/Contents/Home
  Random Testing Seed   : 8F2A30648986F9DB
  In FIPS 140 mode      : false
=======================================

> Task :opensearch-ml-plugin:compileTestJava
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

> Task :opensearch-ml-plugin:integTest

SecureMLRestIT STANDARD_ERROR
    Mar 13, 2025 10:11:32 AM org.apache.lucene.internal.vectorization.VectorizationProvider lookup
    WARNING: Java vector incubator module is not readable. For optimal vector performance, pass '--add-modules jdk.incubator.vector' to enable Vector API.

SecureMLRestIT > testTrainWithReadOnlyMLAccess STANDARD_OUT
    [2025-03-13T14:11:33,018][INFO ][o.o.m.r.SecureMLRestIT   ] [testTrainWithReadOnlyMLAccess] before test
    [2025-03-13T14:11:33,236][INFO ][o.o.m.r.SecureMLRestIT   ] [testTrainWithReadOnlyMLAccess] initializing REST clients against [https://localhost:9200]
    [2025-03-13T14:11:50,914][INFO ][o.o.m.r.SecureMLRestIT   ] [testTrainWithReadOnlyMLAccess] There are still tasks running after this test that might break subsequent tests [indices:data/write/bulk[s], indices:data/write/bulk[s][p], indices:data/write/index].
    [2025-03-13T14:11:50,976][INFO ][o.o.m.r.SecureMLRestIT   ] [testTrainWithReadOnlyMLAccess] after test
[Incubating] Problems report is available at: file:///Users/mingshl/Documents/GitHub/mlcommons-repo/ml-commons/build/reports/problems/problems-report.html

Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

For more on this, please refer to https://docs.gradle.org/8.11.1/userguide/command_line_interface.html#sec:command_line_warnings in the Gradle documentation.

BUILD SUCCESSFUL in 2m 7s ```

@dhrubo-os dhrubo-os merged commit f19614e into opensearch-project:main Mar 13, 2025
10 of 12 checks passed
@mingshl mingshl mentioned this pull request Apr 15, 2025
5 tasks
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 26, 2025
Signed-off-by: Dhrubo Saha <dhrubo@amazon.com>
(cherry picked from commit f19614e)
dhrubo-os added a commit that referenced this pull request Apr 26, 2025
Signed-off-by: Dhrubo Saha <dhrubo@amazon.com>
(cherry picked from commit f19614e)

Co-authored-by: Dhrubo Saha <dhrubo@amazon.com>
akolarkunnu pushed a commit to akolarkunnu/ml-commons that referenced this pull request Jun 6, 2025
Signed-off-by: Dhrubo Saha <dhrubo@amazon.com>
Signed-off-by: Abdul Muneer Kolarkunnu <muneer.kolarkunnu@netapp.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants