Skip to content

Add Durable Task Scheduler skill#919

Open
greenie-msft wants to merge 21 commits intomicrosoft:mainfrom
greenie-msft:add-dts-skill
Open

Add Durable Task Scheduler skill#919
greenie-msft wants to merge 21 commits intomicrosoft:mainfrom
greenie-msft:add-dts-skill

Conversation

@greenie-msft
Copy link

Adds a new reference skill for the Durable Task Scheduler enabling Copilot to guide users through building reliable, fault-tolerant workflows on Azure.

The skill covers:

Framework selection (Durable Functions vs. Durable Task SDKs)
Local development with the DTS emulator
Workflow patterns: chaining, fan-out/fan-in, async HTTP, monitor, human interaction, and saga
Orchestration determinism rules and replay-safe logging
Azure deployment with Bicep and managed identity configuration
Error handling and retry policies

Copilot AI review requested due to automatic review settings February 13, 2026 18:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new Azure Prepare reference document for Durable Task Scheduler (DTS) to help guide users in building durable, fault-tolerant workflow orchestrations on Azure.

Changes:

  • Introduces a new durable-task-scheduler.md reference covering DTS fundamentals, local emulator setup, workflow patterns, and determinism rules.
  • Includes example snippets for .NET and Python (starter, patterns, logging, retry).
  • Adds Azure provisioning guidance including CLI, Bicep, and managed identity role assignment.

Copilot AI review requested due to automatic review settings February 13, 2026 23:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

greenie-msft added a commit to greenie-msft/GitHub-Copilot-for-Azure that referenced this pull request Feb 26, 2026
…nd Bicep options

- Change AuthorizationLevel from Anonymous to Function in all 4 language examples
- Fix 'QuickStarts' capitalization to lowercase 'quickstarts'
- Comment out Bicep Option B (UAMI) to prevent deploying both identity options
Copilot AI review requested due to automatic review settings February 26, 2026 18:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Copilot AI review requested due to automatic review settings February 26, 2026 21:10
greenie-msft and others added 7 commits February 26, 2026 13:12
…nd Bicep options

- Change AuthorizationLevel from Anonymous to Function in all 4 language examples
- Fix 'QuickStarts' capitalization to lowercase 'quickstarts'
- Comment out Bicep Option B (UAMI) to prevent deploying both identity options
…cheduler.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Copilot AI review requested due to automatic review settings February 26, 2026 21:36
Copy link
Member

@paulyuk paulyuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment

Copilot AI review requested due to automatic review settings February 26, 2026 23:39
- Move Learn More sections to top of each language file
- Standardize all files on DURABLE_TASK_SCHEDULER_CONNECTION_STRING
  with TaskHub embedded in connection string (replaces DTS_CONNECTION_STRING
  + TASKHUB_NAME in dotnet.md and bicep.md)
- Add Workflow & Orchestration category in research.md and architecture.md
- Add az extension install instruction for durabletask CLI
- Update troubleshooting references in README.md and deploy recipe
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated no new comments.

Assign Durable Task Data Contributor role to the deploying user's identity
so they can access the DTS dashboard in the Azure portal. Without this,
the dashboard returns 403 Forbidden.

Reference: https://github.com/Azure-Samples/Durable-Task-Scheduler/blob/main/samples/infra/main.bicep#L81
Copilot AI review requested due to automatic review settings February 27, 2026 21:02
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 4 comments.

paulyuk
paulyuk previously approved these changes Feb 27, 2026
Copy link
Member

@paulyuk paulyuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

- Fix malformed markdown table in architecture.md (separator and data row merged)
- Add TaskHub=default to local emulator connection string in README.md
Copy link
Member

@tmeschter tmeschter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not convinced the changes to azure-deploy are actually necessary.

The DTS deployment guidance in azure-deploy was redundant with the Bicep
patterns already in the azure-prepare DTS skill docs. Removed:
- durable-task-scheduler-deploy.md recipe file
- Reference line from azure-deploy SKILL.md
- 'Invoke azure-deploy' pointers from DTS README.md and bicep.md
Copilot AI review requested due to automatic review settings February 28, 2026 01:47
@greenie-msft
Copy link
Author

I'm not convinced the changes to azure-deploy are actually necessary.

Thanks for the review! I've removed that file and any references that pointed to it.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

@greenie-msft greenie-msft requested a review from tmeschter March 2, 2026 15:28
JasonYeMSFT
JasonYeMSFT previously approved these changes Mar 2, 2026
expectFiles(workspacePath!, [/infra\/.*\.bicep$/], [/\.tf$/]);

// Verify DTS-specific Bicep content was generated
const createCalls = getToolCalls(agentMetadata, "create");

Check failure

Code scanning / CodeQL

Invocation of non-function Error test

Callee is not a function: it has type undefined.
…cheduler as the default Durable Functions backend

- Update selection.md: route durable to full recipe with DTS instead of source-only
- Update composition.md: move durable from source-only to full recipe with DTS backend
- Rewrite recipes/durable/README.md: replace Azure Storage backend with DTS
- Add recipes/durable/bicep/durable-task-scheduler.bicep: scheduler + task hub + RBAC module
- Add durable-task-scheduler test in azure-prepare integration tests
- Add durable-task-scheduler-deploy test in azure-deploy integration tests
Copilot AI review requested due to automatic review settings March 3, 2026 00:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 11 comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants