Skip to content

Commit

Permalink
chore: use directory-based structure (#1895) - follow-up
Browse files Browse the repository at this point in the history
  • Loading branch information
aabouzaid committed Jun 6, 2024
1 parent 2c98c82 commit a499a26
Show file tree
Hide file tree
Showing 18 changed files with 12 additions and 646 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
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 a499a26

Please sign in to comment.