Skip to content

Ci commit fixes#41

Merged
dagomez137 merged 5 commits intomainfrom
ci-commit-fixes
Sep 29, 2025
Merged

Ci commit fixes#41
dagomez137 merged 5 commits intomainfrom
ci-commit-fixes

Conversation

@dagomez137
Copy link
Contributor

No description provided.

The archive role was ignoring enhanced commit messages generated by the
CI script and always using a hardcoded template instead. This caused
kdevops-ci and linux-ci runs to produce identical commit formats rather
than the intended distinct formats with proper PASS/FAIL status and
commit details.

Now the role checks for the enhanced format first and falls back to the
legacy template only when the enhanced version is unavailable.

Generated-by: Claude AI
Signed-off-by: Daniel Gomez <da.gomez@samsung.com>
The workflow was incorrectly assigning linux-ci jobs to kdevops-ci
runners and using inconsistent job names. This caused confusion about
which test mode was actually running and wasted resources on wrong
runner pools.

Now scheduled runs properly target linux-ci runners and show correct
job names, while kdevops-ci runs use their designated runners. The
test_mode parameter flows correctly to ensure proper commit message
formatting for each CI type.

Generated-by: Claude AI
Signed-off-by: Daniel Gomez <da.gomez@samsung.com>
Use the proper Makefile variable for the kdevops inventory file (hosts).

Fixes 0987c30 ("Makefile: use inventory from ansible.cfg")

Signed-off-by: Daniel Gomez <da.gomez@samsung.com>
Fix parallel git-config commands by making them local to the cloned repo
in the workflow run.

Fixes error:
+ git config --global --add safe.directory '*'
+ git config --global user.name kdevops
error: could not lock config file /home/gh/.gitconfig: File exists

Fixes: 4dd7684 ("github: refactor into reusable actions and workflows")
Signed-off-by: Daniel Gomez <da.gomez@samsung.com>
For push events, checkout the branch head instead of specific SHA

Signed-off-by: Daniel Gomez <da.gomez@samsung.com>
@dagomez137 dagomez137 merged commit 0f7daa6 into main Sep 29, 2025
26 of 30 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.

1 participant