Skip to content
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.

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

Publish VS workloads assets in requested folder layout to BAR #57559

Merged
merged 1 commit into from
Aug 17, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
11 changes: 11 additions & 0 deletions eng/pipelines/common/upload-intermediate-artifacts-step.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
parameters:
name: ''
publishPackagesCondition: always()
publishVSSetupCondition: false

steps:
- task: CopyFiles@2
Expand All @@ -14,6 +15,16 @@ steps:
TargetFolder: '$(Build.StagingDirectory)/IntermediateArtifacts/${{ parameters.name }}'
CleanTargetFolder: true

- task: CopyFiles@2
displayName: Prepare job-specific intermediate artifacts subdirectory
condition: and(succeeded(), ${{ parameters.publishVSSetupCondition }})
inputs:
SourceFolder: '$(Build.SourcesDirectory)/artifacts/VSSetup/$(_BuildConfig)'
Contents: |
Insertion/**/*
TargetFolder: '$(Build.StagingDirectory)/IntermediateArtifacts/${{ parameters.name }}'
CleanTargetFolder: true

- task: PublishBuildArtifacts@1
displayName: Publish intermediate artifacts
inputs:
Expand Down
12 changes: 5 additions & 7 deletions eng/pipelines/mono/templates/workloads-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,10 +91,8 @@ jobs:
Contents: '*.wixpdb'

# Upload artifacts to be used for generating VS components
- task: PublishPipelineArtifact@1
displayName: Publish workload artifacts
inputs:
targetPath: $(Build.SourcesDirectory)/artifacts/VSSetup/$(_BuildConfig)/Insertion
artifactName: 'Insertion'
continueOnError: true
condition: always()
- template: /eng/pipelines/common/upload-intermediate-artifacts-step.yml
parameters:
name: workloads-vs
publishPackagesCondition: false
publishVSSetupCondition: true
13 changes: 12 additions & 1 deletion src/installer/prepare-artifacts.proj
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,10 @@
<PublishFlatContainer>true</PublishFlatContainer>
</ItemsToPush>

<ItemsToPush Include="@(WorkloadsVSInsertionFile)">
<RelativeBlobPath>$(InstallersRelativePath)workloads/$(SdkBandVersion)/%(Filename)%(Extension)</RelativeBlobPath>
<PublishFlatContainer>true</PublishFlatContainer>
</ItemsToPush>
</ItemGroup>

<!-- Push items to AzDO as build artifacts, generating the asset manifest as a side effect. -->
Expand Down Expand Up @@ -171,14 +175,15 @@

<DownloadedSymbolNupkgFile Include="$(DownloadDirectory)**\*.symbols.nupkg" />
<DownloadedWixPdbFile Include="$(DownloadDirectory)**\*.wixpdb" />
<DownloadedWorkloadsVSInsertionFile Include="$(DownloadDirectory)*\workloads-vs\**\*" />
<DownloadedNupkgFile
Include="$(DownloadDirectory)**\*.nupkg"
Exclude="@(DownloadedSymbolNupkgFile)" />

<!-- Add files that are not affected by filtering. -->
<UploadToBlobStorageFile
Include="@(DownloadedArtifactFile)"
Exclude="@(DownloadedSymbolNupkgFile);@(DownloadedNupkgFile);@(DownloadedWixPdbFile)" />
Exclude="@(DownloadedSymbolNupkgFile);@(DownloadedNupkgFile);@(DownloadedWixPdbFile);@(DownloadedWorkloadsVSInsertionFile)" />

<!--
Filter out the RID-specific (Runtime) nupkgs and RID-agnostic nupkgs. RID-specific packages
Expand All @@ -199,6 +204,12 @@
Include="$(DownloadDirectory)**\VS.Redist.Common.*.nupkg"
Exclude="@(DownloadedSymbolNupkgFile)" />

<!-- Workloads VS insertion artifacts produced by src/workloads/workloads.csproj -->
<WorkloadsVSInsertionFile
Include="
$(DownloadDirectory)*\workloads-vs\**\*.json;
$(DownloadDirectory)*\workloads-vs\**\*.msi" />

<!--
Runtime packages associated with some identity packages. Need to exclude "runtime.native.*"
because Libraries produces some "runtime.native.Foo" packages with
Expand Down