Skip to content

fix: use unique artifact names to avoid collisions in matrix builds #5132

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
thomhurst merged 3 commits into from
Mar 11, 2026
Merged

fix: use unique artifact names to avoid collisions in matrix builds #5132

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
b768575
fix: use unique artifact names to avoid collisions in matrix builds
thomhurst Mar 11, 2026
709df33
refactor: only append job ID suffix on 409 conflict
thomhurst Mar 11, 2026
61a4930
style: use range syntax instead of Substring
thomhurst Mar 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 26 additions & 3 deletions TUnit.Engine/Reporters/Html/GitHubArtifactUploader.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,15 +31,19 @@ internal static class GitHubArtifactUploader
var origin = new Uri(resultsUrl).GetLeftPart(UriPartial.Authority);
var fileName = Path.GetFileName(filePath);

// Build a unique artifact name using the job backend ID to avoid collisions in matrix builds
var baseFileName = BuildUniqueArtifactName(fileName, workflowJobRunBackendId);

// Step 1: CreateArtifact (deduplicate name on 409 conflict)
var createUrl = $"{origin}/twirp/github.actions.results.api.v1.ArtifactService/CreateArtifact";
string? signedUploadUrl = null;
string? acceptedArtifactName = null;

for (var nameAttempt = 0; nameAttempt < 3 && signedUploadUrl is null; nameAttempt++)
{
var artifactName = nameAttempt == 0
? fileName
: $"{Path.GetFileNameWithoutExtension(fileName)}-{nameAttempt + 1}{Path.GetExtension(fileName)}";
? baseFileName
: $"{Path.GetFileNameWithoutExtension(baseFileName)}-{nameAttempt + 1}{Path.GetExtension(baseFileName)}";

var createBody = BuildCreateArtifactJson(workflowRunBackendId, workflowJobRunBackendId, artifactName);

Expand All @@ -66,6 +70,11 @@ internal static class GitHubArtifactUploader
using var doc = JsonDocument.Parse(json);
return doc.RootElement.GetProperty("signed_upload_url").GetString();
}, cancellationToken);

if (signedUploadUrl is not null)
{
acceptedArtifactName = artifactName;
}
}

if (signedUploadUrl is null)
Expand Down Expand Up @@ -105,7 +114,7 @@ internal static class GitHubArtifactUploader

// Step 3: FinalizeArtifact
var finalizeUrl = $"{origin}/twirp/github.actions.results.api.v1.ArtifactService/FinalizeArtifact";
var finalizeBody = BuildFinalizeArtifactJson(workflowRunBackendId, workflowJobRunBackendId, fileName, fileBytes.Length, sha256Hash);
var finalizeBody = BuildFinalizeArtifactJson(workflowRunBackendId, workflowJobRunBackendId, acceptedArtifactName!, fileBytes.Length, sha256Hash);

var artifactId = await RetryAsync(async () =>
{
Expand All @@ -127,6 +136,20 @@ internal static class GitHubArtifactUploader
return artifactId;
}

private static string BuildUniqueArtifactName(string fileName, string jobRunBackendId)
{
// Use the first 8 characters of the job backend ID as a short unique suffix.
// This ensures each matrix job gets a distinct artifact name without relying
// on environment variables that may not differentiate matrix combinations.
var shortId = jobRunBackendId.Length > 8
? jobRunBackendId.Substring(0, 8)
: jobRunBackendId;

var nameWithoutExt = Path.GetFileNameWithoutExtension(fileName);
var ext = Path.GetExtension(fileName);
return $"{nameWithoutExt}-{shortId}{ext}";
}

private static string BuildCreateArtifactJson(string runId, string jobId, string fileName)
{
using var ms = new MemoryStream();
Expand Down
Loading