Remove PID file when exiting

Bug #257081 reported by Andreas Hasenack
2
Affects Status Importance Assigned to Milestone
Landscape Client
Fix Released
Low
Christopher Armstrong
Landscape Server
Fix Released
Low
Christopher Armstrong
landscape-client (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Since the client PID file is created by the daemon when it starts, it should also be removed by the daemon when it exits.

Changed in landscape-client:
importance: Undecided → Low
Changed in landscape:
importance: Undecided → Low
milestone: none → thames
Changed in landscape:
assignee: nobody → radix
milestone: thames → thames-pre-8
Changed in landscape-client:
assignee: nobody → radix
status: New → In Progress
Changed in landscape:
status: New → In Progress
Revision history for this message
Christopher Armstrong (radix) wrote :

Please review the attached branch.

Revision history for this message
Christopher Armstrong (radix) wrote :

I got a +1 from thomas (no really)

Revision history for this message
Jamu Kakar (jkakar) wrote :

+1.

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Works great!

qa + 1

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

A debdiff is available in bug #271886

Changed in landscape-client:
status: New → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package landscape-client - 1.0.21-0ubuntu1

---------------
landscape-client (1.0.21-0ubuntu1) intrepid; urgency=low

  [ Christopher Armstrong ]
  * New upstream release (LP: #271886):
    - Bug fix release:
    - Avoid the PotentialZombieWarning on landscape-client startup.
     (LP: #257346)
    - When run as root, read sysinfo configuration from /etc and and write
      logs to /var/log instead of /root. (LP: #268560)
    - Avoid ZeroDivisionErrors when /home is an autofs. (LP: #269634)
    - Don't corrupt a pid file when trying to start the client when it's
      already running. (LP: #269634)
    - Remove the pid file when shutting down the client. (LP: #257081)

  [ Mathias Gug ]
  * debian/landscape-client.init: specify the pid file and use --startas
    instead of --exec when starting landscape-client so that the init script
    doesn't fail if landscape-client is already running.

 -- Mathias Gug <email address hidden> Fri, 19 Sep 2008 17:28:08 -0400

Changed in landscape-client:
status: In Progress → Fix Released
Changed in landscape:
status: In Progress → Fix Committed
Changed in landscape-client:
status: In Progress → Fix Committed
Changed in landscape-client:
status: Fix Committed → Fix Released
Changed in landscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.