Skip to content

CI: Add Rocky8 workflow with test matrix support#1518

Merged
tuhaihe merged 4 commits into
apache:mainfrom
tuhaihe:rocky8-ci-weekly
Feb 3, 2026
Merged

CI: Add Rocky8 workflow with test matrix support#1518
tuhaihe merged 4 commits into
apache:mainfrom
tuhaihe:rocky8-ci-weekly

Conversation

@tuhaihe

@tuhaihe tuhaihe commented Dec 31, 2025

Copy link
Copy Markdown
Member

This commit introduces a new GitHub Actions workflow for building and testing Apache Cloudberry on Rocky Linux 8, enabling automated builds, RPM packaging, and regresssion testing alongside the existing Rocky 9 and Ubuntu 22.04 pipelines.

Triggers:

  • Push to main branch
  • Pull requests modifying this workflow file
  • Scheduled: Every Monday at 02:00 UTC
  • Manual workflow dispatch with optional test selection

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


@tuhaihe

tuhaihe commented Dec 31, 2025

Copy link
Copy Markdown
Member Author

The CI failed due to #1441 - The PR #1513 will resolve it. So need to rebase this PR once #1513 is merged.

@tuhaihe

tuhaihe commented Jan 7, 2026

Copy link
Copy Markdown
Member Author

Will rebase after #1521

@tuhaihe
tuhaihe force-pushed the rocky8-ci-weekly branch 2 times, most recently from efd48b7 to d17809c Compare January 13, 2026 07:48
@tuhaihe
tuhaihe marked this pull request as draft January 15, 2026 10:24
@tuhaihe
tuhaihe marked this pull request as ready for review January 20, 2026 07:15
@tuhaihe

tuhaihe commented Jan 20, 2026

Copy link
Copy Markdown
Member Author

Fix #1538 #1539

@lss602726449

Copy link
Copy Markdown
Contributor

LGTM.

@leborchuk leborchuk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks like Rocky9 tests, I even checked if we could use template as github actions yml configuration and change only container and on sections. But there seem to be no good solutions. So, let's copy the configuration.

@leborchuk

Copy link
Copy Markdown
Contributor

Contains pxf_fdw, so need to be rebased and fixed after merging #1549

This commit introduces a new GitHub Actions workflow for building and
testing Apache Cloudberry on Rocky Linux 8, enabling automated builds,
RPM packaging, and regresssion testing alongside the existing Rocky 9
and Ubuntu 22.04 pipelines.

Triggers:
- Push to main branch
- Pull requests modifying this workflow file
- Scheduled: Every Monday at 02:00 UTC
- Manual workflow dispatch with optional test selection
The `capture_output` argument for `subprocess.run` is not available
in Python 3.6, which is the default on Rocky Linux 8.

This commit replaces it with `stdout=subprocess.PIPE` and `stderr=
subprocess.PIPE` to ensure backward compatibility.

This enables `ic-good-opt-*` tests to pass on Rocky 8 environments.

See: apache#1538
This commit fixes two issues preventing the `ic-contrib` regression
tests from passing on Rocky Linux 8 environments (FIPS enabled).

1. Fixed regex matching for "Some PX error": The previous `init_file`
   rule missed a space in the error message pattern (`ERROR:  Cannot
   use...`), causing FIPS error masking to fail. Added the missing space
   to correctly match the output.

2. Masked ephemeral line numbers: Different compilation environments
   (Rocky 8 vs 9) produce different line number references in error
   messages (e.g., pgcrypto.c:213 vs 215).  Added `matchsubs` rules to
   mask these line numbers with `(pgcrypto.c:XXX)` and updated
   `expected/fips_2.out` to match, ensuring consistent test results
   across platforms.

Changes:
* Modified src/test/regress/init_file
* Updated contrib/pgcrypto/expected/fips_2.out

See: apache#1539
Similar to PR apache#1549, we will keep the pxf_fdw in `apache/cloudberry-pxf`
as the latest version, so remove pxf_fdw test from this CI workflow
file.
@tuhaihe

tuhaihe commented Feb 3, 2026

Copy link
Copy Markdown
Member Author

All looks like Rocky9 tests, I even checked if we could use template as github actions yml configuration and change only container and on sections. But there seem to be no good solutions. So, let's copy the configuration.

Yes. If we want to make the new Rocky 8 test scheduled, not triggered by the daily PRs. This way can bring us more flexibility.

@tuhaihe
tuhaihe merged commit eeff10a into apache:main Feb 3, 2026
62 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.

4 participants