Skip to content

ci: group behavior test setups by service - #8017

Merged
erickguan merged 8 commits into
apache:mainfrom
rohankumardubey:optimize-behavior-ci
Aug 9, 2026
Merged

ci: group behavior test setups by service#8017
erickguan merged 8 commits into
apache:mainfrom
rohankumardubey:optimize-behavior-ci

Conversation

@rohankumardubey

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #7919.

Rationale for this change

Behavior-test configurations currently run as separate jobs, exceeding ASF concurrency guidelines. Grouping configurations by service reduces concurrent jobs while preserving coverage.

What changes are included in this PR?

  • Group core behavior-test setups by service.
  • Run grouped setups sequentially in one job.
  • Isolate Docker resources and environment variables between setups.
  • Preserve Windows filesystem coverage.
  • Add planner and testkit regression tests.

Are there any user-facing changes?

No. This change only affects CI behavior tests.

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" labels Aug 3, 2026
@erickguan

Copy link
Copy Markdown
Member

Nice progress! Can we also group a related test?

image

e.g.,

  • 0_minio_s3
  • minio_s3_with_annoymous

create a group so we can toggle them?

@rohankumardubey

Copy link
Copy Markdown
Contributor Author

@Xuanwo @tisonkun can you please review this.

@erickguan

Copy link
Copy Markdown
Member

Hi @rohankumardubey,

Thank you for working on this. Can you resolve the failure from redis? Perhaps you can try to list the folder or print contents in the folder in a debug block?

@rohankumardubey

Copy link
Copy Markdown
Contributor Author

Hi @rohankumardubey,

Thank you for working on this. Can you resolve the failure from redis? Perhaps you can try to list the folder or print contents in the folder in a debug block?

@erickguan Thanks for the pointer. The Redis TLS setup ran a nested checkout, which cleaned the workspace and deleted the remaining generated test actions. I removed the redundant checkout from both Redis TLS setups and added a regression test to prevent this from recurring.

@erickguan erickguan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for improving OpenDAL's CI!

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 9, 2026
@erickguan
erickguan merged commit 3c43f24 into apache:main Aug 9, 2026
137 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI optimization: run multiple behavior test jobs in a workflow run

2 participants