[SharePoint-ADFS] update template - #14806
Conversation
|
@Yvand - check this PR for updates that may be needed to documentation that references this sample. [This is an automated message. You are receiving it because you are listed as the docOwner in metadata.json.] |
|
Hello @alex-frankel, I read your comment on my previous update and I wanted to give you additional information. I always built this template in a separate repository first, and then published updates here when it was relevant, so in that aspect, essentially nothing changed.
|
|
@alex-frankel can you please review this PR? |
|
Hi @Yvand — apologies for the long wait here, and thanks for the nudges. One concrete blocker is holding this up. The sample-validation pipeline changed since your last update (#14711, which we validated via The new flow is ADX-backed: you deploy the sample, then record that deployment's For this sample (no "testResult": {
"deployments": {
"templateFileName": "main.bicep",
"correlationId": "<correlationId from your deployment>",
"deploymentName": "<your deployment name>"
}
}Please also bump Docs for reference:
Once you push the metadata update, I'll re-run validation promptly so we can get this merged without further delay. |
|
@alex-frankel no worries, thank you for your response, indeed I missed this change entirely. |
|
/validate |
🤖 Quickstart Sample SummarySample Summary
Resources Deployed(Extracted from main.bicep and associated Bicep modules)
Additional Notes:
Security FindingsNo security issues detected by MSDO scanners (Template Analyzer, Checkov, Trivy, Terrascan). No explicit hardcoded secrets or overly permissive network access noted in the templates reviewed. Secure parameters such as admin passwords are marked with Key Parameters
Notes for Reviewers
Files Touched
Generated by the quickstart summarizer agent (v2 — agentic + MSDO security) · triggered by /validate |
alex-frankel
left a comment
There was a problem hiding this comment.
Validated end-to-end: validate-samples-results (structural) and adx-deployment-validation (ADX deployment lookup + templateHash match) both pass on the current head. The committed azuredeploy.json hashes exactly to the recorded successful deployment. Thanks for the update to the SharePoint-ADFS sample.
PR Checklist
Check these items before submitting a PR...
Contribution Guide
Best Practice Guide
Changelog
Added
customSharePointConfiguration, to choose what SharePoint features will be configured in the farmChanged
enableAzureBastiontoaddBastionSubscription-Latestfor parametersharePointVersionnow installs the November 2025 CU for SharePoint Subscription