Skip to content

Conversation

@dkruces
Copy link
Contributor

@dkruces dkruces commented Sep 26, 2025

Fixes error:

Run set -euxo pipefail

This consolidates docker-tests.yml and linux-ab.yml into a single
compatibility.yml workflow that builds kdevops containers once and
reuses them across all test jobs via GitHub Container Registry.
The unified approach eliminates duplicate container builds while
providing comprehensive validation across Debian, Fedora, and OpenSUSE
distributions.

The A/B testing script is modified to work in GitHub containers
by generating only the configuration files needed for validation
(extra_vars.yaml) instead of running the full make target that requires
systemd services unavailable in container environments.

Generated-by: Claude AI
Signed-off-by: Daniel Gomez <[email protected]>
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]>
@dkruces dkruces force-pushed the dagomez/test-ci-fix branch from 5987a51 to 3d13be4 Compare September 26, 2025 19:46
@dkruces dkruces closed this Sep 26, 2025
@dkruces dkruces deleted the dagomez/test-ci-fix branch September 26, 2025 21:24
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