Skip to content

Conversation

@dkruces
Copy link
Contributor

@dkruces dkruces commented Sep 26, 2025

No description provided.

The unified workflow had two critical issues with repository checkout
on self-hosted runners. For pull requests, it attempted to fetch
non-existent merge refs like "35/merge". For push events, it used
incomplete HTTPS URLs missing the .git suffix.

This fixes both by using proper GitHub PR refs (refs/pull/N/head)
and complete HTTPS clone URLs with .git suffix for the public repository.

Fixes: 1b74004 ("github: unify self-hosted runner workflows")
Generated-by: Claude AI
Signed-off-by: Daniel Gomez <[email protected]>
This displays the kdevops commit being used right after checkout instead
of waiting until the end of the workflow. Users can now see which
version is running right away for validation.

Generated-by: Claude AI
Signed-off-by: Daniel Gomez <[email protected]>
This prevents wasteful duplicate CI execution that occurred when pushing
to a branch and creating a PR from it. Now only main branch pushes and
PRs to main trigger CI, with ci-testing/* branches available for direct
push-based workflow testing while blocking PRs from those branches.

Generated-by: Claude AI
Signed-off-by: Daniel Gomez <[email protected]>
@dkruces dkruces merged commit aa530c8 into main Sep 26, 2025
12 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.

2 participants