Skip to content

Commit

Permalink
[release/2.1.3] Warn on online baseline failure for offline legs (#763)
Browse files Browse the repository at this point in the history
  • Loading branch information
dagood authored Sep 10, 2018
1 parent 774545f commit a5a033b
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
1 change: 1 addition & 0 deletions .vsts.pipelines/phases/ci-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,7 @@ phases:
$(docker.run) $(docker.src.map) $(docker.src.work) $(imageName) ./build.sh \
/p:ArchiveDownloadedPackages=true \
/p:Configuration=$(buildConfiguration) \
/p:ContinueOnPrebuiltBaselineError=$(buildOfflineTarball) \
/p:ProdConBlobFeedUrlPrefix=$(prodConBlobFeedUrlPrefix) \
$(buildLoggingOptions)
displayName: Build source-build
Expand Down
2 changes: 1 addition & 1 deletion netci.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ def addPushJob(String project, String branch, String os, String configuration)
steps{
shell("cd ./source-build;git submodule update --init --recursive");
// First build the product itself
shell("docker run -u=\"\$(id -u):\$(id -g)\" -t --sig-proxy=true -e HOME=/opt/code/home -v \$(pwd)/source-build:/opt/code --rm -w /opt/code ${imageName} /opt/code/build.sh /p:ArchiveDownloadedPackages=true /p:Configuration=${configuration} ${loggingOptions}");
shell("docker run -u=\"\$(id -u):\$(id -g)\" -t --sig-proxy=true -e HOME=/opt/code/home -v \$(pwd)/source-build:/opt/code --rm -w /opt/code ${imageName} /opt/code/build.sh /p:ArchiveDownloadedPackages=true /p:Configuration=${configuration} /p:ContinueOnPrebuiltBaselineError=true ${loggingOptions}");
// Have to make this directory before volume-sharing it unlike non-docker build - existing directory is really only a warning in build-source-tarball.sh
shell("mkdir tarball-output");
// now build the tarball
Expand Down
5 changes: 4 additions & 1 deletion repos/dir.targets
Original file line number Diff line number Diff line change
Expand Up @@ -406,13 +406,16 @@
<BaselineDataFile>$(ToolsLocalDir)prebuilt-baseline-</BaselineDataFile>
<BaselineDataFile Condition="'$(OfflineBuild)' == 'true'">$(BaselineDataFile)offline.xml</BaselineDataFile>
<BaselineDataFile Condition="'$(OfflineBuild)' != 'true'">$(BaselineDataFile)online.xml</BaselineDataFile>

<ContinueOnPrebuiltBaselineError Condition="'$(ContinueOnPrebuiltBaselineError)' == ''">false</ContinueOnPrebuiltBaselineError>
</PropertyGroup>

<ValidateUsageAgainstBaseline
DataFile="$(PackageReportDataFile)"
BaselineDataFile="$(BaselineDataFile)"
OutputBaselineFile="$(PackageReportDir)generated-new-baseline.xml"
OutputReportFile="$(PackageReportDir)baseline-comparison.xml" />
OutputReportFile="$(PackageReportDir)baseline-comparison.xml"
ContinueOnError="$(ContinueOnPrebuiltBaselineError)" />

<ItemGroup>
<PackageVersionPropsSavedSnapshotFiles Include="$(PackageReportDir)snapshots/PackageVersions.props.pre.*.xml" />
Expand Down

0 comments on commit a5a033b

Please sign in to comment.