Skip to content

workflow: retention policy is not retroactive#5074

Merged
marcduiker merged 2 commits intodapr:v1.17from
JoshVanL:workflow-retention-policy-notes
Apr 8, 2026
Merged

workflow: retention policy is not retroactive#5074
marcduiker merged 2 commits intodapr:v1.17from
JoshVanL:workflow-retention-policy-notes

Conversation

@JoshVanL
Copy link
Copy Markdown
Contributor

Clarify that adding or changing a retention policy does not retroactively purge workflows already in a terminal state. Include CLI examples for retroactive cleanup using dapr workflow purge --all-older-than and dapr workflow list --filter-status combined with purge.

@JoshVanL JoshVanL requested review from a team as code owners March 17, 2026 12:39
Copilot AI review requested due to automatic review settings March 17, 2026 12:39
Copy link
Copy Markdown
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

Clarifies Dapr Workflow history retention behavior by documenting that retention policies are not retroactive, and provides CLI guidance for purging existing terminal workflows.

Changes:

  • Added a warning that retention policy updates only apply to workflows that newly reach terminal states after the policy is active.
  • Added Dapr CLI examples for retroactive cleanup using dapr workflow purge and dapr workflow list --filter-status.
  • Linked to the “How to: Manage workflows” doc for additional purge details.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copy link
Copy Markdown
Contributor

@marcduiker marcduiker left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Josh van Leeuwen <me@joshvanl.dev>
@JoshVanL JoshVanL force-pushed the workflow-retention-policy-notes branch from bebe2dc to 17a786b Compare April 8, 2026 14:40
@marcduiker marcduiker merged commit 7b1f971 into dapr:v1.17 Apr 8, 2026
6 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.

3 participants