Skip to content

Commit

Permalink
ci: fix chart publish workflow (#874)
Browse files Browse the repository at this point in the history
* ci: correcting cross job output syntax

* ci: adding job names
  • Loading branch information
toast-gear authored Oct 5, 2021
1 parent b01e193 commit 8657a34
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
1 change: 1 addition & 0 deletions .github/workflows/on-push-lint-charts.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ env:
jobs:
lint-test:
runs-on: ubuntu-latest
name: Lint Chart
steps:
- name: Checkout
uses: actions/checkout@v2
Expand Down
10 changes: 7 additions & 3 deletions .github/workflows/on-push-master-publish-chart.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ env:
jobs:
lint-chart:
runs-on: ubuntu-latest
name: Lint Chart
outputs:
publish-chart: ${{ steps.publish-chart-step.outputs.publish }}
steps:
- name: Checkout
uses: actions/checkout@v2
Expand Down Expand Up @@ -81,7 +84,7 @@ jobs:

# WARNING: This relies on the latest release being inat the top of the JSON from GitHub and a clean chart.yaml
- name: Check if Chart Publish is Needed
id: publish-chart
id: publish-chart-step
run: |
CHART_TEXT=$(curl -fs https://raw.githubusercontent.com/actions-runner-controller/actions-runner-controller/master/charts/actions-runner-controller/Chart.yaml)
NEW_CHART_VERSION=$(echo "$CHART_TEXT" | grep version: | cut -d ' ' -f 2)
Expand All @@ -94,9 +97,10 @@ jobs:
fi
publish-chart:
if: steps.publish-chart.outputs.publish == 'true'
runs-on: ubuntu-latest
if: needs.lint-chart.outputs.publish-chart == 'true'
needs: lint-chart
runs-on: ubuntu-latest
name: Publish Chart

steps:
- name: Checkout
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/wip.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ on:
jobs:
build:
runs-on: ubuntu-latest
name: release-latest
name: Build and Publish Canary Image
env:
DOCKERHUB_USERNAME: ${{ secrets.DOCKER_USER }}
steps:
Expand Down

0 comments on commit 8657a34

Please sign in to comment.