Skip to content

Commit 3a8265a

Browse files
lentidaslentidas
committed
ci: rename the workflow and use new naming scheme
1 parent 0dc9a99 commit 3a8265a

1 file changed

Lines changed: 28 additions & 26 deletions

File tree

.github/workflows/modules-chart-upgrade.yaml renamed to .github/workflows/modules-chart-update.yaml

Lines changed: 28 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,26 @@
11
---
2-
# GitHub Actions workflow to upgrade the Helm chart dependencies on our modules.
2+
# GitHub Actions workflow to update the Helm chart dependencies on our modules.
33
#
44
# IMPORTANT: This workflow is called by other workflows in our DevOps Stack repositories and it is centralized here in
55
# order to be easily maintained across modules. Because of this, please make sure you're not introducing any breaking
66
# changes when modifying this workflow.
77

8-
name: "modules-chart-upgrade"
8+
name: "modules-chart-update"
99

1010
on:
1111
workflow_call:
1212
inputs:
13-
upgrade-strategy:
13+
update-strategy:
1414
description: "Upgrade strategy to use. Valid values are 'major', 'minor' or 'patch'"
1515
type: string
1616
required: true
1717
excluded-dependencies:
18-
description: "Comma-separated list of dependencies to exclude from upgrade (i.e. 'dependency1,dependency2,dependency3')"
18+
description: "Comma-separated list of dependencies to exclude from the update (i.e. 'dependency1,dependency2,dependency3')"
1919
type: string
2020
required: false
2121
default: ""
2222
dry-run:
23-
description: "Whether to run the upgrade in dry-run mode or not"
23+
description: "Whether to run the update in dry-run mode or not"
2424
type: boolean
2525
required: false
2626
default: false
@@ -43,7 +43,7 @@
4343
id: find-charts
4444
run: cd charts && echo "charts=$(find . -maxdepth 2 -name 'Chart.yaml' -exec dirname {} \; | sed 's|^\./||' | sort -u | jq -R -s -c 'split("\n")[:-1]')" >> $GITHUB_OUTPUT
4545

46-
chart-upgrade:
46+
chart-update:
4747
runs-on: ubuntu-latest
4848

4949
needs: list-charts
@@ -66,63 +66,65 @@
6666
uses: actions/checkout@v3
6767

6868
- name: "Upgrade Helm chart dependencies"
69-
id: deps-upgrade
70-
uses: camptocamp/helm-dependency-upgrade-action@v0.3.3
69+
id: deps-update
70+
uses: camptocamp/helm-dependency-update-action@v0.3.3
7171
with:
7272
chart-path: "charts/${{ matrix.chart-name }}"
7373
readme-path: "README.adoc"
7474
excluded-dependencies: ${{ inputs.excluded-dependencies }}
75-
upgrade-strategy: "${{ inputs.upgrade-strategy }}"
75+
update-strategy: "${{ inputs.update-strategy }}"
7676
dry-run: "${{ inputs.dry-run }}"
7777

78-
- name: "Create Pull Request for a minor/patch upgrade"
79-
if: ${{ !inputs.dry-run && steps.deps-upgrade.outputs.upgrade-type != 'none' && !steps.deps-upgrade.outputs.upgrade-type != 'major' }}
78+
- name: "Create Pull Request for a minor/patch update"
79+
if: ${{ !inputs.dry-run && steps.deps-update.outputs.update-type != 'none' && !steps.deps-update.outputs.update-type != 'major' }}
8080
id: minor-pr
8181
uses: peter-evans/create-pull-request@v3
8282
env:
83-
pr-title: "chore(chart): ${{ steps.deps-upgrade.outputs.upgrade-type }} upgrade of dependencies on ${{ matrix.chart-name }} chart"
84-
branch: "chart-autoupgrade-${{ steps.deps-upgrade.outputs.upgrade-type }}-${{ matrix.chart-name }}"
85-
labels: "chart-autoupgrade-${{ steps.deps-upgrade.outputs.upgrade-type }}"
83+
pr-title: "feat(chart): ${{ steps.deps-update.outputs.update-type }} update of dependencies on ${{ matrix.chart-name }} chart"
84+
branch: "chart-autoupdate-${{ steps.deps-update.outputs.update-type }}-${{ matrix.chart-name }}"
85+
labels: "chart-autoupdate-${{ steps.deps-update.outputs.update-type }}"
8686

8787
with:
8888
commit-message: ${{ env.pr-title }}
8989
committer: ${{ env.committer }}
90-
branch: "chart-autoupgrade-${{ steps.deps-upgrade.outputs.upgrade-type }}-${{ matrix.chart-name }}"
90+
branch: "chart-autoupdate-${{ steps.deps-update.outputs.update-type }}-${{ matrix.chart-name }}"
9191
title: ${{ env.pr-title }}
92-
labels: "chart-autoupgrade-${{ steps.deps-upgrade.outputs.upgrade-type }}"
92+
labels: "chart-autoupdate-${{ steps.deps-update.outputs.update-type }}"
9393
body: |
9494
:robot: I have updated the chart *beep* *boop*
9595
---
9696
9797
## Description of the changes
9898
99-
This PR upgrades the dependencies of the **${{ matrix.chart-name }}** Helm chart. The maximum version bump was a **${{ steps.deps-upgrade.outputs.upgrade-type }}**.
99+
This PR updates the dependencies of the **${{ matrix.chart-name }}** Helm chart.
100+
101+
The maximum version bump was a **${{ steps.deps-update.outputs.update-type }}** step.
100102
101-
- name: "Create Pull Request for a major upgrade"
102-
if: ${{ !inputs.dry-run && steps.deps-upgrade.outputs.upgrade-type != 'none' && steps.deps-upgrade.outputs.upgrade-type == 'major' }}
103+
- name: "Create Pull Request for a major update"
104+
if: ${{ !inputs.dry-run && steps.deps-update.outputs.update-type != 'none' && steps.deps-update.outputs.update-type == 'major' }}
103105
id: major-pr
104106
uses: peter-evans/create-pull-request@v3
105107
env:
106-
pr-title: "chore!(chart): major upgrade of dependencies on ${{ matrix.chart-name }} chart"
108+
pr-title: "feat!(chart): major update of dependencies on ${{ matrix.chart-name }} chart"
107109
with:
108110
commit-message: ${{ env.pr-title }}
109111
committer: ${{ env.committer }}
110-
branch: "chart-autoupgrade-major-${{ matrix.chart-name }}"
112+
branch: "chart-autoupdate-major-${{ matrix.chart-name }}"
111113
title: ${{ env.pr-title }}
112-
labels: "chart-autoupgrade-major"
114+
labels: "chart-autoupdate-major"
113115
body: |
114116
:robot: I have updated the chart *beep* *boop*
115117
---
116118
117119
## Description of the changes
118120
119-
This PR upgrades the dependencies of the **${{ matrix.chart-name }}** Helm chart. .
121+
This PR updates the dependencies of the **${{ matrix.chart-name }}** Helm chart. .
120122
121-
:warning: This was a **major** upgrade!. Please check the changelog of the updated dependencies and take notice of any breaking changes before merging. :warning:
123+
:warning: This was a **major** update!. Please check the changelog of the updated dependencies and take notice of any breaking changes before merging. :warning:
122124
123125
add-pr-to-project:
124-
needs: chart-upgrade
125-
if: ${{ needs.chart-upgrade.outputs.minor-pr-number || needs.chart-upgrade.outputs.major-pr-number }}
126+
needs: chart-update
127+
if: ${{ needs.chart-update.outputs.minor-pr-number || needs.chart-update.outputs.major-pr-number }}
126128
uses: camptocamp/devops-stack/.github/workflows/pr-issues-project.yaml@main
127129
secrets:
128130
PROJECT_APP_PRIVATE_KEY: ${{ secrets.PROJECT_APP_PRIVATE_KEY }}

0 commit comments

Comments
 (0)