Bug 1689097

Summary: gfapi: provide an option for changing statedump path in glfs-api.
Product: [Community] GlusterFS Reporter: Amar Tumballi <atumball>
Component: coreAssignee: bugs <bugs>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: high Docs Contact:
Priority: high    
Version: mainlineCC: aliang, areis, berrange, chayang, coli, ddepaula, jen, juzhang, knoel, lolyu, mrezanin, ndevos, pasik, rcyriac, rhel8-maint, rhinduja, sasundar, sunkumar, virt-maint
Target Milestone: ---Keywords: FutureFeature, Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-7.0 Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: 1586201
: 1720461 1720557 1733127 (view as bug list) Environment:
Last Closed: 2020-02-10 17:46:21 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1447694, 1720461, 1720557    

Description Amar Tumballi 2019-03-15 06:25:13 UTC 
+++ This bug was initially created as a clone of Bug #1586201 +++

TL;DR: We need the ability to control the gluster logging from libvirt/QEMU when using the gluster driver.


+++ This bug was initially created as a clone of Bug #1447694 +++

QEMU is unable to store Gluster specific debugging logs under /var/run/gluster/... This is because QEMU runs as user "qemu" and does not have write permissions there. This debugging (called 'statedump') can be triggered from the Gluster CLI which sends an event to QEMU (when libgfapi is used).

When the glusterfs packages make sure that a group "gluster" has write permissions to /var/run/gluster/, could the qemu(-kvm-rhev) package be adjusted to have the "qemu" user in the "gluster" group? This would allow more debugging options that help with potential issues when QEMU runs with disk-images over libgfapi.so.
Comment 1 Worker Ant 2019-03-15 07:41:40 UTC 
REVIEW: https://review.gluster.org/22364 (gfapi: provide an api for setting statedump path) posted (#1) for review on master by Amar Tumballi
Comment 2 Amar Tumballi 2019-06-14 05:28:24 UTC 
The patch is now merged.
Comment 3 Worker Ant 2019-07-25 08:23:12 UTC 
REVIEW: https://review.gluster.org/23103 (gfapi: provide version for statedump path) posted (#1) for review on release-7 by Amar Tumballi
Comment 4 Worker Ant 2019-08-13 05:59:33 UTC 
REVIEW: https://review.gluster.org/23102 (gfapi: provide version for statedump path) merged (#3) on master by Amar Tumballi
Comment 6 Sunny Kumar 2020-02-10 17:46:21 UTC 
Patch is merged closing this bug now.