fix: Improve sparse checkout script reliability and error handling#139
Open
pgabriel-01 wants to merge 23 commits intoAzure:Dec2025patchfrom
Open
fix: Improve sparse checkout script reliability and error handling#139pgabriel-01 wants to merge 23 commits intoAzure:Dec2025patchfrom
pgabriel-01 wants to merge 23 commits intoAzure:Dec2025patchfrom
Conversation
… and private endpoints - Add comprehensive Terraform v4.11 modernization details - Document Python dependencies upgrade from 3.8 to 3.11 - Add Azure SDK updates (azure-ai-ml 1.1.0 → 1.21.1) - Document private endpoint implementation for all services - Update version matrix with all completed changes - Add Key Vault RBAC migration details - Include storage security enhancements (TLS 1.2, retention policies) - Document Container Registry upgrade to Premium SKU
…d compute SKUs GitHub Actions Guide (deployguide_gha.md): - Replace service principal secrets with OIDC workload identity federation - Update authentication setup with federated credentials - Change from single AZURE_CREDENTIALS secret to three separate secrets (CLIENT_ID, TENANT_ID, SUBSCRIPTION_ID) - Update compute SKUs from Standard_DS3_v2 to Standard_D4s_v5 (5th gen) - Update quota requirements to Dsv5 family Azure DevOps Guide (deployguide_ado.md): - Add security best practice note about workload identity federation - Update prerequisites with Terraform v1.9.0+ and Python 3.11 requirements - Update all compute SKU references to Standard_D4s_v5 - Update compute instance examples with 5th generation SKUs These changes align with the infrastructure modernization completed in the project templates.
…vider to ~> 4.52.0 in Azure DevOps deployment guide
…uring Terraform variables for GitHub Actions permissions
- Clarify service principal object ID requirement and usage - Add complete terraform.tfvars configuration example with all parameters - Document Azure ML workspace soft-delete behavior and workarounds - Add comprehensive endpoint testing instructions (online and batch) - Add environment destruction/teardown section with automatic endpoint cleanup - Include troubleshooting guidance for common issues - Based on successful end-to-end dev and prod deployments
…etion wait - Updated from 60-second to 2-minute wait for endpoint deletions - Adjusted total destroy time estimate from 7-10 minutes to 9-12 minutes - Added note that online endpoints can take 2-3 minutes to fully delete - Reflects fix in mlops-templates commit 7f20b6e
…network isolation details
…alise-project.yml
…nd improved file movement
…sitory creation process
… workload identity federation
…nd network isolation details
…ovements and clarity enhancements
…ture, clarity, and additional setup instructions
…figuration and replace manual image with automatic image
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Summary into Azure/mlops-project-template
Checklist
I have:
Changes
Fixed several issues in the sparse checkout script to improve reliability when bootstrapping new ML projects:
Bug Fixes
data,data-science,mlops) in sparse checkout to ensure complete folder structure is pulledgit initto prevent "not a git repository" warnings--confirmflag and corrected repo name format to$github_org_name/$project_nameImprovements
These changes ensure users get a complete project structure when running the sparse checkout script, eliminating the "missing folders" issue reported on different computers.
fixes #