Uploaded image for project: 'Chukwa'
  1. Chukwa
  2. CHUKWA-677

chukwa script fails to check if the process is already running

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.4.0, 0.5.0
    • Fix Version/s: 0.6.0
    • Component/s: Scripts
    • Labels:
      None
    • Environment:

      Linux

      Description

      When the user tries to start the chukwa processes without stopping previous instances, the pids will be overwritten. This prevents the earliers processes from stopping when we issue stop command. The user will have to manually kill the processes.

      1. CHUKWA-677_trunk.patch
        0.7 kB
        Sreepathi Prasanna
      2. CHUKWA-677.patch
        0.5 kB
        Sreepathi Prasanna

        Activity

        Hide
        eyang Eric Yang added a comment -

        +1 looks good.

        Show
        eyang Eric Yang added a comment - +1 looks good.
        Hide
        eyang Eric Yang added a comment -

        Sreepathi, it looks like the patch is generated from 0.5 instead of trunk. Hence, the patch doesn't apply. Could you regenerate another patch for trunk? Thanks

        Show
        eyang Eric Yang added a comment - Sreepathi, it looks like the patch is generated from 0.5 instead of trunk. Hence, the patch doesn't apply. Could you regenerate another patch for trunk? Thanks
        Hide
        sree2k Sreepathi Prasanna added a comment -

        Submitted the patch for trunk.

        Show
        sree2k Sreepathi Prasanna added a comment - Submitted the patch for trunk.
        Hide
        eyang Eric Yang added a comment -

        +1 looks good.

        Show
        eyang Eric Yang added a comment - +1 looks good.
        Hide
        eyang Eric Yang added a comment -

        I just committed this. Thanks Sreepathi

        Show
        eyang Eric Yang added a comment - I just committed this. Thanks Sreepathi
        Hide
        hudson Hudson added a comment -

        Integrated in Chukwa-trunk #462 (See https://builds.apache.org/job/Chukwa-trunk/462/)
        CHUKWA-677. Added pid file check before starting processes. (Sreepathi Prasanna via Eric Yang) (Revision 1424080)

        Result = FAILURE
        eyang : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1424080
        Files :

        • /incubator/chukwa/trunk/CHANGES.txt
        • /incubator/chukwa/trunk/bin/chukwa
        Show
        hudson Hudson added a comment - Integrated in Chukwa-trunk #462 (See https://builds.apache.org/job/Chukwa-trunk/462/ ) CHUKWA-677 . Added pid file check before starting processes. (Sreepathi Prasanna via Eric Yang) (Revision 1424080) Result = FAILURE eyang : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1424080 Files : /incubator/chukwa/trunk/CHANGES.txt /incubator/chukwa/trunk/bin/chukwa

          People

          • Assignee:
            sree2k Sreepathi Prasanna
            Reporter:
            sree2k Sreepathi Prasanna
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development