Skip to content

Commit

Permalink
chore: use directory-based structure (#1895) - follow-up
Browse files Browse the repository at this point in the history
Signed-off-by: Ahmed AbouZaid <[email protected]>
  • Loading branch information
aabouzaid committed Jun 6, 2024
1 parent 2c98c82 commit 5acd286
Show file tree
Hide file tree
Showing 20 changed files with 20 additions and 647 deletions.
9 changes: 7 additions & 2 deletions .github/actions/workflow-vars/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ inputs:
description: The deployment cloud platform like GKE or ROSA.
identifier-base:
description: The fixed string in the identifier of the deployment it could be PR number or another specified name.
chart-dir:
description: A reference for the Camunda Helm chart directory which allows to test unreleased chagnes from Git repo.
# NOTE: This is not an exclusive list, some vars are exported as env var not an output.
outputs:
identifier:
Expand Down Expand Up @@ -82,8 +84,11 @@ runs:
# Chart version.
# Used to install previous supported versions of Camunda Helm charts.
TEST_CHART_VERSION="$(yq '.version' charts/${CI_CHART_DIR}/Chart.yaml)"
echo "TEST_CHART_VERSION=${TEST_CHART_VERSION}" | tee -a $GITHUB_ENV
# This is only needed in the upgrade flow not the setup.
if [[ "${{ inputs.setup-flow }}" == 'upgrade' ]]; then
TEST_CHART_VERSION="$(yq '.version' charts/${{ inputs.chart-dir }}/Chart.yaml)"
echo "TEST_CHART_VERSION=${TEST_CHART_VERSION}" | tee -a $GITHUB_ENV
fi
# Avoid confusion about the chart version since we only change the version during the release process
# as the "version" field in "Chart.yaml" file doesn't reflect the changes since the latest release.
Expand Down
7 changes: 7 additions & 0 deletions .github/renovate.json5
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,13 @@
// "addLabels": ["automerge"],
// "automerge": true
},
{
"matchPaths": [
"charts/camunda-platform*/Chart.yaml",
"charts/camunda-platform*/values*.yaml"
],
"matchUpdateTypes": ["patch"],
},
{
// Disable major version update for all Helm components.
"enabled": false,
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/renovate-post-upgrade.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ jobs:
token: '${{ steps.generate-github-token.outputs.token }}'
- uses: actions/setup-go@cdcb36043654635271a94b9a6d1392de5bb323a7 # v5
with:
go-version-file: go.mod
go-version-file: charts/camunda-platform/go.mod
cache: false
- uses: asdf-vm/actions/setup@05e0d2ed97b598bfce82fd30daf324ae0c4570e6 # v3
- uses: actions/cache@0c45773b623bea8c8e75f6c82b208c3cf94ea4f9 # v4
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-integration-template.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ env:
# Vars with "TEST_" prefix are used in the test runner tool (Task).
CI_PERSISTENT: ${{ inputs.persistent }}
CI_HOSTNAME_BASE: ci.distro.ultrawombat.com
CI_CHART_DIR: ${{ inputs.camunda-helm-dir }}
CI_SCENARIOS_DIR: charts/${{ inputs.camunda-helm-dir }}/test/integration/scenarios
# Docker Hub auth to avoid image pull rate limit.
TEST_CREATE_DOCKER_LOGIN_SECRET: "TRUE"
Expand Down Expand Up @@ -171,6 +170,7 @@ jobs:
platform: ${{ matrix.distro.platform }}
identifier-base: ${{ inputs.identifier }}
ingress-hostname-base: ${{ env.CI_HOSTNAME_BASE }}
chart-dir: ${{ inputs.camunda-helm-dir }}
- name: Install env dependencies
uses: asdf-vm/actions/install@05e0d2ed97b598bfce82fd30daf324ae0c4570e6 # v3
- name: Add Helm repos and dependencies
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ tasks:
deps: [init.seed]
cmds:
- helm install integration {{ .CHART_DIR }}
--version {{ .TEST_CHART_VERSION | default ">0.0.0" }}
--version {{ .TEST_CHART_VERSION | default ">0.0.0" | quote }}
--namespace {{ .TEST_NAMESPACE }}
--values {{ .FIXTURES_DIR }}/values-integration-test.yaml
--values ./values-integration-test-ingress.yaml
Expand Down
57 changes: 0 additions & 57 deletions charts/camunda-platform-8.2/values/values-latest.yaml

This file was deleted.

41 changes: 0 additions & 41 deletions charts/camunda-platform-8.2/values/values-v8.0.yaml

This file was deleted.

60 changes: 0 additions & 60 deletions charts/camunda-platform-8.2/values/values-v8.1.yaml

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ tasks:
deps: [init.seed]
cmds:
- helm install integration {{ .TEST_CHART_NAME }}
--version {{ .TEST_CHART_VERSION | default ">0.0.0" }}
--version {{ .TEST_CHART_VERSION | default ">0.0.0" | quote }}
--namespace {{ .TEST_NAMESPACE }}
--values {{ .FIXTURES_DIR }}/values-integration-test.yaml
--values ./values-integration-test-ingress.yaml
Expand Down
51 changes: 0 additions & 51 deletions charts/camunda-platform-8.3/values/values-latest.yaml

This file was deleted.

41 changes: 0 additions & 41 deletions charts/camunda-platform-8.3/values/values-v8.0-eol.yaml

This file was deleted.

60 changes: 0 additions & 60 deletions charts/camunda-platform-8.3/values/values-v8.1.yaml

This file was deleted.

Loading

0 comments on commit 5acd286

Please sign in to comment.