Skip to content

Comments

Show pause information in "workflow describe" command#899

Merged
gow merged 2 commits intonext-serverfrom
cg/wf_pause_1
Jan 6, 2026
Merged

Show pause information in "workflow describe" command#899
gow merged 2 commits intonext-serverfrom
cg/wf_pause_1

Conversation

@gow
Copy link
Contributor

@gow gow commented Jan 6, 2026

What was changed

temporal workflow describe now shows pause information if the workflow is paused.

Why?

When a workflow is paused we need to show the relevant pause information like paused time, identity & reason.

Checklist

  1. Closes

  2. How was this tested:
    Tested manually

  3. Any docs updates needed?
    N/A

@gow gow requested review from a team as code owners January 6, 2026 06:14
@gow gow changed the title Cg/wf pause 1 Show pause information in "workflow describe" command Jan 6, 2026
@gow gow requested a review from spkane31 January 6, 2026 06:15
WorkflowId: c.WorkflowId,
RunId: c.RunId,
Identity: c.Parent.Identity,
Reason: c.Reason,
Copy link
Contributor

Choose a reason for hiding this comment

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

could revert but not required

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry I forgot to add a note. This is needed to fix a bug. I was not capturing the reason in pause command.

Copy link
Contributor

Choose a reason for hiding this comment

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

Why's that? This looks like it just moves the reason to a different line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh! yeah. That explains it. I might have accidentally removed this line locally and ran into the bug. And later I added it thinking I fixed it 🤦

Copy link
Contributor

Choose a reason for hiding this comment

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

not worth making the change, just pointed it out. no biggie

@gow gow merged commit 5e6f828 into next-server Jan 6, 2026
6 of 7 checks passed
@gow gow deleted the cg/wf_pause_1 branch January 6, 2026 17:34
@chaptersix chaptersix restored the cg/wf_pause_1 branch January 23, 2026 13:55
@chaptersix chaptersix deleted the cg/wf_pause_1 branch January 23, 2026 13:56
gow added a commit that referenced this pull request Jan 23, 2026
<!--- Note to EXTERNAL Contributors -->
<!-- Thanks for opening a PR! 
If it is a significant code change, please **make sure there is an open
issue** for this.
We work best with you when we have accepted the idea first before you
code. -->

<!--- For ALL Contributors 👇 -->

## What was changed
<!-- Describe what has changed in this PR -->
`temporal workflow describe` now shows pause information if the workflow
is paused.

## Why?
When a workflow is paused we need to show the relevant pause information
like paused time, identity & reason.

## Checklist
<!--- add/delete as needed --->

1. Closes <!-- add issue number here -->

2. How was this tested:
Tested manually

3. Any docs updates needed?
N/A
gow added a commit that referenced this pull request Jan 26, 2026
Redoing #899
<!--- Note to EXTERNAL Contributors -->
<!-- Thanks for opening a PR! 
If it is a significant code change, please **make sure there is an open
issue** for this.
We work best with you when we have accepted the idea first before you
code. -->

<!--- For ALL Contributors 👇 -->

## What was changed
<!-- Describe what has changed in this PR -->
`temporal workflow describe` now shows pause information if the workflow
is paused.

## Why?
When a workflow is paused we need to show the relevant pause information
like paused time, identity & reason.
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