You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by lh...@apache.org on 2022/09/12 11:18:38 UTC
[pulsar] branch master updated: [improve][ci] Skip MacOS build for cpp-only changes (#17565)
This is an automated email from the ASF dual-hosted git repository.
lhotari pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 65164eb4437 [improve][ci] Skip MacOS build for cpp-only changes (#17565)
65164eb4437 is described below
commit 65164eb44379ec713fe508c63acba336f7d994df
Author: Nicolò Boschi <bo...@gmail.com>
AuthorDate: Mon Sep 12 13:18:29 2022 +0200
[improve][ci] Skip MacOS build for cpp-only changes (#17565)
---
.github/workflows/ci-cpp-build.yaml | 8 ++---
.github/workflows/pulsar-ci.yaml | 66 +++++++++++++++++++------------------
2 files changed, 38 insertions(+), 36 deletions(-)
diff --git a/.github/workflows/ci-cpp-build.yaml b/.github/workflows/ci-cpp-build.yaml
index fa50abfdebd..77cdf574396 100644
--- a/.github/workflows/ci-cpp-build.yaml
+++ b/.github/workflows/ci-cpp-build.yaml
@@ -57,7 +57,7 @@ jobs:
cpp-build-centos7:
needs: changed_files_job
- name:
+ name: Build CPP Client on CentOS7
runs-on: ubuntu-20.04
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
timeout-minutes: 120
@@ -77,7 +77,7 @@ jobs:
cpp-build-windows:
needs: changed_files_job
timeout-minutes: 120
- name: ${{ matrix.name }}
+ name: Build CPP Client on ${{ matrix.name }}
runs-on: ${{ matrix.os }}
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
env:
@@ -184,7 +184,7 @@ jobs:
fi
cpp-deb-rpm-packaging:
needs: changed_files_job
- name:
+ name: Build CPP Client on RPM
runs-on: ubuntu-20.04
timeout-minutes: 120
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
@@ -233,7 +233,7 @@ jobs:
build-python-wheel:
needs: changed_files_job
- name:
+ name: Build Python Client
runs-on: ubuntu-20.04
timeout-minutes: 120
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
diff --git a/.github/workflows/pulsar-ci.yaml b/.github/workflows/pulsar-ci.yaml
index 3f60f6f3815..2f2e63ab367 100644
--- a/.github/workflows/pulsar-ci.yaml
+++ b/.github/workflows/pulsar-ci.yaml
@@ -124,38 +124,7 @@ jobs:
with:
action: wait
- macos-build:
- name:
- runs-on: macos-11
- timeout-minutes: 120
- needs: [ 'changed_files_job', 'build-and-license-check' ]
- if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
- steps:
- - name: checkout
- uses: actions/checkout@v2
- - name: Tune Runner VM
- uses: ./.github/actions/tune-runner-vm
-
- - name: Cache Maven dependencies
- if: ${{ steps.check_changes.outputs.docs_only != 'true' }}
- uses: actions/cache@v2
- with:
- path: |
- ~/.m2/repository/*/*/*
- !~/.m2/repository/org/apache/pulsar
- key: ${{ runner.os }}-m2-dependencies-all-${{ hashFiles('**/pom.xml') }}
- restore-keys: |
- ${{ runner.os }}-m2-dependencies-all-
-
- - name: Set up JDK 17
- uses: actions/setup-java@v2
- with:
- distribution: 'temurin'
- java-version: 17
-
- - name: build package
- run: mvn -B clean package -DskipTests -T 1C -ntp
unit-tests:
name: CI - Unit - ${{ matrix.name }}
@@ -853,6 +822,39 @@ jobs:
- name: Delete docker image from GitHub Actions Artifacts
run: |
gh-actions-artifact-client.js delete pulsar-test-latest-version-image.zst
+
+ macos-build:
+ name: Build Pulsar on MacOS
+ runs-on: macos-11
+ timeout-minutes: 120
+ needs: ['changed_files_job', 'integration-tests']
+ if: ${{ needs.changed_files_job.outputs.docs_only != 'true' && needs.changed_files_job.outputs.cpp_only != 'true' }}
+ steps:
+ - name: checkout
+ uses: actions/checkout@v2
+
+ - name: Tune Runner VM
+ uses: ./.github/actions/tune-runner-vm
+
+ - name: Cache Maven dependencies
+ uses: actions/cache@v2
+ with:
+ path: |
+ ~/.m2/repository/*/*/*
+ !~/.m2/repository/org/apache/pulsar
+ key: ${{ runner.os }}-m2-dependencies-all-${{ hashFiles('**/pom.xml') }}
+ restore-keys: |
+ ${{ runner.os }}-m2-dependencies-all-
+
+ - name: Set up JDK 17
+ uses: actions/setup-java@v2
+ with:
+ distribution: 'temurin'
+ java-version: 17
+
+ - name: build package
+ run: mvn -B clean package -DskipTests -T 1C -ntp
+
# This job is required for pulls to be merged.
# It depends on all other jobs in this workflow.
@@ -884,4 +886,4 @@ jobs:
- name: Delete maven repository binaries from GitHub Actions Artifacts
if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
run: |
- gh-actions-artifact-client.js delete pulsar-maven-repository-binaries.tar.zst
\ No newline at end of file
+ gh-actions-artifact-client.js delete pulsar-maven-repository-binaries.tar.zst