Bug 1381447

Summary: "glusterfs-events" package installation is happening when gluster processes are running.
Product: Red Hat Gluster Storage Reporter: Byreddy <bsrirama>
Component: buildAssignee: Milind Changire <mchangir>
Status: CLOSED WORKSFORME QA Contact: Rahul Hinduja <rhinduja>
Severity: high Docs Contact:
Priority: unspecified    
Version: rhgs-3.2CC: amukherj, bsrirama, rhs-bugs, storage-qa-internal
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-10-18 09:14:29 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Byreddy 2016-10-04 06:52:49 UTC
Description of problem:
=======================
"glusterfs-events"  package installation is happening when gluster processes are running.



glusterfs_build2]# yum install glusterfs-events-3.8.4-2.el7rhgs.x86_64.rpm
Loaded plugins: langpacks, product-id, rhnplugin, search-disabled-repos, subscription-manager
This system is receiving updates from RHN Classic or Red Hat Satellite.
Examining glusterfs-events-3.8.4-2.el7rhgs.x86_64.rpm: glusterfs-events-3.8.4-2.el7rhgs.x86_64
Marking glusterfs-events-3.8.4-2.el7rhgs.x86_64.rpm to be installed
Resolving Dependencies
--> Running transaction check
---> Package glusterfs-events.x86_64 0:3.8.4-2.el7rhgs will be installed
--> Finished Dependency Resolution

Dependencies Resolved

==============================================================================================================================================================================================
 Package                                   Arch                            Version                                    Repository                                                         Size
==============================================================================================================================================================================================
Installing:
 glusterfs-events                          x86_64                          3.8.4-2.el7rhgs                            /glusterfs-events-3.8.4-2.el7rhgs.x86_64                           82 k

Transaction Summary
==============================================================================================================================================================================================
Install  1 Package

Total size: 82 k
Installed size: 82 k
Is this ok [y/d/N]: y
Downloading packages:
Running transaction check
Running transaction test
Transaction test succeeded
Running transaction
  Installing : glusterfs-events-3.8.4-2.el7rhgs.x86_64                                                                                                                                    1/1 
  Verifying  : glusterfs-events-3.8.4-2.el7rhgs.x86_64                                                                                                                                    1/1 

Installed:
  glusterfs-events.x86_64 0:3.8.4-2.el7rhgs                                                                                                                                                   

Complete!
[root@glusterfs_build2]# 

~]# rpm -qa |grep glusterfs-events
glusterfs-events-3.8.4-2.el7rhgs.x86_64



Version-Release number of selected component (if applicable):
=============================================================
glusterfs-3.8.4-2

How reproducible:
==================
Always


Steps to Reproduce:
====================
1.Have glusterfs processes running setup
2.Install the glusterfs-events package


Actual results:
===============
"glusterfs-events"  package installation is happening when gluster processes are running.


Expected results:
=================
Package installation should not happen if gluster processes are running.


Additional info:

Comment 2 Atin Mukherjee 2016-10-04 07:30:26 UTC
On what basis do you think that glusterfs-events package shouldn't be allowed to install when there is running glusterfs(d) process instances?

Comment 3 Milind Changire 2016-10-18 05:11:31 UTC
Normally even Operating System packages are updated when the operating system is running. eg. glibc, which is the most wide used shared library is updated when processes are actively using the library. The newer library is loaded when the process is restarted, and until then the processes already running continue to use the older library.

Admins are advised to follow standard package updating guidelines, which involves acquiescing the node, i.e. shutting down gluster processes, to update the RPMs.

Adding checks to RPMs is not the solution.

Comment 4 Byreddy 2016-10-18 05:28:07 UTC
(In reply to Milind Changire from comment #3)
> Normally even Operating System packages are updated when the operating
> system is running. eg. glibc, which is the most wide used shared library is
> updated when processes are actively using the library. The newer library is
> loaded when the process is restarted, and until then the processes already
> running continue to use the older library.
> 
> Admins are advised to follow standard package updating guidelines, which
> involves acquiescing the node, i.e. shutting down gluster processes, to
> update the RPMs.
> 
> Adding checks to RPMs is not the solution.


I am not agreeing this is NOT A BUG.

This is my explanation:
Except this events package,  we are not allowing to install any other gluster packages if gluster processes are running
we need to follow the same procedure for this gluster events package as well.

Moving the bug back to assigned state.

Comment 5 Atin Mukherjee 2016-10-18 05:37:25 UTC
(In reply to Byreddy from comment #4)
> (In reply to Milind Changire from comment #3)
> > Normally even Operating System packages are updated when the operating
> > system is running. eg. glibc, which is the most wide used shared library is
> > updated when processes are actively using the library. The newer library is
> > loaded when the process is restarted, and until then the processes already
> > running continue to use the older library.
> > 
> > Admins are advised to follow standard package updating guidelines, which
> > involves acquiescing the node, i.e. shutting down gluster processes, to
> > update the RPMs.
> > 
> > Adding checks to RPMs is not the solution.
> 
> 
> I am not agreeing this is NOT A BUG.
> 
> This is my explanation:
> Except this events package,  we are not allowing to install any other
> gluster packages if gluster processes are running
> we need to follow the same procedure for this gluster events package as well.
> 
> Moving the bug back to assigned state.

As discussed, the point here is disrupting the running services is an overhead and should be only done if there is an impact on a fresh package installation if we don't bring down the other running instances. For glustereventsd as of now, we don't see any reason of introducing that unnecessary overhead to disrupting the running services.

I believe this is the last clarification what you needed and this bug now can be closed. Please ack.

Comment 6 Byreddy 2016-10-18 08:52:59 UTC
(In reply to Atin Mukherjee from comment #5)
> (In reply to Byreddy from comment #4)
> > (In reply to Milind Changire from comment #3)
> > > Normally even Operating System packages are updated when the operating
> > > system is running. eg. glibc, which is the most wide used shared library is
> > > updated when processes are actively using the library. The newer library is
> > > loaded when the process is restarted, and until then the processes already
> > > running continue to use the older library.
> > > 
> > > Admins are advised to follow standard package updating guidelines, which
> > > involves acquiescing the node, i.e. shutting down gluster processes, to
> > > update the RPMs.
> > > 
> > > Adding checks to RPMs is not the solution.
> > 
> > 
> > I am not agreeing this is NOT A BUG.
> > 
> > This is my explanation:
> > Except this events package,  we are not allowing to install any other
> > gluster packages if gluster processes are running
> > we need to follow the same procedure for this gluster events package as well.
> > 
> > Moving the bug back to assigned state.
> 
> As discussed, the point here is disrupting the running services is an
> overhead and should be only done if there is an impact on a fresh package
> installation if we don't bring down the other running instances. For
> glustereventsd as of now, we don't see any reason of introducing that
> unnecessary overhead to disrupting the running services.
> 
> I believe this is the last clarification what you needed and this bug now
> can be closed. Please ack.


python-gluster is a dependency package for glusterfs-events, python-gluster package will not get installed if gluster processes are running so we have to disrupt the running services to install glusterfs-events package.


With out disrupting the existing setup we can't install/update the glusterfs-events package

Comment 7 Byreddy 2016-10-18 09:14:29 UTC
Closing this bug based on above comment details.
We are achieving the required thing with the dependency package.