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.

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

Add PR refs to sparse-checkout.yml #3996

Closed
mikeharder opened this issue Aug 23, 2022 · 2 comments
Closed

Add PR refs to sparse-checkout.yml #3996

mikeharder opened this issue Aug 23, 2022 · 2 comments
Assignees
Labels
Central-EngSys This issue is owned by the Engineering System team.

Comments

@mikeharder
Copy link
Member

The perf automation wants the ability to use sparse-checkout.yml with a PR refs (e.g. refs/pull/1234/merge). Currently, this fails with the following error:

error: pathspec 'refs/pull/1234/merge' did not match any file(s) known to git

Options:

  1. Include all PR refs:

image

  1. Include only the PR ref we need, and only if "commitish" looks like a PR ref.
git clone --no-checkout --filter=tree:0 -c remote.origin.fetch='+refs/pull/1234/merge:refs/pull/1234/merge' https://github.com/$($repository.Name) .
@mikeharder mikeharder self-assigned this Aug 23, 2022
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Aug 23, 2022
@mikeharder mikeharder added Central-EngSys This issue is owned by the Engineering System team. and removed needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. labels Aug 23, 2022
@mikeharder mikeharder moved this to 🤔Triage in Azure SDK EngSys 🚢🎉 Aug 23, 2022
@mikeharder mikeharder moved this from 🤔Triage to 🐝 Dev in Azure SDK EngSys 🚢🎉 Aug 23, 2022
@mikeharder
Copy link
Member Author

mikeharder commented Aug 23, 2022

> git clone
  --no-checkout
  --filter=tree:0
  https://github.com/Azure/azure-sdk-for-net .

remote: Enumerating objects: 25435, done.
remote: Counting objects: 100% (43/43), done.
remote: Compressing objects: 100% (42/42), done.
remote: Total 25435 (delta 1), reused 40 (delta 1), pack-reused 25392
Receiving objects: 100% (25435/25435), 10.02 MiB | 1.16 MiB/s, done.
Resolving deltas: 100% (386/386), done.


> git clone
  --no-checkout
  --filter=tree:0
  -c remote.origin.fetch='+refs/pull/30376/merge:refs/pull/30376/merge'
  https://github.com/Azure/azure-sdk-for-net .

remote: Enumerating objects: 25445, done.
remote: Counting objects: 100% (44/44), done.
remote: Compressing objects: 100% (43/43), done.
remote: Total 25445 (delta 1), reused 40 (delta 1), pack-reused 25401
Receiving objects: 100% (25445/25445), 10.01 MiB | 4.48 MiB/s, done.
Resolving deltas: 100% (388/388), done.


> git clone
  --no-checkout
  --filter=tree:0
  -c remote.origin.fetch='+refs/pull/*:refs/pull/*'
  https://github.com/Azure/azure-sdk-for-net .

remote: Enumerating objects: 90654, done.
remote: Counting objects: 100% (201/201), done.
remote: Compressing objects: 100% (198/198), done.
remote: Total 90654 (delta 4), reused 173 (delta 3), pack-reused 90453
Receiving objects: 100% (90654/90654), 24.02 MiB | 2.90 MiB/s, done.
Resolving deltas: 100% (2914/2914), done.

@mikeharder
Copy link
Member Author

Fixed by #3874

Repository owner moved this from 🐝 Dev to 🎊Closed in Azure SDK EngSys 🚢🎉 Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team.
Projects
None yet
Development

No branches or pull requests

1 participant