Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.20.0
    • Component/s: None
    • Labels:
      None
    • Environment:

      Redhat 5.1, Java 6

    • Hadoop Flags:
      Reviewed
    • Release Note:
      Changed RPM install location to the value specified by build.properties file.

      Description

      When user defines location of Chukwa RPM as /usr/local/chukwa, the RPM should install into the defined directory.
      The rpm build currently creates /usr/local/chukwa/chukwa. This should be changed.

        Activity

        Hide
        Chris Douglas added a comment -

        I committed this. Thanks, Eric

        Show
        Chris Douglas added a comment - I committed this. Thanks, Eric
        Hide
        Eric Yang added a comment -
        • Fixed patch
        Show
        Eric Yang added a comment - Fixed patch
        Hide
        Eric Yang added a comment -

        Patch was not generated properly. Regenerating patch.

        Show
        Eric Yang added a comment - Patch was not generated properly. Regenerating patch.
        Hide
        Eric Yang added a comment -
        • Fixed corrupted patch
        Show
        Eric Yang added a comment - Fixed corrupted patch
        Hide
        Eric Yang added a comment -
        • Update RPM install path. The property specified in default.properties (build.properties) would be the top level directory for Chukwa.
        Show
        Eric Yang added a comment - Update RPM install path. The property specified in default.properties (build.properties) would be the top level directory for Chukwa.
        Hide
        Eric Yang added a comment -

        It's hardwired for RPM because the init.d script does not pass the location of the CHUKWA_CONF_DIR to the chukwa-daemon.sh script. Hence the checkpoint directory is defaulted to CHUKWA_HOME/var.

        Show
        Eric Yang added a comment - It's hardwired for RPM because the init.d script does not pass the location of the CHUKWA_CONF_DIR to the chukwa-daemon.sh script. Hence the checkpoint directory is defaulted to CHUKWA_HOME/var.
        Hide
        Ari Rabkin added a comment -

        How is it hardwired? I thought this was a conf file option, chukwaAgent.checkpoint.dir?

        Show
        Ari Rabkin added a comment - How is it hardwired? I thought this was a conf file option, chukwaAgent.checkpoint.dir?
        Hide
        Eric Yang added a comment -

        The checkpoint location is hardwired to CHUKWA_HOME/var for the RPM, this should be tweakable option at build time.

        Show
        Eric Yang added a comment - The checkpoint location is hardwired to CHUKWA_HOME/var for the RPM, this should be tweakable option at build time.
        Hide
        Eric Yang added a comment -

        another related defect in RPM installation procedure:

        Chukwa RPM is executing permission changes in the POST RPM install, and it should be changed to build phase. Hence,
        the rpm -v would not show:

        .....UG. /grid/0/chukwa/LICENSE.txt
        .....UG. /grid/0/chukwa/bin
        .....UG. /grid/0/chukwa/bin/VERSION
        .....UG. /grid/0/chukwa/bin/agent.sh
        .....UG. /grid/0/chukwa/bin/buildDailyArchive.sh
        .....UG. /grid/0/chukwa/bin/buildHourlyArchive.sh

        Show
        Eric Yang added a comment - another related defect in RPM installation procedure: Chukwa RPM is executing permission changes in the POST RPM install, and it should be changed to build phase. Hence, the rpm -v would not show: .....UG. /grid/0/chukwa/LICENSE.txt .....UG. /grid/0/chukwa/bin .....UG. /grid/0/chukwa/bin/VERSION .....UG. /grid/0/chukwa/bin/agent.sh .....UG. /grid/0/chukwa/bin/buildDailyArchive.sh .....UG. /grid/0/chukwa/bin/buildHourlyArchive.sh

          People

          • Assignee:
            Eric Yang
            Reporter:
            Eric Yang
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development