Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] replay workflow with log interceptor not showing logs #996

Closed
Irvenae opened this issue Dec 9, 2022 · 2 comments
Closed

[Bug] replay workflow with log interceptor not showing logs #996

Irvenae opened this issue Dec 9, 2022 · 2 comments
Labels
bug Something isn't working

Comments

@Irvenae
Copy link

Irvenae commented Dec 9, 2022

What are you really trying to do?

I am trying to get logs in my output when I am replaying a workflow. I am using my own logger, which is based on https://github.com/temporalio/samples-typescript/tree/main/instrumentation before it had the default helper functions.

Describe the bug

No log output when replaying a workflow that uses interceptors.

Screenshot when running the workflow (mind that in my example I changed the winston logger with a console logger, in my non-public code I am using a winston logger though):
Screenshot 2022-12-09 at 13 41 46

Screenshot when replaying the workflow:

Screenshot 2022-12-09 at 13 42 11

(Mind that the sink is known, otherwise you get Workflow referenced an unregistered external sink (which is not seen now))
I am running the test with Jest.

Minimal Reproduction

I think you can reproduce by running https://github.com/temporalio/samples-typescript/tree/main/instrumentation
combined with https://github.com/temporalio/samples-typescript/tree/main/replay-history.

I can make a branch if necessary, but this feels as quite basic functionality.

Environment/Versions

Mac M1
Temporal SDK 1.5.2
Temporal in docker.
Node version: v16.15.0

@Irvenae Irvenae added the bug Something isn't working label Dec 9, 2022
@bergundy
Copy link
Member

Hey,
Thanks for reporting.
I agree that we should allow logs to be emitted during replays, we just got a contribution for Go SDK to make this configurable.

We should add the same in TS.

@Irvenae
Copy link
Author

Irvenae commented Dec 13, 2022

Seems like you can set it on a more general level:

* - `callDuringReplay` - whether or not `fn` will be called during Workflow

thanks @mjameswh for the suggestion.

@Irvenae Irvenae closed this as completed Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants