Skip to content

Fix required CI checks stuck on PRs touching openmetadata-service#25937

Merged
manerow merged 1 commit intomainfrom
fix/ci-required-checks-stuck
Feb 17, 2026
Merged

Fix required CI checks stuck on PRs touching openmetadata-service#25937
manerow merged 1 commit intomainfrom
fix/ci-required-checks-stuck

Conversation

@manerow
Copy link
Copy Markdown
Contributor

@manerow manerow commented Feb 17, 2026

Summary

Two branch protection required checks (maven-postgresql-ci and py-run-tests (3.11)) were permanently stuck as "Waiting for status to be reported" on PRs modifying openmetadata-service/**, blocking merges.

The repo uses a skip/run workflow pattern where a "real" workflow runs on relevant path changes and a "skip" workflow reports a passing no-op status otherwise. The gap:

  • maven-postgresql-ci: The real workflow was disabled (workflow_dispatch only), but the skip workflow still had paths-ignore excluding service paths. Neither workflow reported the check. Fix: remove paths-ignore so the skip workflow always runs.
  • py-run-tests (3.11): The real workflow only had py-version: ['3.10'] in its matrix, so it never produced a 3.11 check. The skip workflow (which includes 3.11) was excluded by paths-ignore. Fix: add '3.11' to the real workflow's matrix.

@manerow manerow self-assigned this Feb 17, 2026
@github-actions github-actions Bot added backend safe to test Add this label to run secure Github workflows on PRs labels Feb 17, 2026
@gitar-bot
Copy link
Copy Markdown

gitar-bot Bot commented Feb 17, 2026

Code Review ✅ Approved

Clean CI configuration fix. Removing paths-ignore from the maven skip workflow and adding Python 3.11 to the real py-tests matrix correctly ensures both required checks always report status for PRs touching openmetadata-service/**.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@manerow manerow merged commit c408db6 into main Feb 17, 2026
20 of 21 checks passed
@manerow manerow deleted the fix/ci-required-checks-stuck branch February 17, 2026 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend safe to test Add this label to run secure Github workflows on PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants