Run CI/CD on Java 8, 11, 14 and 17. #121

dblock · 2022-02-28T19:49:07Z

Description

Uses the standard CI matrix to build and test on JDK 8, 11, and 14.
Since Gradle 6 doesn't support JDK 17, build on 11, and test on 17. Not sure whether there's a better way to do this?

Run ./gradlew build --build-cache
  ./gradlew build --build-cache
  shell: /usr/bin/bash -e {0}
  env:
    JAVA_HOME: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/11.0.14-101/x64

Run ./gradlew test --build-cache
  ./gradlew test --build-cache
  shell: /usr/bin/bash -e {0}
  env:
    JAVA_HOME: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/17.0.2-8/x64

When we upgrade to Gradle 7 we can ran the 11, 14, 17 matrix and remove ci.17.yml.

Issues Resolved

Closes #100.

Check List

New functionality includes testing.
- All tests pass
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.

Signed-off-by: dblock <[email protected]>

dblock · 2022-02-28T21:35:47Z

src/test/java/org/opensearch/commons/InjectSecurityTest.java

@@ -114,7 +114,11 @@ public void testInjectProperty() {
            assertTrue(helper.injectProperty("property1", true));
            assertTrue(helper.injectProperty("property2", "some value"));
            assertTrue(helper.injectProperty("property3", ""));
-            assertTrue(helper.injectProperty("property4", Map.of("key", "value")));
+            assertTrue(helper.injectProperty("property4", new HashMap<String, String>() {


Map.of doesn't exist in Java 8.

saratvemulapalli · 2022-03-01T00:50:31Z

.github/workflows/ci-17.yml

+      - name: Setup Java 11
+        uses: actions/setup-java@v2
+        with:
+          distribution: 'temurin'
+          java-version: 11
+


Do we want to test both Java 11 and 17 here?
If so, we should rename the file

We build with 11 but test with 17 here, so really this is a 17 test.

saratvemulapalli · 2022-03-01T00:54:12Z

.github/workflows/ci-17.yml

+    name: Build and Test (17)
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2


Also instead of duplicating the test information for each one of them, can we use github action matrix?

Something like

jobs: test: runs-on: ${{ matrix.os }} strategy: matrix: os: [ubuntu-18.04] java: [8, 11, 14, 17] fail-fast: false name: Test common-utils with JDK ${{ matrix.java }}, ${{ matrix.os }}

Ignore me :)
Just saw the description and it makes sense.

saratvemulapalli · 2022-03-01T00:57:22Z

.github/workflows/ci-17.yml

+      - name: Setup Java 17
+        uses: actions/setup-java@v2
+        with:
+          distribution: 'temurin'


May be a very dumb question. I dont understand Java distributions yet, but can we use with what with support on OpenSearch for consistency?
We use adoptium.

Ref: opensearch-project/OpenSearch#1358

Temurin is a distribution of adoptium - https://adoptium.net/

Signed-off-by: dblock <[email protected]>

saratvemulapalli

Thanks @dblock !

VachaShah

LGTM! Thank you @dblock!

Following example from: opensearch-project/common-utils#121 Signed-off-by: Peter Nied <[email protected]>

Fix copyright notice and add DCO check workflow (opensearch-project#94) Signed-off-by: Ashish Agrawal <[email protected]> Update build.sh script to include optional platform param. (opensearch-project#95) Signed-off-by: Marc Handalian <[email protected]> Add codeowners support for repo (opensearch-project#96) Signed-off-by: Ryan Bogan <[email protected]> Bump version to 1.3 (opensearch-project#99) Signed-off-by: Ashish Agrawal <[email protected]> Auto-increment version on new release tags. (opensearch-project#106) Signed-off-by: dblock <[email protected]> Remove jcenter repository (opensearch-project#115) Using Github App token to trigger CI for version increment PRs (opensearch-project#116) Signed-off-by: Vacha Shah <[email protected]> Fixes copyright headers (opensearch-project#117) Signed-off-by: Drew Baugher <[email protected]> Remove jcenter repository missed on first pass (opensearch-project#118) Signed-off-by: Peter Nied <[email protected]> Run CI/CD on Java 8, 11, 14 and 17. (opensearch-project#121) * Run CI/CD on Java 8, 11, 14 and 17. Signed-off-by: dblock <[email protected]> * Add JDK 17. Signed-off-by: dblock <[email protected]> Add .whitesource configuration file (opensearch-project#109) Co-authored-by: whitesource-for-garden.eu.org[bot] <50673670+whitesource-for-garden.eu.org[bot]@users.noreply.github.com> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> Co-authored-by: Ashish Agrawal <[email protected]> Co-authored-by: Marc Handalian <[email protected]> Co-authored-by: Ryan Bogan <[email protected]> Co-authored-by: Vacha Shah <[email protected]> Co-authored-by: Drew Baugher <[email protected]> Signed-off-by: Saurabh Singh <[email protected]>

* Run CI/CD on Java 8, 11, 14 and 17. Signed-off-by: dblock <[email protected]> * Add JDK 17. Signed-off-by: dblock <[email protected]>

* Run CI/CD on Java 8, 11, 14 and 17. Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Add JDK 17. Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]>

* Fix copyright notice and add DCO check workflow (#94) Signed-off-by: Ashish Agrawal <[email protected]> * Update build.sh script to include optional platform param. (#95) Signed-off-by: Marc Handalian <[email protected]> * Add codeowners support for repo (#96) Signed-off-by: Ryan Bogan <[email protected]> * Bump version to 1.3 (#99) Signed-off-by: Ashish Agrawal <[email protected]> * Auto-increment version on new release tags. (#106) Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Remove jcenter repository (#115) Signed-off-by: Peter Nied <[email protected]> * Using Github App token to trigger CI for version increment PRs (#116) Signed-off-by: Vacha Shah <[email protected]> * Fixes copyright headers (#117) Signed-off-by: Drew Baugher <[email protected]> * Remove jcenter repository missed on first pass (#118) Signed-off-by: Peter Nied <[email protected]> * Run CI/CD on Java 8, 11, 14 and 17. (#121) * Run CI/CD on Java 8, 11, 14 and 17. Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Add JDK 17. Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Add .whitesource configuration file (#109) Co-authored-by: whitesource-for-garden.eu.org[bot] <50673670+whitesource-for-garden.eu.org[bot]@users.noreply.github.com> Co-authored-by: Ashish Agrawal <[email protected]> Co-authored-by: Marc Handalian <[email protected]> Co-authored-by: Ryan Bogan <[email protected]> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> Co-authored-by: Peter Nied <[email protected]> Co-authored-by: Vacha Shah <[email protected]> Co-authored-by: Drew Baugher <[email protected]> Co-authored-by: Peter Nied <[email protected]> Co-authored-by: whitesource-for-garden.eu.org[bot] <50673670+whitesource-for-garden.eu.org[bot]@users.noreply.github.com>

* Run CI/CD on Java 8, 11, 14 and 17. Signed-off-by: dblock <[email protected]> * Add JDK 17. Signed-off-by: dblock <[email protected]> Signed-off-by: Zelin Hao <[email protected]>

* Run CI/CD on Java 8, 11, 14 and 17. Signed-off-by: dblock <[email protected]> * Add JDK 17. Signed-off-by: dblock <[email protected]>

* Fix copyright notice and add DCO check workflow (opensearch-project#94) Signed-off-by: Ashish Agrawal <[email protected]> * Update build.sh script to include optional platform param. (opensearch-project#95) Signed-off-by: Marc Handalian <[email protected]> * Add codeowners support for repo (opensearch-project#96) Signed-off-by: Ryan Bogan <[email protected]> * Bump version to 1.3 (opensearch-project#99) Signed-off-by: Ashish Agrawal <[email protected]> * Auto-increment version on new release tags. (opensearch-project#106) Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Remove jcenter repository (opensearch-project#115) Signed-off-by: Peter Nied <[email protected]> * Using Github App token to trigger CI for version increment PRs (opensearch-project#116) Signed-off-by: Vacha Shah <[email protected]> * Fixes copyright headers (opensearch-project#117) Signed-off-by: Drew Baugher <[email protected]> * Remove jcenter repository missed on first pass (opensearch-project#118) Signed-off-by: Peter Nied <[email protected]> * Run CI/CD on Java 8, 11, 14 and 17. (opensearch-project#121) * Run CI/CD on Java 8, 11, 14 and 17. Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Add JDK 17. Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Add .whitesource configuration file (opensearch-project#109) Co-authored-by: whitesource-for-garden.eu.org[bot] <50673670+whitesource-for-garden.eu.org[bot]@users.noreply.github.com> Co-authored-by: Ashish Agrawal <[email protected]> Co-authored-by: Marc Handalian <[email protected]> Co-authored-by: Ryan Bogan <[email protected]> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> Co-authored-by: Peter Nied <[email protected]> Co-authored-by: Vacha Shah <[email protected]> Co-authored-by: Drew Baugher <[email protected]> Co-authored-by: Peter Nied <[email protected]> Co-authored-by: whitesource-for-garden.eu.org[bot] <50673670+whitesource-for-garden.eu.org[bot]@users.noreply.github.com> Signed-off-by: AWSHurneyt <[email protected]>

…classic:1.2.13 to address CVE. #602 (#603) * Merge changes in the main branch to the 1.x branch. (#42) * Update Release Notes for GA (#36) * Update Release Notes for GA * Update Release Notes for GA include RC1 Changes as well. Signed-off-by: Aditya Jindal <[email protected]> * add method type in CustomWebhook data model (#39) Signed-off-by: Zhongnan Su <[email protected]> * Fix class loader issue for notifications response (#40) * Fix class loader issue for notifications Signed-off-by: Joshua Li <[email protected]> * Fix formatting Signed-off-by: Joshua Li <[email protected]> * Refactor creation of action listener object Signed-off-by: Joshua Li <[email protected]> * Fix indentation Signed-off-by: Joshua Li <[email protected]> * Remove unused suppresses Signed-off-by: Joshua Li <[email protected]> * Add UT for notification API Signed-off-by: Chen Dai <[email protected]> * Add UT for notification API Signed-off-by: Chen Dai <[email protected]> * Add UT for send notification API Signed-off-by: Chen Dai <[email protected]> * Fix Github workflow failure Signed-off-by: Chen Dai <[email protected]> * Fix Github workflow failure Signed-off-by: Chen Dai <[email protected]> * Refactor UT code Signed-off-by: Chen Dai <[email protected]> Co-authored-by: Joshua Li <[email protected]> Co-authored-by: Zhongnan Su <[email protected]> Co-authored-by: Chen Dai <[email protected]> Co-authored-by: Joshua Li <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * [1.x] Bumping common-utils to build with OpenSearch(1.x) 1.1.0 (#52) * Bumping common-utils to build with OpenSearch(main) 1.1.0 (#48) Signed-off-by: Sarat Vemulapalli <[email protected]> * Updating 1.x to work with OpenSearch 1.x Signed-off-by: Sarat Vemulapalli <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Backport 1.1 changes to 1.x (#72) * Add Commits related to Snapshot build of Common Utils on 1.1 (#67) * Using 1.1 snapshot version for OpenSearch (#57) Signed-off-by: Vacha <[email protected]> * Build snapshot build by default with the same version as OpenSearch. (#58) Signed-off-by: dblock <[email protected]> * Update build.gradle to reflect 1.1.0.0 version Co-authored-by: Vacha <[email protected]> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> * Build snapshot build by default with the same version as OpenSearch. (#58) (#69) Signed-off-by: dblock <[email protected]> * Adding an utility method that allows consumers to set custom thread context property in InjectSecurity class (#47) (#70) Signed-off-by: Ravi Thaluru <[email protected]> Co-authored-by: Ravi <[email protected]> * Add release notes for version 1.1.0.0 * Add release notes for version 1.1.0.0 Co-authored-by: Aditya Jindal <[email protected]> Co-authored-by: Vacha <[email protected]> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> Co-authored-by: Bowen Lan <[email protected]> Co-authored-by: Ravi <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Backport main changes to 1.x (#93) * Add themed logo to README (#41) Signed-off-by: Miki <[email protected]> * Updates common-utils version to 1.2 (#77) * Updates common-utils version to 1.2 and Uses Maven for 1.2 dependencies Signed-off-by: Clay Downs <[email protected]> * Publish .md5 and .sha1 signatures. (#79) (#80) * Publish .md5 and .sha1 signatures. Signed-off-by: dblock <[email protected]> * Use OpenSearch 1.1. Signed-off-by: dblock <[email protected]> * Publish source and javadoc checksums. (#81) Signed-off-by: dblock <[email protected]> * Update copyright notice (#90) Signed-off-by: Mohammad Qureshi <[email protected]> * Update maven publication to include cksums. (#91) This change adds a local staging repo task that will include cksums. It will also update build.sh to use this new task and copy the contents of the staging repo to the output directory. The maven publish plugin will not include these cksums when publishing to maven local but will when published to a separate folder. Signed-off-by: Marc Handalian <[email protected]> * Add release notes for version 1.2.0.0 (#92) * Add release notes for version 1.2.0.0 Signed-off-by: Ashish Agrawal <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Co-authored-by: Marc Handalian <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Backport main changes to 1.x (#126) * Fix copyright notice and add DCO check workflow (#94) Signed-off-by: Ashish Agrawal <[email protected]> * Update build.sh script to include optional platform param. (#95) Signed-off-by: Marc Handalian <[email protected]> * Add codeowners support for repo (#96) Signed-off-by: Ryan Bogan <[email protected]> * Bump version to 1.3 (#99) Signed-off-by: Ashish Agrawal <[email protected]> * Auto-increment version on new release tags. (#106) Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Remove jcenter repository (#115) Signed-off-by: Peter Nied <[email protected]> * Using Github App token to trigger CI for version increment PRs (#116) Signed-off-by: Vacha Shah <[email protected]> * Fixes copyright headers (#117) Signed-off-by: Drew Baugher <[email protected]> * Remove jcenter repository missed on first pass (#118) Signed-off-by: Peter Nied <[email protected]> * Run CI/CD on Java 8, 11, 14 and 17. (#121) * Run CI/CD on Java 8, 11, 14 and 17. Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Add JDK 17. Signed-off-by: Daniel Doubrovkine (dB.) <[email protected]> * Add .whitesource configuration file (#109) Co-authored-by: whitesource-for-garden.eu.org[bot] <50673670+whitesource-for-garden.eu.org[bot]@users.noreply.github.com> Co-authored-by: Ashish Agrawal <[email protected]> Co-authored-by: Marc Handalian <[email protected]> Co-authored-by: Ryan Bogan <[email protected]> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> Co-authored-by: Peter Nied <[email protected]> Co-authored-by: Vacha Shah <[email protected]> Co-authored-by: Drew Baugher <[email protected]> Co-authored-by: Peter Nied <[email protected]> Co-authored-by: whitesource-for-garden.eu.org[bot] <50673670+whitesource-for-garden.eu.org[bot]@users.noreply.github.com> Signed-off-by: AWSHurneyt <[email protected]> * Add release notes for version 1.3.0.0 (#134) Signed-off-by: Saurabh Singh <[email protected]> Co-authored-by: Saurabh Singh <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Bump version to 1.3.1 (#145) Signed-off-by: Sayali Gaikawad <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Incremented version to 1.3.2 (#148) Signed-off-by: Sayali Gaikawad <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Adding signoff option for version workflow PR (#143) (#150) Signed-off-by: Vacha Shah <[email protected]> (cherry picked from commit 6e78f69) Co-authored-by: Vacha Shah <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Incremented version to 1.3.3. (#180) Signed-off-by: dblock <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Incremented version to 1.3.4. (#198) Signed-off-by: Zelin Hao <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Staging for version increment automation (#200) (#208) * Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation: task rename updateVersion Signed-off-by: pgodithi <[email protected]> (cherry picked from commit 366bf16) Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: pgodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]> Co-authored-by: Prudhvi Godithi <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * release 1.3.5 (#219) Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Increment version to 1.3.6-SNAPSHOT (#243) Signed-off-by: opensearch-ci-bot <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Change TrustStoreTest to use File.separator to support Windows path (#258) (#260) Signed-off-by: Peter Zhu <[email protected]> Signed-off-by: Peter Zhu <[email protected]> (cherry picked from commit 7dcb3a0) Co-authored-by: Peter Zhu <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * disable detekt so that snakeyaml <= 1.31 is not used (#266) * disable detekt so that snakeyaml <= 1.31 is not used Signed-off-by: AWSHurneyt <[email protected]> * add release-notes for 1.3.6 (#267) Signed-off-by: Subhobrata Dey <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * [AUTO] Increment version to 1.3.7-SNAPSHOT (#276) * Increment version to 1.3.7-SNAPSHOT Signed-off-by: opensearch-ci-bot <[email protected]> * empty commit trigger Signed-off-by: Peter Zhu <[email protected]> Signed-off-by: opensearch-ci-bot <[email protected]> Signed-off-by: Peter Zhu <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Co-authored-by: Peter Zhu <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Adding CI workflow for Windows OS (#333) Signed-off-by: Subhobrata Dey <[email protected]> Signed-off-by: Subhobrata Dey <[email protected]> Co-authored-by: Surya Sashank Nistala <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Increment version to 1.3.8-SNAPSHOT (#338) Signed-off-by: opensearch-ci-bot <[email protected]> Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Add auto-release workflow (#376) (#377) Signed-off-by: Craig Perkins <[email protected]> (cherry picked from commit 89b7457) Co-authored-by: Craig Perkins <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Increment version to 1.3.9-SNAPSHOT (#355) Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Incremented version to 1.3.10 (#388) Signed-off-by: GitHub <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Increment version to 1.3.11-SNAPSHOT (#453) Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Increment version to 1.3.12-SNAPSHOT (#471) Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Increment version to 1.3.13-SNAPSHOT (#504) Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Increment version to 1.3.14-SNAPSHOT (#540) Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * [AUTO] Increment version to 1.3.15-SNAPSHOT (#575) * Increment version to 1.3.15-SNAPSHOT Signed-off-by: opensearch-ci-bot <[email protected]> * Empty-Commit Signed-off-by: Ashish Agrawal <[email protected]> * Remove jdk 8 CI test Signed-off-by: Ashish Agrawal <[email protected]> --------- Signed-off-by: opensearch-ci-bot <[email protected]> Signed-off-by: Ashish Agrawal <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Co-authored-by: Ashish Agrawal <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Forced ktlint to use logback-core:1.2.13, and logback-classic:1.2.13 to address CVE. (#602) Signed-off-by: AWSHurneyt <[email protected]> --------- Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: Sarat Vemulapalli <[email protected]> Signed-off-by: Sayali Gaikawad <[email protected]> Signed-off-by: dblock <[email protected]> Signed-off-by: Zelin Hao <[email protected]> Signed-off-by: pgodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: opensearch-ci-bot <[email protected]> Signed-off-by: Subhobrata Dey <[email protected]> Signed-off-by: Peter Zhu <[email protected]> Signed-off-by: GitHub <[email protected]> Signed-off-by: Ashish Agrawal <[email protected]> Co-authored-by: Aditya Jindal <[email protected]> Co-authored-by: Zhongnan Su <[email protected]> Co-authored-by: Chen Dai <[email protected]> Co-authored-by: Joshua Li <[email protected]> Co-authored-by: Sarat Vemulapalli <[email protected]> Co-authored-by: Sriram <[email protected]> Co-authored-by: Vacha <[email protected]> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> Co-authored-by: Bowen Lan <[email protected]> Co-authored-by: Ravi <[email protected]> Co-authored-by: Ashish Agrawal <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Co-authored-by: Marc Handalian <[email protected]> Co-authored-by: Saurabh Singh <[email protected]> Co-authored-by: Ryan Bogan <[email protected]> Co-authored-by: Peter Nied <[email protected]> Co-authored-by: Drew Baugher <[email protected]> Co-authored-by: Peter Nied <[email protected]> Co-authored-by: whitesource-for-garden.eu.org[bot] <50673670+whitesource-for-garden.eu.org[bot]@users.noreply.github.com> Co-authored-by: Saurabh Singh <[email protected]> Co-authored-by: Sayali Gaikawad <[email protected]> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: opensearch-ci-bot <[email protected]> Co-authored-by: Daniel (dB.) Doubrovkine <[email protected]> Co-authored-by: Zelin Hao <[email protected]> Co-authored-by: Prudhvi Godithi <[email protected]> Co-authored-by: Peter Zhu <[email protected]> Co-authored-by: Subhobrata Dey <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Co-authored-by: Surya Sashank Nistala <[email protected]> Co-authored-by: Craig Perkins <[email protected]>

dblock requested a review from a team February 28, 2022 19:49

dblock marked this pull request as draft February 28, 2022 19:49

dblock force-pushed the java-versions branch 7 times, most recently from 63aa43a to 94be034 Compare February 28, 2022 20:56

Run CI/CD on Java 8, 11, 14 and 17.

8023141

Signed-off-by: dblock <[email protected]>

dblock force-pushed the java-versions branch 3 times, most recently from ac62eb6 to 8467110 Compare February 28, 2022 21:25

dblock changed the title ~~WIP: Run CI/CD on Java 8, 11, 14 and 17.~~ Run CI/CD on Java 8, 11, 14 and 17. Feb 28, 2022

dblock force-pushed the java-versions branch 2 times, most recently from 7b8d2b0 to 9743be1 Compare February 28, 2022 21:29

dblock marked this pull request as ready for review February 28, 2022 21:34

dblock requested a review from saratvemulapalli February 28, 2022 21:34

dblock commented Feb 28, 2022

View reviewed changes

saratvemulapalli reviewed Mar 1, 2022

View reviewed changes

dblock requested review from saratvemulapalli and VachaShah March 1, 2022 15:11

dblock mentioned this pull request Mar 1, 2022

Run CI on JDK 8, 11, and 14. opensearch-project/job-scheduler#130

Merged

3 tasks

Add JDK 17.

de4d860

Signed-off-by: dblock <[email protected]>

dblock force-pushed the java-versions branch from 9743be1 to de4d860 Compare March 1, 2022 17:21

dblock mentioned this pull request Mar 1, 2022

Default CI Java Version to Java 11, run tests on 8, 11, 14 (and attempt 17) opensearch-project/opensearch-plugins#64

Closed

15 tasks

saratvemulapalli approved these changes Mar 1, 2022

View reviewed changes

VachaShah approved these changes Mar 1, 2022

View reviewed changes

dblock merged commit 3919543 into opensearch-project:main Mar 1, 2022

martin-gaievski mentioned this pull request Mar 2, 2022

Run CI on JDK 8 opensearch-project/k-NN#302

Merged

1 task

peternied added a commit to peternied/security that referenced this pull request Mar 2, 2022

Use adoptjdk and support building in one jdk and testing in another

1eb32b9

Following example from: opensearch-project/common-utils#121 Signed-off-by: Peter Nied <[email protected]>

peternied mentioned this pull request Mar 4, 2022

Downgrade Gradle version opensearch-project/security#1661

Merged

3 tasks

getsaurabh02 mentioned this pull request Mar 9, 2022

Backport main changes to 1.x #126

Merged

5 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Run CI/CD on Java 8, 11, 14 and 17. #121

Run CI/CD on Java 8, 11, 14 and 17. #121

dblock commented Feb 28, 2022 •

edited

Loading

dblock Feb 28, 2022

saratvemulapalli Mar 1, 2022

dblock Mar 1, 2022

saratvemulapalli Mar 1, 2022

saratvemulapalli Mar 1, 2022

saratvemulapalli Mar 1, 2022

dblock Mar 1, 2022

saratvemulapalli left a comment

VachaShah left a comment

Run CI/CD on Java 8, 11, 14 and 17. #121

Run CI/CD on Java 8, 11, 14 and 17. #121

Conversation

dblock commented Feb 28, 2022 • edited Loading

Description

Issues Resolved

Check List

dblock Feb 28, 2022

Choose a reason for hiding this comment

saratvemulapalli Mar 1, 2022

Choose a reason for hiding this comment

dblock Mar 1, 2022

Choose a reason for hiding this comment

saratvemulapalli Mar 1, 2022

Choose a reason for hiding this comment

saratvemulapalli Mar 1, 2022

Choose a reason for hiding this comment

saratvemulapalli Mar 1, 2022

Choose a reason for hiding this comment

dblock Mar 1, 2022

Choose a reason for hiding this comment

saratvemulapalli left a comment

Choose a reason for hiding this comment

VachaShah left a comment

Choose a reason for hiding this comment

dblock commented Feb 28, 2022 •

edited

Loading