Skip to content

Commit e1a095d

Browse files
mo-getterclaude
andcommitted
ci(testing-only): skip helm tests, publish pre-release charts anyway
Force-skip helm-unit and helm-integration and let helm-pre-release accept skipped results so every PR push still publishes the fiftyone-teams-app sha chart for ephemeral environment testing. DO NOT MERGE — revert before this branch lands in main. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 415af24 commit e1a095d

1 file changed

Lines changed: 9 additions & 5 deletions

File tree

.github/workflows/pr.yml

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -58,15 +58,17 @@ jobs:
5858

5959
helm-integration:
6060
needs: modified-files
61-
if: ${{ needs.modified-files.outputs.helm-integration-required == 'true' }}
61+
# TESTING ONLY — do not merge: force-skip so charts publish without tests
62+
if: ${{ needs.modified-files.outputs.helm-integration-required == 'never' }}
6263
uses: ./.github/workflows/test-integration-helm.yml
6364
secrets:
6465
ORG_RO_ALOHA_GITHUB_WORKFLOWS_FG_PAT: ${{ secrets.ORG_RO_ALOHA_GITHUB_WORKFLOWS_FG_PAT }}
6566
REPO_GOOGLE_WORKLOAD_IDP: ${{ secrets.REPO_GOOGLE_WORKLOAD_IDP }}
6667

6768
helm-unit:
6869
needs: modified-files
69-
if: ${{ needs.modified-files.outputs.helm-unit-required == 'true' }}
70+
# TESTING ONLY — do not merge: force-skip so charts publish without tests
71+
if: ${{ needs.modified-files.outputs.helm-unit-required == 'never' }}
7072
uses: ./.github/workflows/test-helm.yml
7173
secrets:
7274
ORG_RO_ALOHA_GITHUB_WORKFLOWS_FG_PAT: ${{ secrets.ORG_RO_ALOHA_GITHUB_WORKFLOWS_FG_PAT }}
@@ -88,11 +90,13 @@ jobs:
8890

8991
helm-pre-release:
9092
needs: [all-tests, helm-unit, helm-integration, validate-chart-version]
93+
# TESTING ONLY — do not merge: accept skipped helm tests so the chart
94+
# still publishes
9195
if: |
9296
always() &&
93-
(needs.helm-unit.result == 'success' &&
94-
needs.helm-integration.result == 'success' &&
95-
needs.validate-chart-version.result == 'success')
97+
(needs.helm-unit.result == 'success' || needs.helm-unit.result == 'skipped') &&
98+
(needs.helm-integration.result == 'success' || needs.helm-integration.result == 'skipped') &&
99+
(needs.validate-chart-version.result == 'success')
96100
uses: ./.github/workflows/pre-release-internal-env.yml
97101
secrets:
98102
REPO_GOOGLE_WORKLOAD_IDP: ${{ secrets.REPO_GOOGLE_WORKLOAD_IDP }}

0 commit comments

Comments
 (0)