Skip to content

Gradle to 7.6.1 and Gradle test-retry to 1.5.2, updating dockerbuild PA Artifact pull logic#358

Merged
khushbr merged 1 commit intoopensearch-project:2.xfrom
khushbr:2.x
May 12, 2023
Merged

Gradle to 7.6.1 and Gradle test-retry to 1.5.2, updating dockerbuild PA Artifact pull logic#358
khushbr merged 1 commit intoopensearch-project:2.xfrom
khushbr:2.x

Conversation

@khushbr
Copy link
Copy Markdown
Collaborator

@khushbr khushbr commented May 12, 2023

Is your feature request related to a problem? Please provide an existing Issue # , or describe.
Backporting: 1. #357 and 2. #354

Addressing the build failure due to gradle version upgrade.

* What went wrong:
A problem was found with the configuration of task ':clonePaGitRepo' (type 'GitClone').
  - Type 'org.ajoberstar.gradle.git.tasks.GitClone' property 'authentication' is missing an input or output annotation.
    
    Reason: A property without annotation isn't considered during up-to-date checking.

Describe the solution you are proposing
Removing the redundant git clone and replacing with pulling Snapshots from Maven Snapshot repo(https://aws.oss.sonatype.org/content/repositories/snapshots). Refer: https://opensearch.org/blog/opensearch-plugin-zips-now-in-maven-repo/

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.

…rBuild to use Maven repo for PA Artifact instead of git clone plugin

Signed-off-by: Khushboo Rajput <khushbr@amazon.com>
@codecov
Copy link
Copy Markdown

codecov bot commented May 12, 2023

Codecov Report

Merging #358 (f9e2c66) into 2.x (b7b9bc2) will decrease coverage by 0.01%.
The diff coverage is n/a.

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

@@             Coverage Diff              @@
##                2.x     #358      +/-   ##
============================================
- Coverage     72.76%   72.75%   -0.01%     
+ Complexity     3045     3043       -2     
============================================
  Files           381      381              
  Lines         19013    19013              
  Branches       1472     1472              
============================================
- Hits          13834    13833       -1     
+ Misses         4584     4583       -1     
- Partials        595      597       +2     

see 3 files with indirect coverage changes

@khushbr khushbr merged commit 3408269 into opensearch-project:2.x May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet