Issue Details (XML | Word | Printable)

Key: HADOOP-5030
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Eric Yang
Reporter: Eric Yang
Votes: 0
Watchers: 5
Operations

If you were logged in you would be able to see more operations.
Hadoop Common

Chukwa RPM build improvements

Created: 14/Jan/09 06:47 PM   Updated: 08/Jul/09 04:40 PM
Return to search
Component/s: None
Affects Version/s: None
Fix Version/s: 0.20.0

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works HADOOP-5030-1.patch 2009-01-16 11:37 PM Eric Yang 3 kB
Environment: Redhat 5.1, Java 6

Hadoop Flags: Reviewed
Release Note: Changed RPM install location to the value specified by build.properties file.
Resolution Date: 17/Jan/09 03:26 AM


 Description  « Hide
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.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Eric Yang added a comment - 15/Jan/09 06:14 PM
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


Eric Yang added a comment - 15/Jan/09 06:25 PM
The checkpoint location is hardwired to CHUKWA_HOME/var for the RPM, this should be tweakable option at build time.

Eric Yang made changes - 15/Jan/09 06:26 PM
Field Original Value New Value
Summary Chukwa RPM build should install to the user defined directory rahter than subdirectory Chukwa RPM build improvements
Ari Rabkin added a comment - 15/Jan/09 06:28 PM
How is it hardwired? I thought this was a conf file option, chukwaAgent.checkpoint.dir?

Eric Yang added a comment - 15/Jan/09 07:00 PM
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.

Eric Yang added a comment - 16/Jan/09 02:43 AM
  • Update RPM install path. The property specified in default.properties (build.properties) would be the top level directory for Chukwa.

Eric Yang made changes - 16/Jan/09 02:43 AM
Attachment HADOOP-5030.patch [ 12398039 ]
Eric Yang made changes - 16/Jan/09 02:45 AM
Release Note What is new in HADOOP-5030:

- Changed RPM install location to the value specified by build.properties file.
Status Open [ 1 ] Patch Available [ 10002 ]
Eric Yang added a comment - 16/Jan/09 11:25 PM
  • Fixed corrupted patch

Eric Yang made changes - 16/Jan/09 11:25 PM
Attachment HADOOP-5030.patch [ 12398120 ]
Eric Yang added a comment - 16/Jan/09 11:27 PM
Patch was not generated properly. Regenerating patch.

Eric Yang made changes - 16/Jan/09 11:27 PM
Status Patch Available [ 10002 ] Open [ 1 ]
Eric Yang added a comment - 16/Jan/09 11:37 PM
  • Fixed patch

Eric Yang made changes - 16/Jan/09 11:37 PM
Attachment HADOOP-5030-1.patch [ 12398121 ]
Eric Yang made changes - 16/Jan/09 11:37 PM
Attachment HADOOP-5030.patch [ 12398120 ]
Eric Yang made changes - 16/Jan/09 11:37 PM
Attachment HADOOP-5030.patch [ 12398039 ]
Repository Revision Date User Message
ASF #735220 Sat Jan 17 03:25:46 UTC 2009 cdouglas HADOOP-5030. Build Chukwa RPM to install into configured directory. Contributed by Eric Yang
Files Changed
MODIFY /hadoop/core/trunk/CHANGES.txt
MODIFY /hadoop/core/trunk/src/contrib/chukwa/build.xml

Repository Revision Date User Message
ASF #735221 Sat Jan 17 03:25:58 UTC 2009 cdouglas HADOOP-5030. Build Chukwa RPM to install into configured directory. Contributed by Eric Yang
Files Changed
MODIFY /hadoop/core/branches/branch-0.20/src/contrib/chukwa/build.xml
MODIFY /hadoop/core/branches/branch-0.20/CHANGES.txt

Chris Douglas added a comment - 17/Jan/09 03:26 AM
I committed this. Thanks, Eric

Chris Douglas made changes - 17/Jan/09 03:26 AM
Resolution Fixed [ 1 ]
Release Note What is new in HADOOP-5030:

- Changed RPM install location to the value specified by build.properties file.
Changed RPM install location to the value specified by build.properties file.
Fix Version/s 0.20.0 [ 12313438 ]
Hadoop Flags [Reviewed]
Status Open [ 1 ] Resolved [ 5 ]
Nigel Daley made changes - 23/Apr/09 07:18 PM
Status Resolved [ 5 ] Closed [ 6 ]
Owen O'Malley made changes - 08/Jul/09 04:40 PM
Component/s contrib/chukwa [ 12312445 ]