Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-5784

Usage of kill in logrotate script is non-portable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • other
    • None

      The logrotate script uses

      kill -SIGALRM `cat /var/run/hudson.pid`

      which works fine for the bash builtin, but fails for /bin/kill, which only accepts

      usage: kill [ -s signal | -p ] [ -a ] pid ...
             kill -l [ signal ]

      on CentOS 5.

      Using kill -s SIGALRM would make both variants happy and increase portability.

          [JENKINS-5784] Usage of kill in logrotate script is non-portable

          Robert Munteanu created issue -
          pdurbin made changes -
          Attachment New: hudson.logrotate.patch2010-10-20pdurbin [ 19874 ]
          sannies made changes -
          Priority Original: Minor [ 4 ] New: Major [ 3 ]
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          Olav Reinert made changes -
          Resolution Original: Fixed [ 1 ]
          Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
          Olav Reinert made changes -
          Attachment New: jenkins-logrotate.patch [ 20237 ]
          Olav Reinert made changes -
          Assignee New: Olav Reinert [ oreinert ]
          Olav Reinert made changes -
          Status Original: Reopened [ 4 ] New: In Progress [ 3 ]
          Olav Reinert made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          Sami Salonen made changes -
          Resolution Original: Fixed [ 1 ]
          Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
          Olav Reinert made changes -
          Assignee Original: Olav Reinert [ oreinert ]

            Unassigned Unassigned
            rombert Robert Munteanu
            Votes:
            6 Vote for this issue
            Watchers:
            8 Start watching this issue

              Created:
              Updated:
              Resolved: