Skip to content

ci: read the automation commit email from an org variable - #636

Merged
ShuaiShao93 merged 1 commit into
mainfrom
ss/automation-email-cisco
Oct 2, 2026
Merged

ShuaiShao93 merged 1 commit into
mainfrom
ss/automation-email-cisco

Conversation

@ShuaiShao93

@ShuaiShao93 ShuaiShao93 commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

These workflows commit as galileo-automation <ci@rungalileo.io>. That address is gone after the Google Workspace migration. The verified email on the galileo-automation GitHub account is now splunk-ao-platform@cisco.com.

GitHub verifies an SSH-signed commit only when the committer email is verified on the signing account. So every signed commit from these workflows is now unverified. A push to a branch that requires signed commits then fails:

remote: - Commits must have verified signatures.

This PR reads the committer email from the org-wide Actions variable SAO_PLATFORM_AUTOMATION_EMAIL, set to splunk-ao-platform@cisco.com. A later email change then needs only the variable, not a PR in every repo. The signing key does not change. It is still a signing key on the account.

Test plan

  • After merge, start a new run of the failed workflow from main. Do not use "Re-run": it replays the old commit, which still has the old email.
  • Check that its push succeeds and the new commit shows as Verified.

🤖 Generated with Claude Code

@baz-reviewer

baz-reviewer Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Review this PR on Baz

Baz Summary

Update automated GitHub workflows to use the organization-level SAO_PLATFORM_AUTOMATION_EMAIL variable for commits and releases. Preserve the galileo-automation identity and SSH signing configuration while ensuring generated commits use the account’s verified email.

Latest Contributors(2)
UserCommitDate
ShuaiShao93ci: read the automatio...October 02, 2026
sankar@galileo.aifeat: harden a2a distr...April 14, 2026

Merger  Activate to get a short verdict whether this PR is good to go or not

Skills  Activate Skill Maintainer to keep your skills up to date

Planner  This PR would have been improved with Baz Planner - Try it now

@codecov

codecov Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.62%. Comparing base (a1a0821) to head (a5ad1f0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #636   +/-   ##
=======================================
  Coverage   83.62%   83.62%           
=======================================
  Files         124      124           
  Lines       11037    11037           
=======================================
  Hits         9230     9230           
  Misses       1807     1807           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ShuaiShao93 ShuaiShao93 changed the title ci: commit as splunk-ao-platform@cisco.com ci: read the automation commit email from an org variable Oct 2, 2026
@ShuaiShao93
ShuaiShao93 enabled auto-merge (squash) October 2, 2026 16:23
ci@rungalileo.io is no longer a verified email on the galileo-automation
account, so the signed commits these workflows push are unverified. Read
the committer email from the SAO_PLATFORM_AUTOMATION_EMAIL org variable.

Co-Authored-By: Claude Code <noreply@anthropic.com>
@ShuaiShao93
ShuaiShao93 force-pushed the ss/automation-email-cisco branch from f2f181d to a5ad1f0 Compare October 2, 2026 16:28
@ShuaiShao93
ShuaiShao93 merged commit dd2ac16 into main Oct 2, 2026
19 checks passed
@ShuaiShao93
ShuaiShao93 deleted the ss/automation-email-cisco branch October 2, 2026 16:46
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