Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bull
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 175
    • Issues 175
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 9
    • Merge requests 9
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • OptimalBits
  • bull
  • Merge requests
  • !1168

Fix breakpoints in child processes

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/Vortec4800/breakpoint into develop Jan 10, 2019
  • Overview 10
  • Commits 5
  • Pipelines 0
  • Changes 2

Created by: Vortec4800

Bull currently strips out the inspect flags from the parent process when spawning child processes. This means if you run the program in an IDE, any breakpoints you set in your sandboxed processors will not fire, which can make development and debugging them much more difficult.

The solution is to detect if the inspect flag is set, then modify it to automatically select an open debug port. This keeps from conflicting with ports that are already in use by other processes. In a production environment where this flag is not set, there is no change to the arguments.

Tested by setting a breakpoint in a sandboxed process and running with WebStorm. Breakpoint in the child process was hit as expected.

If there are side-effects to this change that I'm not thinking of then please let me know, otherwise I think this is a pretty big quality-of-life improvement during development of the processors.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/Vortec4800/breakpoint