From b262e6d278db2b7a5fb87a672b63cc99511952a6 Mon Sep 17 00:00:00 2001 From: michelle-clayton-work Date: Tue, 12 May 2026 15:16:48 -0700 Subject: [PATCH] Drop obsolete retain-build placeholder for go-lab#59 The TODO comment was a marker that 1ES PT migration removed Arcade's retain-build step. The replacement is now in microsoft/go-infra's release pipeline (release-build-steps.yml, after wait-ci-trigger), which discovers the auto-triggered microsoft-go build and pins it externally. The microsoft-go pipeline itself can't gate retention because CI-triggered runs don't carry release parameters; the release pipeline is the only caller that knows it's a release. See microsoft/go-infra#480. --- eng/pipeline/stages/run-stage.yml | 4 ---- 1 file changed, 4 deletions(-) diff --git a/eng/pipeline/stages/run-stage.yml b/eng/pipeline/stages/run-stage.yml index da30e5b5d2..cbf66ed390 100644 --- a/eng/pipeline/stages/run-stage.yml +++ b/eng/pipeline/stages/run-stage.yml @@ -301,10 +301,6 @@ stages: targetPath: $(Build.SourcesDirectory)/eng/artifacts/RawTestOutput artifactName: raw-test-output-${{ parameters.builder.id }}-retry-$(System.StageAttempt)-job-$(System.JobAttempt) - # - ${{ if eq(parameters.builder.config, 'buildandpack' ) }}: - # - ${{ if ne(parameters.releaseVersion, 'nil') }}: - # # Retain the build: https://github.com/microsoft/go-lab/issues/59 - - ${{ if eq(parameters.builder.os, 'linux') }}: # Files may be owned by root because builds don't set user ID. If this build is running on a # persistent machine, later builds may fail to clean up this build's directory as as