Bug 1570958
| Summary: | [RFE] GlusterFS client mount should not fail if /var/log/glusterfs is not present. | ||
|---|---|---|---|
| Product: | [Red Hat Storage] Red Hat Gluster Storage | Reporter: | Ryan Howe <rhowe> |
| Component: | glusterfs | Assignee: | Amar Tumballi <atumball> |
| Status: | CLOSED ERRATA | QA Contact: | Bala Konda Reddy M <bmekala> |
| Severity: | medium | Docs Contact: | |
| Priority: | low | ||
| Version: | cns-3.9 | CC: | amukherj, atumball, jcrumple, kiyer, nchilaka, rhs-bugs, sankarshan, sheggodu, smali, storage-qa-internal, vbellur |
| Target Milestone: | --- | Keywords: | FutureFeature, ZStream |
| Target Release: | RHGS 3.4.z Batch Update 4 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | glusterfs-3.12.2-41 | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2019-03-27 03:43:39 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: | |||
Considering we do similar (mkdir_p()) directory creation for different files (like monitor, statedump etc), we should do this for logging too IMO. Upstream Fix: https://review.gluster.org/#/c/glusterfs/+/21536 Upstream fix is already in. Proposing this for 3.4 BU4. > Scenario 1: Valid test case, but no need for step 6 Before the fix: * as the mount itself would fail, that can be checked with df (like step 7) After the fix: * df -h would show successful mount. > Scenario 2: Not valid test case, ie, behavior would be no changes to process, but user wouldn't see log files, as the files were removed after they are 'opened'. Which means, fd will be valid, but the entries are deleted. The behavior would be expected that, there would be no /var/log/glusterfs at the end of it. Please test, but the expected result is, no crashes in process, but no logs. > Scenario 3: I am not sure of the outcome. Technically, even with permission 000, a process started by root would be able to use the files. So it may work without issues IMO. Please try and see what to expect. ---------------- Regardless of all these, a user can easily hit this bug if they install glusterfs through RPM on a machine, which is expected to be a client, and they don't install glusterfs-server part of the RPM. (because earlier, RPM used to create /var/log/glusterfs before installing glusterd etc). Now, if you try to mount client, (regardless of volume type, it can be tested with just 1 brick in volume), it would not start because there is no /var/log/glusterfs (ie, before this version). With the fix, the mount works fine. Hope this gives the idea for what does it involve for testing this fix. Ack! But interestingly, IMO, we mostly need a separate Blocker bug for this, as without the fix, you wouldn't even succeed to mount. I am looking at the issue, and will update what is the issue soon. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2019:0658 |
Description of problem: If /var/log/glusterfs gets removed, a mount.glusterfs should still be possible and not fail. Version-Release number of selected component (if applicable): glusterfs 3.8.4-53.el7 How reproducible: 100% Steps to Reproduce: 1. Remove /var/log/* 2. mount.glusterfs host:/test /mnt/ Actual results: Fail Error mounting /tmp/openshift-glusterfs-registry-niCewy: ERROR: failed to create logfile "/var/log/glusterfs/mnt.log" (No such file or directory) ERROR: failed to open logfile /var/log/glusterfs/mnt.log Mount failed. Please check the log file for more details. Expected results: Success or creating of log file.