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

Better error messaging for pid file errors

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Upon starting the collector, if the configured pid dir fs doesn't support locking (i.e., nfs) you get this:
      Initializaiton failed: can not write pid file.

      This should be improved to 1.) echo back the pid file, 2.) specify that it's a locking issue and 3.) correct the typo.

      1. CHUKWA-414.patch
        0.8 kB
        Bill Graham

        Activity

        Hide
        billgraham Bill Graham added a comment -

        Attaching a patch.

        Show
        billgraham Bill Graham added a comment - Attaching a patch.
        Hide
        asrabkin Ari Rabkin added a comment -

        +1 on patch. I will commit it later today, barring objection.

        Show
        asrabkin Ari Rabkin added a comment - +1 on patch. I will commit it later today, barring objection.
        Hide
        eyang Eric Yang added a comment -

        +1 looks good.

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

        I just committed this. Thanks, Bill, both for the patch and your other feedback on our documentation.

        Show
        asrabkin Ari Rabkin added a comment - I just committed this. Thanks, Bill, both for the patch and your other feedback on our documentation.
        Hide
        asrabkin Ari Rabkin added a comment -

        Note committed to TRUNK only.

        Show
        asrabkin Ari Rabkin added a comment - Note committed to TRUNK only.
        Hide
        hudson Hudson added a comment -
        Show
        hudson Hudson added a comment - Integrated in Chukwa-trunk #213 (See http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/213/ )

          People

          • Assignee:
            billgraham Bill Graham
            Reporter:
            billgraham Bill Graham
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development