#5328 taskd should set process name for each task's type or _id

v1.0.0
closed
Tim Van Steenburgh
sf-1 (616)
General
Dave Brondsema
2015-08-20
2012-11-19
Dave Brondsema
No

setproctitle is a nice library (Zarkov uses it) to set process names. We should use it to set each taskd instance's name to something nice (so it doesn't show up as just python or paster). Moreover, we could set title each time a job changes so we can easily see what is currently running. Possible name format: taskd:allura.tasks.mail_tasks.sendmail:509c56029c1040420e2d8487

Related

Tickets: #5127
Tickets: #5721
Tickets: #7803

Discussion

  • Dave Brondsema

    Dave Brondsema - 2013-05-14

    Need to make sure that changing the process name doesn't break taskd_cleanup command. And at SourceForge, our reactor init script and monit.

     

  • Dave Brondsema

    Dave Brondsema - 2013-05-20
    • Milestone: forge-backlog --> forge-jun-06
     

  • Dave Brondsema

    Dave Brondsema - 2013-06-03
    • Size: --> 1
     

  • Tim Van Steenburgh - 2013-06-05
    • status: open --> in-progress
    • assigned_to: Tim Van Steenburgh
     

  • Tim Van Steenburgh - 2013-06-05

    forge:tv/5328

    The ticket to add setproctitle is here: https://trac.sdot.me/trac/siteops/ticket/53845. In the meantime you might need to pip install it manually.

    To see this working, I ran top, sorted by proc name and showing full cmd line. Kick off a task that will take a while (I did an SVN import). You'll see the proc name change in top, then change back when the task is finished.

    I discussed with wayned and he said this would not cause any problems for the init or monit scripts.

     

  • Tim Van Steenburgh - 2013-06-05
    • status: in-progress --> code-review
     

  • Dave Brondsema

    Dave Brondsema - 2013-06-12
    • QA: Dave Brondsema
     

  • Dave Brondsema

    Dave Brondsema - 2013-06-12
    • status: code-review --> closed
     

Log in to post a comment.