Skip to content

Stable 25.8 - Port the Community PR workflow - #2270

Open
strtgbb wants to merge 5 commits into
Altinity:stable-25.8from
strtgbb:port/stable-25.8/community-pr-workflow
Open

Stable 25.8 - Port the Community PR workflow#2270
strtgbb wants to merge 5 commits into
Altinity:stable-25.8from
strtgbb:port/stable-25.8/community-pr-workflow

Conversation

@strtgbb

@strtgbb strtgbb commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Related: #2226

Changelog category (leave one):

  • CI Fix or Improvement (changelog entry is not required)

CI/CD Options

Exclude tests:

  • Fast test
  • Integration Tests
  • Stateless tests
  • Stateful tests
  • Performance tests
  • Aarch64 tests
  • All with ASAN
  • All with TSAN
  • All with MSAN
  • All with UBSAN
  • All with Coverage
  • All Regression
  • Disable CI Cache

Regression jobs to run:

  • Fast suites (mostly <1h)
  • Aggregate Functions (2h)
  • Alter (1.5h)
  • Benchmark (30m)
  • CAS (content-addressed storage; Antalya only)
  • ClickHouse Keeper (1h)
  • Iceberg (2h)
  • LDAP (1h)
  • OAuth (5m)
  • Parquet (1.5h)
  • RBAC (1.5h)
  • SSL Server (1h)
  • S3 (2h)
  • S3 Export (2h)
  • Swarms (30m)
  • Tiered Storage (2h)

@strtgbb strtgbb added stable cicd Improvements and fixes to the CICD process stable-25.8 labels Aug 26, 2026
strtgbb and others added 4 commits August 26, 2026 09:31
…dent.

Config Workflow failed because upload-artifact paths were baked as
absolute worktree paths; CI regenerates them under /home/ubuntu/_work.

Co-authored-by: Cursor <cursoragent@cursor.com>
Community PR has no secrets, so the unguarded step posted a dead `View Report` link. Match antalya-25.8.

Co-authored-by: Cursor <cursoragent@cursor.com>
Port of Altinity#2226 so fork contributors can see failure output without S3 access.

Co-authored-by: Cursor <cursoragent@cursor.com>
`if_condition` only skipped config; downstream `fromJson(needs.config_workflow.outputs.data)` then failed with empty input. Guard with `needs.config_workflow.result != 'skipped'` (26.6 uses `pipeline_status` for the same short-circuit).

https: //github.com/Altinity/ClickHouse/actions/runs/33010847269
https: //github.com/Altinity/pull/2270
Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cicd Improvements and fixes to the CICD process stable stable-25.8

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant