Skip to content

Draft: fix: close subprocesses in senders

David Flor requested to merge xflord/sender_close_processes into main

Description

fix: close subprocesses in senders

  • sometimes zombie tar and cat processes would remain on instances
  • this is most-likely a result of just closing stdout stream
  • these changes should properly wait for subprocess to finish

How to test

TODO: Provide clear steps on how to test this merge request

Author's checklist

  • I have followed the contribution guidelines
  • This MR has been tested or does not change functionality
  • I have added relevant merge request dependencies (if this MR has any)
  • I have added the correct labels
  • I have assigned reviewers (if any are relevant)
  • I have edited the documentation (if the changes require it) or I have noted the need for the change if I do not have access to the documentation
  • I have marked all introduced BREAKING CHANGES or necessary DEPLOYMENT NOTES in the commit message(s)

Reviewer's checklist

  • This MR has been tested or does not change functionality
  • This MR has correct commit message format

Other information

N/A

Related issues

N/A

Merge request reports