Skip to content

HDDS-17708. Fork CI Status should mirror each check - #11941

Closed
adoroszlai wants to merge 4 commits into
apache:masterfrom
adoroszlai:HDDS-17708
Closed

adoroszlai wants to merge 4 commits into
apache:masterfrom
adoroszlai:HDDS-17708

Conversation

@adoroszlai

@adoroszlai adoroszlai commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Fork CI Status mirrors post-commit.yml, so status appears as a single check. This change drops post-commit.yml, moving triggers and concurrency settings to ci.yml, and changes Fork CI Status to mirror ci.yml. Hope it would help get back detailed checks in PR summary.

See also #11944.

https://issues.apache.org/jira/browse/HDDS-17708

How was this patch tested?

Suggestions on how to test this are welcome.

@adoroszlai adoroszlai added the CI label Nov 5, 2026
@adoroszlai adoroszlai self-assigned this Nov 5, 2026
@adoroszlai
adoroszlai marked this pull request as draft November 5, 2026 09:13
@adoroszlai

Copy link
Copy Markdown
Contributor Author

The reason we had separate post-commit.yml and ci.yml was to avoid the double-check shown in the screenshot. Maybe it can be avoided with HDDS-17509.

HDDS-17708-1

@adoroszlai

Copy link
Copy Markdown
Contributor Author

Instead of merging post-commit.yml into ci.yml, we might want to do the opposite to keep the workflow name and filename. Only workflows directly calling ci.yml would need to be updated.

@adoroszlai adoroszlai changed the title HDDS-17708. Fork CI Status should mirror ci.yml HDDS-17708. Fork CI Status should mirror each check Nov 5, 2026
@adoroszlai

Copy link
Copy Markdown
Contributor Author

Instead of merging post-commit.yml into ci.yml, we might want to do the opposite to keep the workflow name and filename. Only workflows directly calling ci.yml would need to be updated.

This is implemented in #11949.

@adoroszlai adoroszlai closed this Nov 6, 2026
@adoroszlai

Copy link
Copy Markdown
Contributor Author

Closed in favor of the other PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant