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

Provide an option to run HDFS usage plugin as a different user other that CHUKWA_USER

    Details

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

      Activity

      Hide
      zhangyongjiang Cheng added a comment -

      The default user id for running hdfsusage plugin is chukwa. It could be set in the default.properties at build time. It could be also changed in the build.propeties. If a value exists in build.properties, value in the default.properties has no use.

      Show
      zhangyongjiang Cheng added a comment - The default user id for running hdfsusage plugin is chukwa. It could be set in the default.properties at build time. It could be also changed in the build.propeties. If a value exists in build.properties, value in the default.properties has no use.
      Hide
      eyang Eric Yang added a comment -

      -1

      sed 's:CHUKWA_USER=chukwa:CHUKWA_USER=%

      {hdfsusage_uid}:' | \

      should be:

      sed 's:CHUKWA_USER=chukwa:CHUKWA_USER=${hdfsusage_uid}

      :' | \

      With %

      {hdfsusage_uid}

      , when rpm.hdfsusage.user is undefined, this will not work. hdfsusage.user should be optional.

      Show
      eyang Eric Yang added a comment - -1 sed 's:CHUKWA_USER=chukwa:CHUKWA_USER=% {hdfsusage_uid}:' | \ should be: sed 's:CHUKWA_USER=chukwa:CHUKWA_USER=${hdfsusage_uid} :' | \ With % {hdfsusage_uid} , when rpm.hdfsusage.user is undefined, this will not work. hdfsusage.user should be optional.
      Hide
      zhangyongjiang Cheng added a comment - - edited

      changed to %

      {hdfsusage_uid}

      . new patch submitted.

      Show
      zhangyongjiang Cheng added a comment - - edited changed to % {hdfsusage_uid} . new patch submitted.
      Hide
      eyang Eric Yang added a comment -

      +1, the new patch looks good.

      Show
      eyang Eric Yang added a comment - +1, the new patch looks good.
      Hide
      eyang Eric Yang added a comment -

      I just committed this, thanks Cheng.

      Show
      eyang Eric Yang added a comment - I just committed this, thanks Cheng.
      Hide
      hudson Hudson added a comment -
      Show
      hudson Hudson added a comment - Integrated in Chukwa-trunk #24 (See http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/24/ )
      Hide
      hudson Hudson added a comment -
      Show
      hudson Hudson added a comment - Integrated in Chukwa-trunk #45 (See http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/45/ )

        People

        • Assignee:
          zhangyongjiang Cheng
          Reporter:
          zhangyongjiang Cheng
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development