Skip to content

[CI] Remove usage of diff-only action with paths-ignore #9527

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.

Sign up for GitHub

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

Merged
jiazhai merged 1 commit into from
Feb 9, 2021
Merged

[CI] Remove usage of diff-only action with paths-ignore #9527

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 2 additions & 12 deletions .github/workflows/ci-cpp.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,11 @@ on:
pull_request:
branches:
- master
paths-ignore: ['site2/**', 'deployment/**']
push:
branches:
- branch-*
paths-ignore: ['site2/**', 'deployment/**']

env:
MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.count=3
Expand All @@ -43,14 +45,7 @@ jobs:
fetch-depth: 25
ref: ${{ github.event.pull_request.head.sha }}

- name: Check if this pull request only changes documentation
id: docs
uses: apache/pulsar-test-infra/diff-only@master
with:
args: site2 deployment .asf.yaml .ci ct.yaml

- name: Cache local Maven repository
if: steps.docs.outputs.changed_only == 'no'
uses: actions/cache@v2
with:
path: ~/.m2/repository
Expand All @@ -60,12 +55,10 @@ jobs:

- name: Set up JDK 1.8
uses: actions/setup-java@v1
if: steps.docs.outputs.changed_only == 'no'
with:
java-version: 1.8

- name: clean disk
if: steps.docs.outputs.changed_only == 'no'
run: |
sudo swapoff -a
sudo rm -f /swapfile
Expand All @@ -74,16 +67,13 @@ jobs:
df -h

- name: build package
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -ntp -q install -Pcore-modules -DskipTests

- name: build cpp artifacts
if: steps.docs.outputs.changed_only == 'no'
run: |
echo "Build C++ client library"
export CMAKE_ARGS="-DCMAKE_BUILD_TYPE=Debug -DBUILD_DYNAMIC_LIB=OFF"
pulsar-client-cpp/docker-build.sh

- name: run c++ tests
if: steps.docs.outputs.changed_only == 'no'
run: ./build/retry.sh pulsar-client-cpp/docker-tests.sh
10 changes: 0 additions & 10 deletions .github/workflows/ci-go-functions-style.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,34 +51,24 @@ jobs:
fetch-depth: 25
ref: ${{ github.event.pull_request.head.sha }}

- name: Check if this pull request only changes documentation
id: docs
uses: apache/pulsar-test-infra/diff-only@master
with:
args: site2 deployment .asf.yaml .ci ct.yaml

- name: Set up Go
uses: actions/setup-go@v2
if: steps.docs.outputs.changed_only == 'no'
with:
go-version: ${{ matrix.go-version }}
id: go

- name: InstallTool
if: steps.docs.outputs.changed_only == 'no'
run: |
cd pulsar-function-go
wget -O - -q https://github.com/ghraw/golangci/golangci-lint/master/install.sh| sh -s v1.18.0
./bin/golangci-lint --version

- name: Build
if: steps.docs.outputs.changed_only == 'no'
run: |
cd pulsar-function-go
go build ./...

- name: CheckStyle
if: steps.docs.outputs.changed_only == 'no'
run: |
cd pulsar-function-go
./bin/golangci-lint run -c ./golangci.yml ./pf
8 changes: 0 additions & 8 deletions .github/workflows/ci-go-functions-test.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,21 +53,13 @@ jobs:
fetch-depth: 25
ref: ${{ github.event.pull_request.head.sha }}

- name: Check if this pull request only changes documentation
id: docs
uses: apache/pulsar-test-infra/diff-only@master
with:
args: site2 deployment .asf.yaml .ci ct.yaml

- name: Set up Go
uses: actions/setup-go@v2
if: steps.docs.outputs.changed_only == 'no'
with:
go-version: ${{ matrix.go-version }}
id: go

- name: Run tests
if: steps.docs.outputs.changed_only == 'no'
run: |
cd pulsar-function-go
go test -v $(go list ./... | grep -v examples)
16 changes: 2 additions & 14 deletions .github/workflows/ci-integration-backwards-compatibility.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,11 @@ on:
pull_request:
branches:
- master
paths-ignore: ['site2/**', 'deployment/**']
push:
branches:
- branch-*

paths-ignore: ['site2/**', 'deployment/**']
env:
MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.count=3

Expand All @@ -51,20 +52,12 @@ jobs:
restore-keys: |
${{ runner.os }}-maven-

- name: Check if this pull request only changes documentation
id: docs
uses: apache/pulsar-test-infra/diff-only@master
with:
args: site2 deployment .asf.yaml .ci ct.yaml

- name: Set up JDK 1.8
if: steps.docs.outputs.changed_only == 'no'
uses: actions/setup-java@v1
with:
java-version: 1.8

- name: clean disk
if: steps.docs.outputs.changed_only == 'no'
run: |
sudo swapoff -a
sudo rm -rf /swapfile /usr/share/dotnet /usr/local/lib/android /opt/ghc
Expand All @@ -73,21 +66,16 @@ jobs:
df -h

- name: run install by skip tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -q -B -ntp clean install -DskipTests

- name: build pulsar image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f docker/pulsar/pom.xml install -am -Pdocker -DskipTests -Ddocker.nocache=true

- name: build pulsar-all image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f docker/pulsar-all/pom.xml install -am -Pdocker -DskipTests -Ddocker.nocache=true

- name: build artifacts and docker image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/docker-images/pom.xml install -am -Pdocker -DskipTests

- name: run integration tests
if: steps.docs.outputs.changed_only == 'no'
run: ./build/retry.sh mvn -B -f tests/pom.xml test -DintegrationTestSuiteFile=pulsar-backwards-compatibility.xml -DintegrationTests -DredirectTestOutputToFile=false
17 changes: 2 additions & 15 deletions .github/workflows/ci-integration-cli.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,11 @@ on:
pull_request:
branches:
- master
paths-ignore: ['site2/**', 'deployment/**']
push:
branches:
- branch-*

paths-ignore: ['site2/**', 'deployment/**']
env:
MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.count=3

Expand All @@ -51,20 +52,12 @@ jobs:
restore-keys: |
${{ runner.os }}-maven-

- name: Check if this pull request only changes documentation
id: docs
uses: apache/pulsar-test-infra/diff-only@master
with:
args: site2 deployment .asf.yaml .ci ct.yaml

- name: Set up JDK 1.8
uses: actions/setup-java@v1
if: steps.docs.outputs.changed_only == 'no'
with:
java-version: 1.8

- name: clean disk
if: steps.docs.outputs.changed_only == 'no'
run: |
sudo swapoff -a
sudo rm -rf /swapfile /usr/share/dotnet /usr/local/lib/android /opt/ghc
Expand All @@ -73,26 +66,20 @@ jobs:
df -h

- name: run install by skip tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -q -B -ntp clean install -DskipTests

- name: build pulsar image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f docker/pulsar/pom.xml install -am -Pdocker -DskipTests -Ddocker.nocache=true

- name: build pulsar-all image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f docker/pulsar-all/pom.xml install -am -Pdocker -DskipTests -Ddocker.nocache=true

- name: build artifacts and docker image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/docker-images/pom.xml install -am -Pdocker -DskipTests

- name: run pulsar cli integration tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/pom.xml test -DintegrationTestSuiteFile=pulsar-cli.xml -DintegrationTests -DredirectTestOutputToFile=false

- name: run pulsar auth integration tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/pom.xml test -DintegrationTestSuiteFile=pulsar-auth.xml -DintegrationTests -DredirectTestOutputToFile=false

16 changes: 2 additions & 14 deletions .github/workflows/ci-integration-function-state.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,11 @@ on:
pull_request:
branches:
- master
paths-ignore: ['site2/**', 'deployment/**']
push:
branches:
- branch-*

paths-ignore: ['site2/**', 'deployment/**']
env:
MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.count=3

Expand All @@ -51,20 +52,12 @@ jobs:
restore-keys: |
${{ runner.os }}-maven-

- name: Check if this pull request only changes documentation
id: docs
uses: apache/pulsar-test-infra/diff-only@master
with:
args: site2 deployment .asf.yaml .ci ct.yaml

- name: Set up JDK 1.8
uses: actions/setup-java@v1
if: steps.docs.outputs.changed_only == 'no'
with:
java-version: 1.8

- name: clean disk
if: steps.docs.outputs.changed_only == 'no'
run: |
sudo swapoff -a
sudo rm -rf /swapfile /usr/share/dotnet /usr/local/lib/android /opt/ghc
Expand All @@ -73,21 +66,16 @@ jobs:
df -h

- name: run install by skip tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -q -B -ntp clean install -DskipTests

- name: build pulsar image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f docker/pulsar/pom.xml install -am -Pdocker -DskipTests -Ddocker.nocache=true

- name: build pulsar-all image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f docker/pulsar-all/pom.xml install -am -Pdocker -DskipTests -Ddocker.nocache=true

- name: build artifacts and docker image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/docker-images/pom.xml install -am -Pdocker -DskipTests

- name: run integration tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/pom.xml test -DintegrationTestSuiteFile=pulsar-function-state.xml -DintegrationTests -DredirectTestOutputToFile=false
18 changes: 2 additions & 16 deletions .github/workflows/ci-integration-messaging.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,11 @@ on:
pull_request:
branches:
- master
paths-ignore: ['site2/**', 'deployment/**']
push:
branches:
- branch-*

paths-ignore: ['site2/**', 'deployment/**']
env:
MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.count=3

Expand All @@ -51,20 +52,12 @@ jobs:
restore-keys: |
${{ runner.os }}-maven-

- name: Check if this pull request only changes documentation
id: docs
uses: apache/pulsar-test-infra/diff-only@master
with:
args: site2 deployment .asf.yaml .ci ct.yaml

- name: Set up JDK 1.8
uses: actions/setup-java@v1
if: steps.docs.outputs.changed_only == 'no'
with:
java-version: 1.8

- name: clean disk
if: steps.docs.outputs.changed_only == 'no'
run: |
sudo swapoff -a
sudo rm -rf /swapfile /usr/share/dotnet /usr/local/lib/android /opt/ghc
Expand All @@ -73,29 +66,22 @@ jobs:
df -h

- name: run install by skip tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -q -B -ntp clean install -DskipTests

- name: build pulsar image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f docker/pulsar/pom.xml install -am -Pdocker -DskipTests -Ddocker.nocache=true

- name: build pulsar-all image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f docker/pulsar-all/pom.xml install -am -Pdocker -DskipTests -Ddocker.nocache=true

- name: build artifacts and docker image
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/docker-images/pom.xml install -am -Pdocker -DskipTests -Ddocker.nocache=true

- name: run integration messaging tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/pom.xml test -DintegrationTestSuiteFile=pulsar-messaging.xml -DintegrationTests -DredirectTestOutputToFile=false

- name: run integration proxy tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/pom.xml test -DintegrationTestSuiteFile=pulsar-proxy.xml -DintegrationTests -DredirectTestOutputToFile=false

- name: run integration proxy with WebSocket tests
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -f tests/pom.xml test -DintegrationTestSuiteFile=pulsar-proxy-websocket.xml -DintegrationTests -DredirectTestOutputToFile=false
Loading