From a04ec347ac45db4077fd945c8f3835f31054e37d Mon Sep 17 00:00:00 2001 From: Mitch Denny Date: Fri, 13 Dec 2019 17:36:05 +1100 Subject: [PATCH 1/2] Removes the repo capture logic since it is relatively expensive. --- eng/pipelines/templates/jobs/archetype-sdk-client.yml | 5 ----- 1 file changed, 5 deletions(-) diff --git a/eng/pipelines/templates/jobs/archetype-sdk-client.yml b/eng/pipelines/templates/jobs/archetype-sdk-client.yml index 63b6a3427b2b..60bbd0b86469 100644 --- a/eng/pipelines/templates/jobs/archetype-sdk-client.yml +++ b/eng/pipelines/templates/jobs/archetype-sdk-client.yml @@ -101,11 +101,6 @@ jobs: displayName: 'Publish Artifacts $(ArtifactName)' artifact: '$(ArtifactName)' - - publish: $(System.DefaultWorkingDirectory) - condition: failed() - displayName: 'Capture repo state for analysis' - artifact: repository - - job: 'Analyze' condition: ne(variables['Skip.Analyze'], 'true') From f5096cf62cef736bbf8fd70884b51b16e556a77c Mon Sep 17 00:00:00 2001 From: Mitch Denny Date: Fri, 13 Dec 2019 17:37:40 +1100 Subject: [PATCH 2/2] Move to condition for repo capture so we can do it easily in the future. --- eng/pipelines/templates/jobs/archetype-sdk-client.yml | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/eng/pipelines/templates/jobs/archetype-sdk-client.yml b/eng/pipelines/templates/jobs/archetype-sdk-client.yml index 60bbd0b86469..cbe405cb52d7 100644 --- a/eng/pipelines/templates/jobs/archetype-sdk-client.yml +++ b/eng/pipelines/templates/jobs/archetype-sdk-client.yml @@ -101,6 +101,11 @@ jobs: displayName: 'Publish Artifacts $(ArtifactName)' artifact: '$(ArtifactName)' + - publish: $(System.DefaultWorkingDirectory) + condition: and(eq(variables['CaptureRepositoryOnFailure'], 'true'), failed()) + displayName: 'Capture repo state for analysis' + artifact: repository + - job: 'Analyze' condition: ne(variables['Skip.Analyze'], 'true')