This is an automated email from the ASF dual-hosted git repository.
mbudiu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/calcite.git
The following commit(s) were added to refs/heads/main by this push:
new 7be93a9757 [CALCITE-6526] Refactor DEVELOCITY_ACCESS_KEY definition in
one place
7be93a9757 is described below
commit 7be93a9757b98a7151515b01871e15594fcdac10
Author: Stamatis Zampetakis <[email protected]>
AuthorDate: Tue Aug 13 11:43:59 2024 +0300
[CALCITE-6526] Refactor DEVELOCITY_ACCESS_KEY definition in one place
The DEVELOCITY_ACCESS_KEY environment variable definition is used in every
CI job inside the main.yml file and it appears ~16 times. Instead of repeating
the same definition multiple times we can set it only once for all jobs.
---
.github/workflows/main.yml | 33 +--------------------------------
1 file changed, 1 insertion(+), 32 deletions(-)
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
index 1e76482540..56a9276cbd 100644
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@@ -45,6 +45,7 @@ concurrency:
# This avoids never-ending GC trashing if memory gets too low in case of a
memory leak
env:
_JAVA_OPTIONS: '-XX:GCTimeLimit=90 -XX:GCHeapFreeLimit=35'
+ DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
jobs:
windows-jdk8:
@@ -62,8 +63,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -94,8 +93,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -126,8 +123,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -161,8 +156,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -208,8 +201,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -232,8 +223,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -256,8 +245,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -280,8 +267,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -302,8 +287,6 @@ jobs:
git clone --branch main --depth 100
https://github.com/apache/calcite-avatica.git ../calcite-avatica
- uses: burrunan/gradle-cache-action@v1
name: Build Avatica
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: avatica-jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -317,8 +300,6 @@ jobs:
fetch-depth: 50
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk${{ matrix.jdk }}
remote-build-cache-proxy-enabled: false
@@ -343,8 +324,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk21
remote-build-cache-proxy-enabled: false
@@ -376,8 +355,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: errprone
remote-build-cache-proxy-enabled: false
@@ -397,8 +374,6 @@ jobs:
distribution: 'zulu'
- name: 'Run CheckerFramework'
uses: burrunan/gradle-cache-action@v1
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: checkerframework-jdk11
remote-build-cache-proxy-enabled: false
@@ -420,8 +395,6 @@ jobs:
distribution: 'zulu'
- name: 'Run CheckerFramework'
uses: burrunan/gradle-cache-action@v1
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: checkerframework-jdk11
remote-build-cache-proxy-enabled: false
@@ -444,8 +417,6 @@ jobs:
distribution: 'zulu'
- uses: burrunan/gradle-cache-action@v1
name: Test
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
job-id: jdk8
remote-build-cache-proxy-enabled: false
@@ -489,8 +460,6 @@ jobs:
- uses: burrunan/gradle-cache-action@v1
name: 'Run Druid tests'
timeout-minutes: 10
- env:
- DEVELOCITY_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
with:
build-root-directory: ./calcite
job-id: Druid8