Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1624006
Summary: | /var/run/gluster/metrics/ wasn't created automatically | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Pavel Znamensky <kompastver> |
Component: | core | Assignee: | Amar Tumballi <atumball> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 4.1 | CC: | amarts, atumball, bugs, kkeithle |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | glusterfs-6.0 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2018-12-26 16:27:38 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: | |
Embargoed: |
Description
Pavel Znamensky
2018-08-30 16:08:39 UTC
This is not a packaging bug. It's not the responsibility of packaging (i.e. the glusterfs.spec(.in) file) to make directories. Either the `make install` or the process — at runtime — should create the directories, just as already happens for (/var)/run/gluster/{bitd,glustershd,nfs,quotad,scrubshared_storage,snaps,vols} The packaging already "owns" (/var)/run/gluster, and all files and directories under it. (But the various distribution packagers may provide work-arounds until upstream catches up.) Amar, can the code create the directory in case it is missing? (/var)/run/gluster has permissions for the gluster group to create contents there. The directory will automatically be cleaned on reboot. Alternatively this could be done through a /etc/tmpfiles.d snippet. While adding the code, I thought of creating it automatically, but later felt, it can be done by application needing the 'metrics' too. (Like glustermetrics creates this directory). For 4.1.x series, I guess it should be just documentation bug, and we should be creating the directory in master for now, if everyone feels that is the right thing. (In reply to Amar Tumballi from comment #3) > While adding the code, I thought of creating it automatically, but later > felt, it can be done by application needing the 'metrics' too. (Like > glustermetrics creates this directory). I think that whatever generates the metrics should create the directory. Unless gluster gets instructed to place the metrics somewhere the application chooses (volume option?). > For 4.1.x series, I guess it should be just documentation bug, and we should > be creating the directory in master for now, if everyone feels that is the > right thing. This feels like a rather crucial failure, the missing directory prevents metrics from being stored. It would be just as easy and quick to fix the problem in 4.1.x as documenting a fix? I'd be in favour for a backport :) REVIEW: https://review.gluster.org/21141 (monitoring: create dump dir if it doesn't exist) posted (#1) for review on master by Amar Tumballi COMMIT: https://review.gluster.org/21141 committed in master by "Amar Tumballi" <amarts> with a commit message- monitoring: create dump dir if it doesn't exist Fixes: bz#1624006 Change-Id: Ie78be72e2492cd02c1376852bb90f1e6661d9bea Signed-off-by: Amar Tumballi <amarts> glusterfs-4.1.5-1.fc29 has been pushed to the Fedora 29 stable repository. If problems still persist, please make note of it in this bug report. This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-6.0, please open a new bug report. glusterfs-6.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution. [1] https://lists.gluster.org/pipermail/announce/2019-March/000120.html [2] https://www.gluster.org/pipermail/gluster-users/ |