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

Sparse checkout should handle private repos #3222

Closed
danieljurek opened this issue Apr 26, 2022 · 6 comments
Closed

Sparse checkout should handle private repos #3222

danieljurek opened this issue Apr 26, 2022 · 6 comments
Assignees
Labels
Central-EngSys This issue is owned by the Engineering System team.

Comments

@danieljurek
Copy link
Member

No description provided.

@danieljurek danieljurek added the Central-EngSys This issue is owned by the Engineering System team. label Apr 26, 2022
@danieljurek danieljurek self-assigned this Apr 26, 2022
@benbp
Copy link
Member

benbp commented Apr 26, 2022

@danieljurek I couldn't get the PAT formatting syntax right at the time based on what devops gave the task, but I believe there are some promising examples here: https://github.com/MicrosoftDocs/azure-devops-docs/issues/2455

@danieljurek
Copy link
Member Author

danieljurek commented Apr 27, 2022

Have we considered ~/.git-credential?

          git config --global user.name "azure-sdk"
          git config --global user.email "[email protected]"
          echo "https://azure-sdk:$(example-pat-secret)@github.com" > ~/.git-credentials
          git config --global credential.helper "store"

I haven't needed to use a PAT on a per-repo basis. At this point one PAT has sufficed but I'd be glad to investigate if we'll need that.

@benbp

@weshaggard
Copy link
Member

There is also persistCredentials option for an explicit checkout https://docs.microsoft.com/en-us/azure/devops/pipelines/yaml-schema/steps-checkout?view=azure-pipelines that we might be able to use to get the correct credentials setup.

@benbp
Copy link
Member

benbp commented Apr 27, 2022

@weshaggard possibly if we can also set checkout: none but it might get tricky regarding whether you can define that field multiple times or if it would conflict with usage of pipelines where it should not be set to none (e.g. when defining multiple checkouts).

@JimSuplizio
Copy link
Member

@danieljurek, @benbp and I chatted and we're closing this one out. It's been almost 2 1/2 years, and this clearly isn't going to get fixed.

@github-project-automation github-project-automation bot moved this from 📋 Backlog to 🎊 Closed in Azure SDK EngSys 🚢🎉 Sep 9, 2024
@weshaggard
Copy link
Member

I'm fine closing this but we may need to solve this for other reasons in the future.

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
Archived in project
Development

No branches or pull requests

4 participants