Bug 764011 (GLUSTER-2279) - rpm spec file changes
Summary: rpm spec file changes
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: GLUSTER-2279
Product: GlusterFS
Classification: Community
Component: unclassified
Version: 3.1.3
Hardware: All
OS: Linux
urgent
low
Target Milestone: ---
Assignee: Lakshmipathi G
QA Contact:
URL:
Whiteboard:
: GLUSTER-3218 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-01-06 10:05 UTC by Lakshmipathi G
Modified: 2015-12-01 16:45 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Regression: RTP
Mount Type: ---
Documentation: DP
CRM:
Verified Versions: 3.3beta2, 3.2.4qa2

Attachments (Terms of Use)
Creates and includes links to the /opt/usr/bin/gluster* binaries (1.78 KB, patch)
2011-03-16 03:54 UTC, Joe Julian 		no flags Details | Diff
Patch to correct ldconfig path addition. (1.49 KB, patch)
2011-04-08 15:53 UTC, Joe Julian 		no flags Details | Diff
Test Cases to verify the Bug (21.26 KB, application/vnd.oasis.opendocument.text)
2011-09-21 08:12 UTC, Shwetha H Panduranga 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Description Lakshmipathi G 2011-01-06 10:05:31 UTC 
move gluster,glusterfs,glusterd,glusterfsd binaries to /opt/glusterfs/<version> and create symbolic link at /usr/sbin/ . rpm install should start glusterd if glusterd was already running.
Comment 1 Anand Avati 2011-02-22 14:35:05 UTC 
PATCH: http://patches.gluster.com/patch/6239 in master (install gluster binaries under /opt/glusterfs/version with sym.link at /usr/bin)
Comment 2 Amar Tumballi 2011-03-03 03:37:01 UTC 
We need to mention in documentation that with 3.1.3 release, the binaries will be placed in '/opt/glusterfs/<version>'
Comment 3 Joe Julian 2011-03-16 01:44:08 UTC 
This patch breaks rpm upgrades from 3.1.(<3) as the remove process to remove the old files happens after postinstall. This causes the /usr/sbin/gluster* files to be removed.

1. rpm -i glusterfs-core-3.1.2-1.x86_64.rpm
2. rpm -Uvv glusterfs-core-3.1.3-1.x86_64.rpm
3. ls /usr/sbin/gluster*

expected result: 
/usr/sbin/gluster  /usr/sbin/glusterd  /usr/sbin/glusterfs  /usr/sbin/glusterfsd

actual result:
ls: /usr/sbin/gluster*: No such file or directory
Comment 4 Joe Julian 2011-03-16 03:54:49 UTC 
Created attachment 453
Comment 5 Lakshmipathi G 2011-03-17 10:14:56 UTC 
(In reply to comment #4)
> Created an attachment (id=453) [details]
> Creates and includes links to the /opt/usr/bin/gluster* binaries

thanks for the patch.We will apply it.
Comment 6 Vijay Bellur 2011-04-08 07:35:38 UTC 
PATCH: http://patches.gluster.com/patch/6755 in release-3.1 (preserve symlink during rpm upgrade and remove gsync references)
Comment 7 Joe Julian 2011-04-08 15:53:49 UTC 
Created attachment 475


The spec file also doesn't correctly create the library references for i386 installations, and doesn't correctly manage the ld.so.conf.d/glusterfs.conf file. This additional patch corrects that (thanks to Scott Zahn).
Comment 8 Joe Julian 2011-07-22 04:25:31 UTC 
*** Bug 3218 has been marked as a duplicate of this bug. ***
Comment 9 Lakshmipathi G 2011-07-26 08:44:54 UTC 
Thanks (In reply to comment #7)
> Created an attachment (id=475) [details]
> Patch to correct ldconfig path addition.
> 
> The spec file also doesn't correctly create the library references for i386
> installations, and doesn't correctly manage the ld.so.conf.d/glusterfs.conf
> file. This additional patch corrects that (thanks to Scott Zahn).

Added the ldconfig,these changes should be available in future releases.
Thanks JoeJulian,Scott Zahn.
Comment 10 Shwetha H Panduranga 2011-09-21 08:12:49 UTC 
Created attachment 666
Comment 11 Shwetha H Panduranga 2011-09-21 08:19:51 UTC 
Verified on the 3.3beta2 and 3.2.4qa2 (O.S: CentsOS). 

The ldconfig patch will be verified and bug will be updated by Lakshmipathi.
Comment 12 Anand Avati 2011-09-21 08:27:44 UTC 
CHANGE: http://review.gluster.com/470 (Thanks to) merged in release-3.2 by Vijay Bellur (vijay)
Comment 13 Anand Avati 2011-09-21 08:28:08 UTC 
CHANGE: http://review.gluster.com/466 (Thanks to) merged in master by Vijay Bellur (vijay)
Note You need to log in before you can comment on or make changes to this bug.