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

Unsupported signal 2 delivered #28

Open
jazzdan opened this issue Oct 21, 2021 · 5 comments
Open

Unsupported signal 2 delivered #28

jazzdan opened this issue Oct 21, 2021 · 5 comments
Labels

Comments

@jazzdan
Copy link

jazzdan commented Oct 21, 2021

I tried to record a recording of an infinite loop because I wanted to see how it was getting stuck. The only way to terminate my program was to send it a SIGKILL via ctrl+c. Unfortunately that seems to render the recording unusable. When I run replay-recordings ls I get:

  {
    "id": 663276012,
    "createTime": "Thu Oct 21 2021 16:39:05 GMT-0400 (Eastern Daylight Time)",
    "runtime": "node",
    "metadata": {
      "argv": [
        "./node_modules/.bin/ts-node",
        "scripts/dev-backfill.ts",
      ]
    },
    "status": "unusable",
    "path": "/Users/dan/.replay/recording-663276012.dat",
    "unusableReason": "Recording invalidated: Unsupported signal 2 delivered"
  }
@nathggns
Copy link

I have the same issue.

@gideonred
Copy link

@bhackett1024 do we have plans to support breaking out of infinite loops during node recordings?

@bhackett1024
Copy link

@bhackett1024 do we have plans to support breaking out of infinite loops during node recordings?

Not presently, but this could go on the roadmap if we want it fixed. Both infinite loops and infinite recursions can be handled, but both will require special handling / participation from the runtime being recorded.

@gideonred
Copy link

@ceceliacreates it might be worth calling out in the documents that in our Node recording you currently need to have code that actually finishes since we don't support SIGKILL-ing out of the Node processes (and having a usable recording).

@gideonred
Copy link

This ticket has been moved to RUN-309

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants