You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by si...@apache.org on 2020/04/13 04:57:34 UTC
[pulsar] branch master updated: Skip helm chart testing if the
changes are docs-only changes (#6726)
This is an automated email from the ASF dual-hosted git repository.
sijie 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 dcb7499 Skip helm chart testing if the changes are docs-only changes (#6726)
dcb7499 is described below
commit dcb7499c69d13ac7004ec61b35721224be7003f4
Author: Sijie Guo <si...@apache.org>
AuthorDate: Sun Apr 12 21:57:27 2020 -0700
Skip helm chart testing if the changes are docs-only changes (#6726)
*Motivation*
We don't need to run chart tests if the change only changes documentation
---
.github/workflows/pulsar.yml | 14 ++++++++++++--
.github/workflows/pulsar_bk_tls.yml | 14 ++++++++++++--
.github/workflows/pulsar_broker_tls.yml | 14 ++++++++++++--
.github/workflows/pulsar_function.yml | 14 ++++++++++++--
.github/workflows/pulsar_image.yml | 14 ++++++++++++--
.github/workflows/pulsar_jwt_asymmetric.yml | 14 ++++++++++++--
.github/workflows/pulsar_jwt_symmetric.yml | 16 +++++++++++++---
.github/workflows/pulsar_tls.yml | 18 ++++++++++++++----
.github/workflows/pulsar_zk_tls.yml | 18 ++++++++++++++----
.github/workflows/pulsar_zkbk_tls.yml | 18 ++++++++++++++----
10 files changed, 127 insertions(+), 27 deletions(-)
diff --git a/.github/workflows/pulsar.yml b/.github/workflows/pulsar.yml
index 486423a..f37b624 100644
--- a/.github/workflows/pulsar.yml
+++ b/.github/workflows/pulsar.yml
@@ -29,11 +29,21 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- name: Lint chart
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
@@ -41,4 +51,4 @@ jobs:
run: |
.ci/chart_test.sh .ci/clusters/values-local-pv.yaml
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'
diff --git a/.github/workflows/pulsar_bk_tls.yml b/.github/workflows/pulsar_bk_tls.yml
index 3e45495..1415003 100644
--- a/.github/workflows/pulsar_bk_tls.yml
+++ b/.github/workflows/pulsar_bk_tls.yml
@@ -29,11 +29,21 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- name: Run chart-testing (lint)
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
@@ -41,4 +51,4 @@ jobs:
run: |
.ci/chart_test.sh .ci/clusters/values-bk-tls.yaml
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'
diff --git a/.github/workflows/pulsar_broker_tls.yml b/.github/workflows/pulsar_broker_tls.yml
index 4c531cf..cc55778 100644
--- a/.github/workflows/pulsar_broker_tls.yml
+++ b/.github/workflows/pulsar_broker_tls.yml
@@ -29,11 +29,21 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- name: Run chart-testing (lint)
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
@@ -41,4 +51,4 @@ jobs:
run: |
.ci/chart_test.sh .ci/clusters/values-broker-tls.yaml
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'
diff --git a/.github/workflows/pulsar_function.yml b/.github/workflows/pulsar_function.yml
index 1b873cc8..814e7c9 100644
--- a/.github/workflows/pulsar_function.yml
+++ b/.github/workflows/pulsar_function.yml
@@ -29,11 +29,21 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- name: Lint chart
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
@@ -43,4 +53,4 @@ jobs:
env:
FUNCTION: "true"
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'
diff --git a/.github/workflows/pulsar_image.yml b/.github/workflows/pulsar_image.yml
index 9860da0..6c324ae 100644
--- a/.github/workflows/pulsar_image.yml
+++ b/.github/workflows/pulsar_image.yml
@@ -29,11 +29,21 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- name: Lint chart
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
@@ -41,4 +51,4 @@ jobs:
run: |
.ci/chart_test.sh .ci/clusters/values-pulsar-image.yaml
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'
diff --git a/.github/workflows/pulsar_jwt_asymmetric.yml b/.github/workflows/pulsar_jwt_asymmetric.yml
index 5dfc967..e444a32 100644
--- a/.github/workflows/pulsar_jwt_asymmetric.yml
+++ b/.github/workflows/pulsar_jwt_asymmetric.yml
@@ -29,11 +29,21 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- name: Run chart-testing (lint)
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
@@ -43,4 +53,4 @@ jobs:
env:
SYMMETRIC: "false"
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'
diff --git a/.github/workflows/pulsar_jwt_symmetric.yml b/.github/workflows/pulsar_jwt_symmetric.yml
index 2f62167..711248d 100644
--- a/.github/workflows/pulsar_jwt_symmetric.yml
+++ b/.github/workflows/pulsar_jwt_symmetric.yml
@@ -29,11 +29,21 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- - name: Run chart-testing (lint)
+ - name: Lint chart
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
@@ -43,4 +53,4 @@ jobs:
env:
SYMMETRIC: "true"
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'
diff --git a/.github/workflows/pulsar_tls.yml b/.github/workflows/pulsar_tls.yml
index 881aa34..6ac9399 100644
--- a/.github/workflows/pulsar_tls.yml
+++ b/.github/workflows/pulsar_tls.yml
@@ -29,16 +29,26 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- - name: Run chart-testing (lint)
+ - name: Lint chart
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
- - name: Run chart-testing (install)
+ - name: Install chart
run: |
.ci/chart_test.sh .ci/clusters/values-tls.yaml
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'
diff --git a/.github/workflows/pulsar_zk_tls.yml b/.github/workflows/pulsar_zk_tls.yml
index 6adf891..cf8a34a 100644
--- a/.github/workflows/pulsar_zk_tls.yml
+++ b/.github/workflows/pulsar_zk_tls.yml
@@ -29,16 +29,26 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- - name: Run chart-testing (lint)
+ - name: Lint chart
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
- - name: Run chart-testing (install)
+ - name: Install chart
run: |
.ci/chart_test.sh .ci/clusters/values-zk-tls.yaml
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'
diff --git a/.github/workflows/pulsar_zkbk_tls.yml b/.github/workflows/pulsar_zkbk_tls.yml
index 13bb122..045b0b4 100644
--- a/.github/workflows/pulsar_zkbk_tls.yml
+++ b/.github/workflows/pulsar_zkbk_tls.yml
@@ -29,16 +29,26 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
- uses: actions/checkout@v1
+ uses: actions/checkout@v2
+ with:
+ fetch-depth: 0
+ 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 .asf.yaml ct.yaml
- - name: Run chart-testing (lint)
+ - name: Lint chart
id: lint
uses: helm/chart-testing-action@master
+ if: steps.docs.outputs.changed_only == 'no'
with:
command: lint
- - name: Run chart-testing (install)
+ - name: Install chart
run: |
.ci/chart_test.sh .ci/clusters/values-zkbk-tls.yaml
# Only build a kind cluster if there are chart changes to test.
- if: steps.lint.outputs.changed == 'true'
+ if: steps.lint.outputs.changed == 'true' && steps.docs.outputs.changed_only == 'no'