Add workflow trace - Githubissues

sebneira commented 3 months ago

What was changed

Ported over trace to the new cli rewrite.

Added a couple of fixes to get this to work:

Moved all the tracer printing logic to the WorkflowTracer (see https://github.com/temporalio/cli/pull/507 for the isolated changes). This was the initial fix I was working on. This should remove the issues when state == nil and the weirdness with the setupUpdateChannels.
Removed ExecutionState's GetDuration and GetStartTime time reference since the events now use durationpb.Duration and timestamppb.Timestamp and the conversion to non-pointer values makes more sense. Adjusted the templates to present this data correctly.

Ported the command to the cli rewrite and provided fixes for the nil pointer issues reported by Chad.

All unit tests are running and I've run the tool against a couple different workflow executions with temporalite.

CLAassistant commented 3 months ago

All committers have signed the CLA.

cretz commented 3 months ago

This is great! Will review when you mark "ready for review"