From 4b522de2d702e7b9d8abcb7d455b36e1f8ca9731 Mon Sep 17 00:00:00 2001 From: "release-github-action[bot]" <154503118+release-github-action[bot]@users.noreply.github.com> Date: Mon, 8 Jul 2024 10:05:48 +0200 Subject: [PATCH] Update github-actions to 4.14.12 (#166) * Don't activate modules for components * Fix field renames in component template * Fix cache handling for ee/pe * Handle Tests and tests folders in run_test_script * Pass github token to script in run_test_script --- .github/oxid-esales/defaults/7.0.x.yaml | 1 - .github/oxid-esales/defaults/7.0.x.yml | 1 - .github/oxid-esales/defaults/7.1.x.yaml | 1 - .github/oxid-esales/defaults/7.1.x.yml | 1 - .github/oxid-esales/defaults/7.2.x.yaml | 1 - .github/oxid-esales/defaults/7.2.x.yml | 1 - .github/oxid-esales/defaults/component.yaml | 9 +++- .github/oxid-esales/defaults/component.yml | 9 +++- .github/oxid-esales/defaults/ee.yaml | 11 ++++- .github/oxid-esales/defaults/ee.yml | 11 ++++- .github/oxid-esales/defaults/pe.yaml | 11 ++++- .github/oxid-esales/defaults/pe.yml | 11 ++++- .github/oxid-esales/defaults/scheduled.yaml | 1 + .github/oxid-esales/defaults/scheduled.yml | 1 + .../workflows/universal_workflow_light.yaml | 5 ++- .../workflows/universal_workflow_light.yml | 5 ++- run_test_script/action.yaml | 44 +++++++++++++++---- 17 files changed, 93 insertions(+), 31 deletions(-) diff --git a/.github/oxid-esales/defaults/7.0.x.yaml b/.github/oxid-esales/defaults/7.0.x.yaml index 50c60840..8ec2f71b 100644 --- a/.github/oxid-esales/defaults/7.0.x.yaml +++ b/.github/oxid-esales/defaults/7.0.x.yaml @@ -9,7 +9,6 @@ workflow: global: git: &git default_ref: &default_ref 'b-7.0.x' - safe_ref_name: 'b-7.0.x' enterprise_ref: 'b-7.0.x' composer: &composer diff --git a/.github/oxid-esales/defaults/7.0.x.yml b/.github/oxid-esales/defaults/7.0.x.yml index 50c60840..8ec2f71b 100644 --- a/.github/oxid-esales/defaults/7.0.x.yml +++ b/.github/oxid-esales/defaults/7.0.x.yml @@ -9,7 +9,6 @@ workflow: global: git: &git default_ref: &default_ref 'b-7.0.x' - safe_ref_name: 'b-7.0.x' enterprise_ref: 'b-7.0.x' composer: &composer diff --git a/.github/oxid-esales/defaults/7.1.x.yaml b/.github/oxid-esales/defaults/7.1.x.yaml index e50ea10a..d2615db0 100644 --- a/.github/oxid-esales/defaults/7.1.x.yaml +++ b/.github/oxid-esales/defaults/7.1.x.yaml @@ -8,7 +8,6 @@ workflow: global: git: &git default_ref: &default_ref 'b-7.1.x' - safe_ref_name: 'b-7.1.x' enterprise_ref: 'b-7.1.x' composer: &composer diff --git a/.github/oxid-esales/defaults/7.1.x.yml b/.github/oxid-esales/defaults/7.1.x.yml index e50ea10a..d2615db0 100644 --- a/.github/oxid-esales/defaults/7.1.x.yml +++ b/.github/oxid-esales/defaults/7.1.x.yml @@ -8,7 +8,6 @@ workflow: global: git: &git default_ref: &default_ref 'b-7.1.x' - safe_ref_name: 'b-7.1.x' enterprise_ref: 'b-7.1.x' composer: &composer diff --git a/.github/oxid-esales/defaults/7.2.x.yaml b/.github/oxid-esales/defaults/7.2.x.yaml index 0dc7455c..aefa1269 100644 --- a/.github/oxid-esales/defaults/7.2.x.yaml +++ b/.github/oxid-esales/defaults/7.2.x.yaml @@ -9,7 +9,6 @@ workflow: global: git: &git default_ref: &default_ref 'b-7.2.x' - safe_ref_name: 'b-7.2.x' enterprise_ref: 'b-7.2.x' composer: &composer diff --git a/.github/oxid-esales/defaults/7.2.x.yml b/.github/oxid-esales/defaults/7.2.x.yml index 0dc7455c..aefa1269 100644 --- a/.github/oxid-esales/defaults/7.2.x.yml +++ b/.github/oxid-esales/defaults/7.2.x.yml @@ -9,7 +9,6 @@ workflow: global: git: &git default_ref: &default_ref 'b-7.2.x' - safe_ref_name: 'b-7.2.x' enterprise_ref: 'b-7.2.x' composer: &composer diff --git a/.github/oxid-esales/defaults/component.yaml b/.github/oxid-esales/defaults/component.yaml index 4733b287..e123923c 100644 --- a/.github/oxid-esales/defaults/component.yaml +++ b/.github/oxid-esales/defaults/component.yaml @@ -1,9 +1,14 @@ install_shop_with_modules: + method: legacy cache: prepared_shop: false git: + # Deprecated, remove in v5 shop_url: 'https://github.com/{{ .Data.global.repo }}.git' - shop_ref: '{{ .Data.global.git.safe_ref_name }}' + # Deprecated, remove in v5 + shop_ref: '{{ .Data.install_shop_with_modules.git.safe_ref_name }}' + repository: '{{ .Data.global.repo }}' + ref: '{{ .Data.install_shop_with_modules.git.safe_ref_name }}' composer: root_url: '' transform: '' @@ -38,4 +43,4 @@ runslim: <<: *runscript finish: - slack_title: '{{ print .Data.global.name}} ({{ .Data.global.git.shop_ref }}) by {{ .Github.Actor }}' + slack_title: '{{ print .Data.global.name}} ({{ .Data.global.git.safe_ref_name }}) by {{ .Github.Actor }}' diff --git a/.github/oxid-esales/defaults/component.yml b/.github/oxid-esales/defaults/component.yml index 4733b287..e123923c 100644 --- a/.github/oxid-esales/defaults/component.yml +++ b/.github/oxid-esales/defaults/component.yml @@ -1,9 +1,14 @@ install_shop_with_modules: + method: legacy cache: prepared_shop: false git: + # Deprecated, remove in v5 shop_url: 'https://github.com/{{ .Data.global.repo }}.git' - shop_ref: '{{ .Data.global.git.safe_ref_name }}' + # Deprecated, remove in v5 + shop_ref: '{{ .Data.install_shop_with_modules.git.safe_ref_name }}' + repository: '{{ .Data.global.repo }}' + ref: '{{ .Data.install_shop_with_modules.git.safe_ref_name }}' composer: root_url: '' transform: '' @@ -38,4 +43,4 @@ runslim: <<: *runscript finish: - slack_title: '{{ print .Data.global.name}} ({{ .Data.global.git.shop_ref }}) by {{ .Github.Actor }}' + slack_title: '{{ print .Data.global.name}} ({{ .Data.global.git.safe_ref_name }}) by {{ .Github.Actor }}' diff --git a/.github/oxid-esales/defaults/ee.yaml b/.github/oxid-esales/defaults/ee.yaml index b997e01b..45223ae2 100644 --- a/.github/oxid-esales/defaults/ee.yaml +++ b/.github/oxid-esales/defaults/ee.yaml @@ -66,7 +66,8 @@ install_shop_with_modules: &install is_enterprise: *is_enterprise cache: # Needs to be here as $shop_type is different - prefix: 'preparedShop-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prepared_shop_prefix: 'preparedShop-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prefix: &install_shop_prefix 'shopInstallation-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' # BEGIN Deprecated, will be superseded by install: prepare_shop: *install @@ -76,7 +77,7 @@ install_shop: git: *git is_enterprise: *is_enterprise cache: - prefix: &install_shop_prefix 'shopInstallation-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prefix: *install_shop_prefix install_module: cache: @@ -140,6 +141,12 @@ runtest: coverage: path: '' +runscript: + load_shop: *install_shop_prefix + +runslim: + load_shop: *install_shop_prefix + sonarcloud: matrix: testplan: 'skip' diff --git a/.github/oxid-esales/defaults/ee.yml b/.github/oxid-esales/defaults/ee.yml index b997e01b..45223ae2 100644 --- a/.github/oxid-esales/defaults/ee.yml +++ b/.github/oxid-esales/defaults/ee.yml @@ -66,7 +66,8 @@ install_shop_with_modules: &install is_enterprise: *is_enterprise cache: # Needs to be here as $shop_type is different - prefix: 'preparedShop-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prepared_shop_prefix: 'preparedShop-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prefix: &install_shop_prefix 'shopInstallation-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' # BEGIN Deprecated, will be superseded by install: prepare_shop: *install @@ -76,7 +77,7 @@ install_shop: git: *git is_enterprise: *is_enterprise cache: - prefix: &install_shop_prefix 'shopInstallation-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prefix: *install_shop_prefix install_module: cache: @@ -140,6 +141,12 @@ runtest: coverage: path: '' +runscript: + load_shop: *install_shop_prefix + +runslim: + load_shop: *install_shop_prefix + sonarcloud: matrix: testplan: 'skip' diff --git a/.github/oxid-esales/defaults/pe.yaml b/.github/oxid-esales/defaults/pe.yaml index 1a35190e..f84ec845 100644 --- a/.github/oxid-esales/defaults/pe.yaml +++ b/.github/oxid-esales/defaults/pe.yaml @@ -56,7 +56,8 @@ install_shop_with_modules: &install is_enterprise: *is_enterprise cache: # Needs to be here as $shop_type is different - prefix: 'preparedShop-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prepared_shop_prefix: 'preparedShop-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prefix: &install_shop_prefix 'shopInstallation-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' # BEGIN Deprecated, will be superseded by install: prepare_shop: *install @@ -66,7 +67,7 @@ install_shop: git: *git is_enterprise: *is_enterprise cache: - prefix: &install_shop_prefix 'shopInstallation-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prefix: *install_shop_prefix install_module: cache: @@ -124,6 +125,12 @@ runtest: coverage: path: '' +runscript: + load_shop: *install_shop_prefix + +runslim: + load_shop: *install_shop_prefix + sonarcloud: matrix: testplan: 'skip' diff --git a/.github/oxid-esales/defaults/pe.yml b/.github/oxid-esales/defaults/pe.yml index 1a35190e..f84ec845 100644 --- a/.github/oxid-esales/defaults/pe.yml +++ b/.github/oxid-esales/defaults/pe.yml @@ -56,7 +56,8 @@ install_shop_with_modules: &install is_enterprise: *is_enterprise cache: # Needs to be here as $shop_type is different - prefix: 'preparedShop-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prepared_shop_prefix: 'preparedShop-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prefix: &install_shop_prefix 'shopInstallation-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' # BEGIN Deprecated, will be superseded by install: prepare_shop: *install @@ -66,7 +67,7 @@ install_shop: git: *git is_enterprise: *is_enterprise cache: - prefix: &install_shop_prefix 'shopInstallation-{{ $shop_type }}-{{ .Github.SHA }}-{{ .Github.RunID }}' + prefix: *install_shop_prefix install_module: cache: @@ -124,6 +125,12 @@ runtest: coverage: path: '' +runscript: + load_shop: *install_shop_prefix + +runslim: + load_shop: *install_shop_prefix + sonarcloud: matrix: testplan: 'skip' diff --git a/.github/oxid-esales/defaults/scheduled.yaml b/.github/oxid-esales/defaults/scheduled.yaml index ad1f0f53..a3ed13d9 100644 --- a/.github/oxid-esales/defaults/scheduled.yaml +++ b/.github/oxid-esales/defaults/scheduled.yaml @@ -8,6 +8,7 @@ global: shop_ref: '{{ .Data.global.git.default_ref }}' ref: '{{ .Data.global.git.default_ref }}' enterprise_ref: '{{ .Data.global.git.default_ref }}' + safe_ref_name: '{{ .Data.global.git.default_ref }}' composer: &composer ref_name: 'dev-{{ .Data.global.git.default_ref }}' diff --git a/.github/oxid-esales/defaults/scheduled.yml b/.github/oxid-esales/defaults/scheduled.yml index ad1f0f53..a3ed13d9 100644 --- a/.github/oxid-esales/defaults/scheduled.yml +++ b/.github/oxid-esales/defaults/scheduled.yml @@ -8,6 +8,7 @@ global: shop_ref: '{{ .Data.global.git.default_ref }}' ref: '{{ .Data.global.git.default_ref }}' enterprise_ref: '{{ .Data.global.git.default_ref }}' + safe_ref_name: '{{ .Data.global.git.default_ref }}' composer: &composer ref_name: 'dev-{{ .Data.global.git.default_ref }}' diff --git a/.github/workflows/universal_workflow_light.yaml b/.github/workflows/universal_workflow_light.yaml index 40ab788f..a70d4ace 100644 --- a/.github/workflows/universal_workflow_light.yaml +++ b/.github/workflows/universal_workflow_light.yaml @@ -501,7 +501,7 @@ jobs: debug: ${{ needs.init.outputs.debug }} - name: Cache preparedShop on s3 - if: ${{ steps.iltp.outputs.install_cache_prepared_shop == 'true' && steps.iltp.outputs.use_private_cache == 'true' }} + if: ${{ steps.iltp.outputs.install_cache_prepared_shop == 'true' && needs.init.outputs.use_private_cache == 'true' }} uses: tespkg/actions-cache/save@v1 with: path: | @@ -513,7 +513,7 @@ jobs: bucket: ${{ steps.iltp.outputs.install_cache_bucket }} - name: Cache preparedShop on github - if: ${{ steps.iltp.outputs.install_cache_prepared_shop == 'true' && steps.iltp.outputs.use_private_cache != 'true'}} + if: ${{ steps.iltp.outputs.install_cache_prepared_shop == 'true' && needs.init.outputs.use_private_cache != 'true'}} uses: actions/cache/save@v4 with: path: | @@ -595,6 +595,7 @@ jobs: fi - name: Activate modules + if: ${{ steps.iltp.outputs.install_skip_shop_installation == 'false' }} shell: bash run: | # Activate modules diff --git a/.github/workflows/universal_workflow_light.yml b/.github/workflows/universal_workflow_light.yml index 40ab788f..a70d4ace 100644 --- a/.github/workflows/universal_workflow_light.yml +++ b/.github/workflows/universal_workflow_light.yml @@ -501,7 +501,7 @@ jobs: debug: ${{ needs.init.outputs.debug }} - name: Cache preparedShop on s3 - if: ${{ steps.iltp.outputs.install_cache_prepared_shop == 'true' && steps.iltp.outputs.use_private_cache == 'true' }} + if: ${{ steps.iltp.outputs.install_cache_prepared_shop == 'true' && needs.init.outputs.use_private_cache == 'true' }} uses: tespkg/actions-cache/save@v1 with: path: | @@ -513,7 +513,7 @@ jobs: bucket: ${{ steps.iltp.outputs.install_cache_bucket }} - name: Cache preparedShop on github - if: ${{ steps.iltp.outputs.install_cache_prepared_shop == 'true' && steps.iltp.outputs.use_private_cache != 'true'}} + if: ${{ steps.iltp.outputs.install_cache_prepared_shop == 'true' && needs.init.outputs.use_private_cache != 'true'}} uses: actions/cache/save@v4 with: path: | @@ -595,6 +595,7 @@ jobs: fi - name: Activate modules + if: ${{ steps.iltp.outputs.install_skip_shop_installation == 'false' }} shell: bash run: | # Activate modules diff --git a/run_test_script/action.yaml b/run_test_script/action.yaml index 17f4e3e1..7bb49a00 100644 --- a/run_test_script/action.yaml +++ b/run_test_script/action.yaml @@ -59,9 +59,18 @@ runs: run: | # run_test_script: Run composer script ${{ inputs.script }} ${{ inputs.debug }} - mkdir -p source/${{ inputs.path }}/tests/Output source/${{ inputs.path }}/tests/Reports + TESTDIR='tests' + if [ ! -d "source/${{ inputs.path }}/${TESTDIR}" ]; then + TESTDIR='Tests' + if [ ! -d "source/${{ inputs.path }}/${TESTDIR}" ]; then + echo -e "\033[0;31m### Could not find folder tests or Tests in ${{ inputs.path }} ###\033[0m" + exit 1 + fi + fi + mkdir -p source/${{ inputs.path }}/${TESTDIR}/Output source/${{ inputs.path }}/${TESTDIR}/Reports echo -e "\033[0;35m### Run composer ${{ inputs.script }} in ${{ inputs.path }} ###\033[0m" docker compose ${{ inputs.container_method }} -T \ + -e GITHUB_TOKEN=${{ inputs.github_token }} \ ${{ inputs.container_options }} \ --workdir '/var/www/${{ inputs.path }}' \ ${{ inputs.container_name }} \ @@ -73,10 +82,18 @@ runs: run: | # run_test_script: Run shell script ${{ inputs.script }} ${{ inputs.debug }} - mkdir -p source/${{ inputs.path }}/tests/Output source/${{ inputs.path }}/tests/Reports - SCRIPT=$(echo '${{ inputs.script }}'|sed -e "s|^~/|tests/Scripts/|") + TESTDIR='tests' + if [ ! -d "source/${{ inputs.path }}/${TESTDIR}" ]; then + TESTDIR='Tests' + if [ ! -d "source/${{ inputs.path }}/${TESTDIR}" ]; then + echo -e "\033[0;31m### Could not find folder tests or Tests in ${{ inputs.path }} ###\033[0m" + exit 1 + fi + fi + mkdir -p "source/${{ inputs.path }}/${TESTDIR}/Output" "source/${{ inputs.path }}/${TESTDIR}/Reports" + SCRIPT=$(echo '${{ inputs.script }}'|sed -e "s|^~/|${TESTDIR}/Scripts/|") if [ ! -f "source/${{ inputs.path }}/${SCRIPT}" ]; then - SCRIPT=$(echo '${{ inputs.script }}'|sed -e "s|^~/|tests/scripts/|") + SCRIPT=$(echo '${{ inputs.script }}'|sed -e "s|^~/|${TESTDIR}/scripts/|") fi if [ ! -f "source/${{ inputs.path }}/${SCRIPT}" ]; then echo -e "\033[0;31m### Could not find source/${{ inputs.path }}/${SCRIPT} ###\033[0m" @@ -84,6 +101,7 @@ runs: chmod a+x "source/${{ inputs.path }}/${SCRIPT}" echo -e "\033[0;35m### Run ${SCRIPT} in ${{ inputs.path }} ###\033[0m" docker compose ${{ inputs.container_method }} -T \ + -e GITHUB_TOKEN=${{ inputs.github_token }} \ ${{ inputs.container_options }} \ --workdir '/var/www/${{ inputs.path }}' \ ${{ inputs.container_name }} \ @@ -95,18 +113,26 @@ runs: run: | # run_test_script: Output php error log ${{ inputs.debug }} + TESTDIR='tests' + if [ ! -d "source/${{ inputs.path }}/${TESTDIR}" ]; then + TESTDIR='Tests' + if [ ! -d "source/${{ inputs.path }}/${TESTDIR}" ]; then + echo -e "\033[0;31m### Could not find folder tests or Tests in ${{ inputs.path }} ###\033[0m" + exit 1 + fi + fi if [ -s data/php/logs/error_log.txt ]; then echo -e "\033[0;35mPHP error log\033[0m" cat data/php/logs/error_log.txt fi touch data/php/logs/error_log.txt - cp data/php/logs/error_log.txt source/${{ inputs.path }}/tests/Output/ - cp data/mysql/mysql/slow_log* source/${{ inputs.path }}/tests/Output/ - cp data/mysql/mysql/general_log* source/${{ inputs.path }}/tests/Output/ - docker compose logs --tail=all >source/${{ inputs.path }}/tests/Output/docker-compose.log + cp data/php/logs/error_log.txt source/${{ inputs.path }}/${TESTDIR}/Output/ + cp data/mysql/mysql/slow_log* source/${{ inputs.path }}/${TESTDIR}/Output/ + cp data/mysql/mysql/general_log* source/${{ inputs.path }}/${TESTDIR}/Output/ + docker compose logs --tail=all >source/${{ inputs.path }}/${TESTDIR}/Output/docker-compose.log for c in $(docker ps -aq); do echo -e "\033[0;35m### Logs for $c ###\033[0m" - docker logs -n 2000 $c >source/${{ inputs.path }}/tests/Output/docker-$c.log + docker logs -n 2000 $c >source/${{ inputs.path }}/${TESTDIR}/Output/docker-$c.log done - name: Upload artifact