From c6babea1fdcc4a4aa8cd503a490f8691bf30d993 Mon Sep 17 00:00:00 2001 From: agrasth Date: Sun, 21 Jun 2026 02:36:49 +0530 Subject: [PATCH] Add timestamp suffix to Nix repos and build name for test isolation NixLocalRepo, NixRemoteRepo, NixVirtualRepo and NixBuildName were missing from AddTimestampToGlobalVars, so every Nix integration test run used the literal names cli-nix-local / cli-nix-remote / cli-nix-virtual / cli-nix-build while every other package manager's repos and build name receive the unique "--" suffix. This is invisible in CI here because each job runs against its own ephemeral local Artifactory. But when multiple Nix test runs execute concurrently against a shared Artifactory instance, InitBuildToolsTests -> cleanUpOldRepositories deletes another run's just-created cli-nix-* repositories, so initNixTest's isRepoExist checks fail with "Nix test virtual repository doesn't exist". Adding the suffix makes each run's Nix repos and build name unique, matching how every other package manager is already handled (mirrors the earlier UvBuildName fix). --- utils/tests/utils.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/utils/tests/utils.go b/utils/tests/utils.go index deb2101ef..569243a2d 100644 --- a/utils/tests/utils.go +++ b/utils/tests/utils.go @@ -587,6 +587,9 @@ func AddTimestampToGlobalVars() { UvLocalRepo += uniqueSuffix UvRemoteRepo += uniqueSuffix UvVirtualRepo += uniqueSuffix + NixLocalRepo += uniqueSuffix + NixRemoteRepo += uniqueSuffix + NixVirtualRepo += uniqueSuffix ConanLocalRepo += uniqueSuffix ConanRemoteRepo += uniqueSuffix ConanVirtualRepo += uniqueSuffix @@ -619,6 +622,7 @@ func AddTimestampToGlobalVars() { PipenvBuildName += uniqueSuffix PoetryBuildName += uniqueSuffix UvBuildName += uniqueSuffix + NixBuildName += uniqueSuffix ConanBuildName += uniqueSuffix HelmBuildName += uniqueSuffix HuggingFaceBuildName += uniqueSuffix