Skip to content

Some condition fixes on nuget pipeline, to get it green #2195

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.

Sign up for GitHub

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

Merged
snnn merged 3 commits into from
Oct 20, 2019
Merged

Some condition fixes on nuget pipeline, to get it green #2195

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d738c89
some fixes on nuget CPU pipeline
shahasad Oct 19, 2019
6a8c851
fix typo
shahasad Oct 19, 2019
433cba7
no need continue on error on the template
shahasad Oct 19, 2019
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
3 changes: 2 additions & 1 deletion tools/ci_build/github/azure-pipelines/nuget/templates/cpu.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ jobs:
# So, all the copy/publish jobs are being run on hosted agent
# TODO: install the desired azureps on our VMs or use later bugfixed version of AzureFileCopy
demands: azureps
condition: and (${{ parameters.DoEsrp }}, eq(variables['Build.SourceBranch'], 'refs/heads/master'))
condition: and (succeeded(), and (${{ parameters.DoEsrp }}, eq(variables['Build.SourceBranch'], 'refs/heads/master')))
dependsOn:
- NuGet_Test_Win
- NuGet_Test_Linux
Expand Down Expand Up @@ -175,6 +175,7 @@ jobs:
storage: ortpackages
containerName: ortpackages
blobPrefix: '$(CurrentDate)/'
continueOnError: true

- task: DotNetCoreCLI@2
displayName: 'Copy Signed NuGet Package to Internal NuGet Feed'
Expand Down