Skip to content

Fix systemd seccomp test#18196

Merged
peterzhuamazon merged 2 commits intoopensearch-project:mainfrom
RajatGupta02:fix-seccomp
May 5, 2025
Merged

Fix systemd seccomp test#18196
peterzhuamazon merged 2 commits intoopensearch-project:mainfrom
RajatGupta02:fix-seccomp

Conversation

@RajatGupta02
Copy link
Copy Markdown
Contributor

Description

Fix test that checks if Seccomp is enabled

Related Issues

Suite: Test class org.opensearch.systemdinteg.SystemdIntegTests
  2> May 01, 2025 9:36:29 PM org.apache.lucene.internal.vectorization.VectorizationProvider lookup
  2> WARNING: C2 compiler is disabled; Java vector incubator API can't be enabled
  2> java.lang.AssertionError: Seccomp should be enabled
        at __randomizedtesting.SeedInfo.seed([69E4FBFD7BE9AE21:3DC557472D5D715]:0)
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.assertTrue(Assert.java:42)
        at org.junit.Assert.assertFalse(Assert.java:65)
        at org.opensearch.systemdinteg.SystemdIntegTests.testSeccompEnabled(SystemdIntegTests.java:141)
  2> NOTE: reproduce with: gradlew test --tests SystemdIntegTests.testSeccompEnabled -Dtests.seed=69E4FBFD7BE9AE21 -Dtests.locale=vun-Latn-TZ -Dtests.timezone=Asia/Bishkek -Dtests.asserts=true -Dtests.file.encoding=UTF-8
  2> NOTE: test params are: codec=Asserting(Lucene101): {}, docValues:{}, maxPointsInLeafNode=267, maxMBSortInHeap=7.443377624226953, sim=Asserting(RandomSimilarity(queryNorm=false): {}), locale=vun-Latn-TZ, timezone=Asia/Bishkek
  2> NOTE: Linux 6.1.109-118.189.amzn2023.aarch64 aarch64/Eclipse Adoptium 21.0.7 (64-bit)/cpus=16,threads=1,free=435481744,total=536870912
  2> NOTE: All tests run in this JVM: [SystemdIntegTests]
7 tests completed, 1 failed

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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: Rajat Gupta <gptrajat@amazon.com>
@RajatGupta02 RajatGupta02 requested a review from a team as a code owner May 5, 2025 10:18
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 5, 2025

✅ Gradle check result for 021157d: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented May 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.59%. Comparing base (4c2d043) to head (113ae6c).
Report is 6 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #18196      +/-   ##
============================================
+ Coverage     72.54%   72.59%   +0.04%     
- Complexity    67234    67263      +29     
============================================
  Files          5476     5476              
  Lines        310444   310445       +1     
  Branches      45120    45120              
============================================
+ Hits         225211   225353     +142     
+ Misses        66834    66743      -91     
+ Partials      18399    18349      -50     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Rajat Gupta <gptrajat@amazon.com>
Copy link
Copy Markdown
Member

@peterzhuamazon peterzhuamazon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @RajatGupta02 ,
Running a local test with this code and passed:


> Task :qa:systemd-test:integTest
WARNING: Using incubator modules: jdk.incubator.vector

[Incubating] Problems report is available at: file:///home/ci-runner/OpenSearch/build/reports/problems/problems-report.html

BUILD SUCCESSFUL in 3m 9s
60 actionable tasks: 60 executed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 5, 2025

❕ Gradle check result for 113ae6c: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@peterzhuamazon peterzhuamazon merged commit aca411a into opensearch-project:main May 5, 2025
33 of 34 checks passed
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