-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Switched DownloadBuildArtifactV0 to agent plugin. Aligned DownloadBuildArtifactV1 task with V0 #15244
Switched DownloadBuildArtifactV0 to agent plugin. Aligned DownloadBuildArtifactV1 task with V0 #15244
Conversation
…puts for V0 and V1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should remove ts implementation of the task from DownloadBuildArtifactsV0, since it won't be used in fact. Do you have any concerns about it?
…://github.com/microsoft/azure-pipelines-tasks into users/anatolybolshakov/update-build-artifacts
Yes make sense, let me update also |
LGTM! Tested these changes manually found no issues, everything work as expected |
Despite that this change has been reverted, I would like to report that the DownloadBuildArtifact v0.193.0 is breaking our deployment pipeline since 2 weeks ago. Our deployment pool agent is installed in CentOS 7 host. Outbound communications are protected by corporate proxy. The error message we got are pasted below. It might be related to this issue?
|
Task name: DownloadBuildArtifactV1, DownloadBuildArtifactV0
Description: switched DownloadBuildArtifactV0 to agent plugin. Aligned inputs for DownloadBuildArtifactV1 and DownloadBuildArtifactV0. Bumped minimum agent version for these tasks to 2.191.1 - to align with plugin changes.
Documentation changes required: Y (in progress)
Added unit tests: N
Attached related issue: N
Checklist: