-
Notifications
You must be signed in to change notification settings - Fork 30
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
fix: redundant CLI downloads [HEAD-450][HEAD-504][HEAD-505] #163
Conversation
This reverts commit 6b3b08d.
Have you folks looked into the official tool installer and the tool caching directory for the Azure Pipelines Agent? |
I'm not familiar with neither, but it sounds like there's a way to save us some cloud costs so I'm interested! |
Tool lib is here: https://github.com/microsoft/azure-pipelines-tool-lib It provides a set of standard APIs and a naming convention to store tools on the agent in a way that allows multiple jobs to use them, but also shields multiple agents installed on the same machine from eachother. The tools are installed in the agent's Example here: https://github.com/microsoft/azure-devops-extension-tasks/tree/main/BuildTasks/TfxInstaller/v4 In this case the tool installer was broken out into a separate task, but you can merge the 2 together. The tool installer can maybe be extended with the caching features native to Azure DevOps to store a copy of the tool in the Organization's cache to speed up downloads and reduce bandwidth. Let me check with the Azure Pipelines Team to see if this option exists. The agent has a build-in caching plugin, but I'm unsure how to leverage this from a custom task outside of the built-in |
Do we actually use GZIP compression on the downloads here? Would be good, too :). |
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.
Let's see if these changes fix the immediate issues we are seeing and add an improvement task for using the proposed tooling support.
Thanks a lot @jessehouwing for bringing this to our attention!
I also added 2 PRs some time back to improve the representation of the extension on the marketplace. I think they may have gotten lost in the team transfer. |
🎉 This PR is included in version 1.1.3 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
No description provided.