Skip to content

Conversation

@mikeharder
Copy link
Member

@mikeharder mikeharder commented Oct 22, 2025

  • Other files in azure-sdk-tools repo are unnecessary
  • Unnecessary checkout hurts perf and increases security footprint

Verification Run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5483253&view=results

- Other files in azure-sdk-tools repo are unnecessary
- Unnecessary checkout hurts perf and increases security footprint
@azure-sdk
Copy link
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@mikeharder mikeharder marked this pull request as ready for review October 22, 2025 01:58
@mikeharder mikeharder requested a review from a team as a code owner October 22, 2025 01:58
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR optimizes the checkout process in the performance pipeline by implementing sparse checkout for the azure-sdk-tools repository, reducing unnecessary file downloads and improving security posture.

Key Changes:

  • Implements sparse checkout to only fetch the PerfAutomation tool directory from azure-sdk-tools
  • Reduces the checkout footprint by excluding unnecessary files from the azure-sdk-tools repository

@mikeharder mikeharder moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 🍔🌭 Oct 22, 2025
azure-sdk added a commit to Azure/azure-sdk-for-js that referenced this pull request Oct 22, 2025
azure-sdk added a commit to Azure/azure-sdk-for-rust that referenced this pull request Oct 22, 2025
@mikeharder mikeharder merged commit 31d1ee4 into Azure:main Oct 22, 2025
12 checks passed
@benbp benbp moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 🍔🌭 Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants