temporalio / temporalio/cli

workflow delete should (be able to) delete completed workflows

Open
#409 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
Go
Stars
379
Forks
103
Avg merge
2d 11h
Merged PRs (30d)
23

Description

temporal workflow delete -w $my-completed-workflow-id fails with

Error: unable to delete workflow: operation GetCurrentExecution encountered not found

but a user will expect it to delete running and not-running workflows alike.

The two cases are handled differently by the server deleteworkflow handler:

https://github.com/temporalio/temporal/blob/b53dafa4789981e3dfe6bf5767f7d641b2e690c5/service/history/api/deleteworkflow/api.go#L65-L69

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the temporal workflow delete -w ... command and compare its handling with the server deleteworkflow handler linked in the issue. Reproduce the failure for a completed workflow, then trace the reported GetCurrentExecution error; done means the command can delete both completed and running workflows without that error.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.