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.

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

Jump to bottom

Fix the displayName and -TestLogPostfix String in the Phase.yml file #107

Closed
kiazhi opened this issue Dec 14, 2018 · 0 comments · Fixed by #108
Closed

Fix the displayName and -TestLogPostfix String in the Phase.yml file #107

kiazhi opened this issue Dec 14, 2018 · 0 comments · Fixed by #108

Comments

@kiazhi
Copy link
Contributor

kiazhi commented Dec 14, 2018

Issue
Community release job steps are displaying incorrect displayName and log file name.

PowerShell-Docker/.vsts-ci/phase.yml

Lines 40 to 50 in 7f6699f

displayName: $(ImageName) Servicing
condition: succeededOrFailed()
- ${{ if eq(parameters.communityPreview, 'true') }}:
- powershell: ./build.ps1 -build -name '$(ImageName)' -CI -Channel 'community-preview' -TestLogPostfix '$(ImageName)-servicing'
displayName: $(ImageName) Servicing
condition: succeededOrFailed()
- ${{ if eq(parameters.communityServicing, 'true') }}:
- powershell: ./build.ps1 -build -name '$(ImageName)' -CI -Channel 'community-servicing' -TestLogPostfix '$(ImageName)-servicing'
displayName: $(ImageName) Servicing

Originally posted by @TravisEz13 in #101 (comment)
@kiazhi kiazhi mentioned this issue Dec 14, 2018
Merged
9 tasks
TravisEz13 pushed a commit that referenced this issue Dec 14, 2018
@kiazhi @TravisEz13
Fix displayName in phase.yml for community release job steps (#108)
7fa2ddc 
fixes #107

- Update `displayName`
- Update the `String` after `$(ImageName)-` for the `-TestLogPostfix` parameter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix displayName in phase.yml for community release job steps
1 participant
@kiazhi