Ticket #4356 enhancement closed fixed

Opened 3 years ago

Last modified 3 years ago

Add a ProcessMonitor plugin for twistd

Reported by: rwall Owned by:
Priority: normal Milestone:
Component: runner Keywords:
Cc: Branch: branches/procmon-plugin-4365
Author: rwall Launchpad Bug:

Description

It would be useful to be able to launch t.r.procmon.ProcessMonitor from the command line as a simple way to monitor and automatically restart another (unreliable) process. eg

twistd procmon squid -N

A plugin would also provide a nice example of procmon use and replace the ugly main function currently at the bottom of the module.

Attachments

procmon-plugin-4365.1.patch Download (6.4 KB) - added by rwall 3 years ago.
A procmon plugin and tests.

Change History

Changed 3 years ago by rwall

A procmon plugin and tests.

1

Changed 3 years ago by rwall

  • keywords review added
  • owner rwall deleted
  • branch_author set to rwall

2

Changed 3 years ago by rwall

Looks like this isn't a new idea. Tv added a ProcMon plugin in 2003 which was later removed by moshez:

3

Changed 3 years ago by glyph

Applied patch to a branch so we can see  build results.

4

Changed 3 years ago by glyph

  • branch set to branches/procmon-plugin-4365

Gave the branch the wrong number, unfortunately. I'll fix it if I merge forward.

5

Changed 3 years ago by cyli

  • keywords review removed
  • owner set to cyli

The tests pass, and running twistd procmon sleep 10 works - sleep 10 is restarted over and over.

Made two small changes:

  1. Gave t.r.test.test_procmontap.ProcessMonitorTapTest a class (unittest.TestCase) level docstring.
  2. Reflowed some of the text in t.r.procmontap.py's and t.r.test.test_procmontap.py's lines wrapped to 79 columns. Also put in extra newlines so that t.r.procmontap.py has the right number of newlines between methods/class and methods.

6

Changed 3 years ago by cyli

  • status changed from new to closed
  • resolution set to fixed

(In [29212]) Merge procmon-plugin-4365: Add a ProcessMonitor plugin for twistd

Author: rwall Reviewer: cyli Fixes: #4356

Added a twistd plugin to launch twisted.runner.procmon.ProcessMonitor from the command line.

7

Changed 2 years ago by <automation>

  • owner cyli deleted
Note: See TracTickets for help on using tickets.