Skip to content

CI: fix 'Check and Display Regression Diffs' step to use bash shell#1673

Merged
yjhjstz merged 2 commits intoapache:mainfrom
yjhjstz:fix_ci
Apr 10, 2026
Merged

CI: fix 'Check and Display Regression Diffs' step to use bash shell#1673
yjhjstz merged 2 commits intoapache:mainfrom
yjhjstz:fix_ci

Conversation

@yjhjstz
Copy link
Copy Markdown
Member

@yjhjstz yjhjstz commented Apr 10, 2026

The step used bash-specific [[ ]] syntax but lacked shell: bash {0}, causing failures when the runner defaulted to sh.

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


The step used bash-specific [[ ]] syntax but lacked shell: bash {0},
causing failures when the runner defaulted to sh.
@tuhaihe
Copy link
Copy Markdown
Member

tuhaihe commented Apr 10, 2026

Hi, could you help handle the same issues in other files, like:

  • .github/workflows/build-cloudberry-rocky8.yml
  • .github/workflows/build-deb-cloudberry.yml

Thanks!

…orkflows

Same fix as build-cloudberry.yml: add shell: bash {0} to steps using
bash-specific [[ ]] syntax to prevent sh fallback failures.
@yjhjstz yjhjstz merged commit 90f9dc3 into apache:main Apr 10, 2026
67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants