Skip to content

Fix git committer identity error in update-dependencies pipeline#7013

Merged
lbussell merged 1 commit intodotnet:nightlyfrom
lbussell:update-deps-commit-identity
Jan 26, 2026
Merged

Fix git committer identity error in update-dependencies pipeline#7013
lbussell merged 1 commit intodotnet:nightlyfrom
lbussell:update-deps-commit-identity

Conversation

@lbussell
Copy link
Member

Summary

Fix "Committer identity unknown" error in the update-dependencies-internal pipeline by configuring git user.name and user.email on cloned repositories.

The pipeline was failing because DarcLib's CommitAsync only sets --author but Git also requires a committer identity to be configured. This issue surfaced after the recent change to Azure Linux 3 build agents, which don't have global git config set.

Changes

  • Add optional gitIdentity parameter to GitRepoHelperFactory.CreateAndCloneAsync()
  • Configure user.name and user.email on cloned repos when identity is provided
  • Pass committer identity when cloning in FromStagingPipelineCommand and SyncInternalReleaseCommand

Testing

@lbussell lbussell requested a review from a team as a code owner January 23, 2026 21:34
@lbussell lbussell force-pushed the update-deps-commit-identity branch from 0c81c42 to 3a56837 Compare January 23, 2026 21:39
@lbussell lbussell merged commit 70409aa into dotnet:nightly Jan 26, 2026
124 checks passed
@lbussell lbussell deleted the update-deps-commit-identity branch January 26, 2026 15:59
@github-actions github-actions bot added the needs-backport PR that should be backported to another branch label Jan 26, 2026
@lbussell
Copy link
Member Author

lbussell commented Feb 5, 2026

/backport to release/2026-02B

@lbussell lbussell removed the needs-backport PR that should be backported to another branch label Feb 5, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

Started backporting to release/2026-02B (link to workflow run)

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