Fail the chart-testing job if linting fails (#429)
This commit is contained in:
parent
8061a8b7aa
commit
62569f4023
13
.github/workflows/pulsar-helm-chart-ci.yaml
vendored
13
.github/workflows/pulsar-helm-chart-ci.yaml
vendored
@ -86,8 +86,6 @@ jobs:
|
|||||||
runs-on: ubuntu-22.04
|
runs-on: ubuntu-22.04
|
||||||
timeout-minutes: 45
|
timeout-minutes: 45
|
||||||
if: ${{ needs.preconditions.outputs.docs_only != 'true' }}
|
if: ${{ needs.preconditions.outputs.docs_only != 'true' }}
|
||||||
outputs:
|
|
||||||
no_chart_changes: ${{ steps.ct-lint.outputs.no_chart_changes }}
|
|
||||||
steps:
|
steps:
|
||||||
- name: checkout
|
- name: checkout
|
||||||
uses: actions/checkout@v4
|
uses: actions/checkout@v4
|
||||||
@ -127,16 +125,7 @@ jobs:
|
|||||||
run: |
|
run: |
|
||||||
ct lint --check-version-increment=false \
|
ct lint --check-version-increment=false \
|
||||||
--validate-maintainers=false \
|
--validate-maintainers=false \
|
||||||
--target-branch ${{ github.event.repository.default_branch }} \
|
--target-branch ${{ github.event.repository.default_branch }}
|
||||||
| tee /tmp/ct-lint.log || {
|
|
||||||
if grep -q "No chart changes detected." /tmp/ct-lint.log; then
|
|
||||||
echo no_chart_changes=true >> $GITHUB_OUTPUT
|
|
||||||
exit 0
|
|
||||||
else
|
|
||||||
echo no_chart_changes=false >> $GITHUB_OUTPUT
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
}
|
|
||||||
|
|
||||||
- name: Wait for ssh connection when build fails
|
- name: Wait for ssh connection when build fails
|
||||||
# ssh access is enabled for builds in own forks
|
# ssh access is enabled for builds in own forks
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user