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

[Backport] [2.x] [distribution/archives] [Linux] [x64] Provide the variant of the distributions bundled with JRE (#8195) #8978

Merged

Conversation

reta
Copy link
Collaborator

@reta reta commented Jul 28, 2023

Backport of #8195 to 2.x

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Jul 28, 2023

Codecov Report

Merging #8978 (a72c2cc) into 2.x (558b56d) will decrease coverage by 0.09%.
The diff coverage is 32.59%.

❗ Current head a72c2cc differs from pull request most recent head 3d0c539. Consider uploading reports for the commit 3d0c539 to get more accurate results

@@             Coverage Diff              @@
##                2.x    #8978      +/-   ##
============================================
- Coverage     70.82%   70.73%   -0.09%     
- Complexity    57375    57382       +7     
============================================
  Files          4749     4752       +3     
  Lines        271395   271439      +44     
  Branches      40055    40037      -18     
============================================
- Hits         192213   192006     -207     
- Misses        62638    62972     +334     
+ Partials      16544    16461      -83     
Files Changed Coverage Δ
.../java/org/opensearch/gradle/JdkDownloadPlugin.java 38.63% <0.00%> (+1.27%) ⬆️
...ldSrc/src/main/java/org/opensearch/gradle/Jre.java 0.00% <0.00%> (ø)
.../java/org/opensearch/gradle/JreDownloadPlugin.java 0.00% <0.00%> (ø)
...e/internal/InternalDistributionDownloadPlugin.java 0.00% <0.00%> (ø)
...a/org/opensearch/gradle/test/DistroTestPlugin.java 0.00% <0.00%> (ø)
.../main/java/org/opensearch/monitor/jvm/JvmInfo.java 74.46% <0.00%> (+7.67%) ⬆️
server/src/main/java/org/opensearch/node/Node.java 86.06% <0.00%> (ø)
.../java/org/opensearch/gradle/VersionProperties.java 52.00% <26.66%> (-9.12%) ⬇️
.../org/opensearch/gradle/OpenSearchDistribution.java 73.91% <66.66%> (ø)
...c/main/java/org/opensearch/gradle/JavaVariant.java 67.60% <67.60%> (ø)
... and 2 more

... and 545 files with indirect coverage changes

@reta reta force-pushed the backport/backport-8195-to-2.x branch from a3aeba9 to a72c2cc Compare August 1, 2023 14:30
@github-actions
Copy link
Contributor

github-actions bot commented Aug 1, 2023

Gradle Check (Jenkins) Run Completed with:

@reta
Copy link
Collaborator Author

reta commented Aug 1, 2023

@dblock mind approving the backport please? thank you.

…ributions bundled with JRE (opensearch-project#8195)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@reta reta force-pushed the backport/backport-8195-to-2.x branch from a72c2cc to 3d0c539 Compare August 2, 2023 13:56
@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      2 org.opensearch.search.SearchWeightedRoutingIT.testStrictWeightedRoutingWithCustomString
      1 org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testIndexCreateBlockIsRemovedWhenAnyNodesNotExceedHighWatermarkWithAutoReleaseEnabled

@Poojita-Raj
Copy link
Contributor

Poojita-Raj commented Aug 2, 2023

Gradle Check (Jenkins) Run Completed with:

* **RESULT:** UNSTABLE ❕

* **TEST FAILURES:**
      2 org.opensearch.search.SearchWeightedRoutingIT.testStrictWeightedRoutingWithCustomString
      1 org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testIndexCreateBlockIsRemovedWhenAnyNodesNotExceedHighWatermarkWithAutoReleaseEnabled

#6445
#8030

@andrross andrross merged commit 6ab46bd into opensearch-project:2.x Aug 3, 2023
6 checks passed
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.

3 participants