Skip to content

docs(on-demand-sandboxes): restructure guides, modernize identity attach, drop sample VNet#337

Merged
torosent merged 3 commits into
Azure-Samples:mainfrom
greenie-msft:nigreenf-microsoft/update-ondemand-docs-managed-identity
Jun 18, 2026
Merged

docs(on-demand-sandboxes): restructure guides, modernize identity attach, drop sample VNet#337
torosent merged 3 commits into
Azure-Samples:mainfrom
greenie-msft:nigreenf-microsoft/update-ondemand-docs-managed-identity

Conversation

@greenie-msft

Copy link
Copy Markdown
Collaborator

Summary

Updates the On-demand Sandboxes (private preview) docs and the .NET sample so the guides are current and the sample is ready for end-to-end azd testing.

Docs

  • Move the scheduler-identity, worker-profile reference, and dashboard-logs content out of the overview README.md and into the per-language guides (docs/dotnet.md, docs/python.md), where each is tailored to that SDK's property names.
  • Fold the worker-profile reference table into Step 1 (right where the profile is declared) and inline the View-logs explanation as Step 3.
  • Merge "Choose your language" into Get started; access is now via dts-team@microsoft.com.
  • Switch the managed-identity attach flow from raw az rest PATCH to the new az durabletask scheduler identity assign CLI extension command (additive / merge-safe), in both guides and the attach-scheduler-identity.sh hooks.
  • Remove em dashes throughout the sandbox docs and sample READMEs.

.NET sample

  • Drop the unnecessary VNet (AKS auto-provisions a managed VNet; the cluster module's subnetId is already optional).
  • Fix stale gpt-4o references to gpt-5.1 to match the infra default.
  • Wire AZURE_OPENAI_LOCATION in main.parameters.json so the documented override actually takes effect (defaults to eastus).

Verification

  • az bicep build on infra/main.bicep compiles clean.
  • main.parameters.json is valid JSON.
  • Both .NET projects (main-app, sandbox-worker) build with 0 warnings / 0 errors.
  • Attach scripts pass bash -n.
  • No em dashes remain in any sandbox .md.

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

Nick Greenfield and others added 3 commits June 18, 2026 11:29
…ach, drop sample VNet

- Move scheduler-identity, worker-profile reference, and dashboard-logs
  content from the overview README into the per-language guides
- Merge "Choose your language" into "Get started"; point access to
  dts-team@microsoft.com
- Switch identity attach to the `az durabletask scheduler identity assign`
  CLI extension command (additive/merge-safe) in both guides and the
  attach-scheduler-identity.sh hooks
- Remove em dashes from all sandbox docs and sample READMEs
- .NET sample: drop the unnecessary VNet (AKS auto-provisions a managed
  VNet), fix gpt-4o -> gpt-5.1 references, and wire AZURE_OPENAI_LOCATION
  in main.parameters.json

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…, wire OpenAI location

Mirror the .NET sample cleanup in the Python sample:
- Remove the VNet module (AKS auto-provisions a managed VNet) and delete
  the unused vnet.bicep
- Fix stale gpt-4o references to gpt-5.1 to match the infra default
- Wire AZURE_OPENAI_LOCATION in main.parameters.json (defaults to eastus)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…default version

AKS 1.32 moved to Long-Term Support only, so pinning it fails preflight
validation on non-Premium clusters with K8sVersionNotSupported. Default
kubernetesVersion to empty in both samples and omit the property when
unset so AKS provisions its current default supported version.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@torosent
torosent merged commit 746d6d4 into Azure-Samples:main Jun 18, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants