Bug 1024505

Summary: fs.sh: Fix "support" of tmpfs
Product: Red Hat Enterprise Linux 6 Reporter: Jaroslav Kortus <jkortus>
Component: resource-agentsAssignee: Oyvind Albrigtsen <oalbrigt>
Status: CLOSED ERRATA QA Contact: cluster-qe <cluster-qe>
Severity: low Docs Contact:
Priority: low    
Version: 6.5CC: agk, cfeist, cluster-maint, fdinitto, mjuricek, mnovacek
Target Milestone: rcKeywords: EasyFix
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: resource-agents-3.9.5-28.el6 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-05-10 19:12:59 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:
Attachments:
Description Flags
Working patch
none
Updated working patch none

Description Jaroslav Kortus 2013-10-29 19:53:30 UTC
Description of problem:
Currently tmpfs is recognized as a supported filesystem of fs.sh resource.
This filesystem cannot work in current implementation ("none" device is not supported and later it requires fsck).

As probably no one has ever run it, we could drop it from the agent.

Version-Release number of selected component (if applicable):
resource-agents-3.9.2-40.el6.x86_64

How reproducible:
always

Steps to Reproduce:
1. add <fs device="/dev/loop0" fstype="tmpfs" mountpoint="/tmp/fstest" name="filesystem"/> to your cluster service
2.
3.

Actual results:
tmpfs cannot be mounted via rgmanager

Expected results:
message that tmpfs is not supported

Additional info:

Comment 5 Oyvind Albrigtsen 2015-11-02 10:39:17 UTC
Created attachment 1088533 [details]
Working patch

Comment 6 Oyvind Albrigtsen 2015-11-02 10:40:31 UTC
Tested patch. Had to change the return code from OCF_ERR_ARGS as utils/fs-lib.sh thought that means success.

Comment 7 Oyvind Albrigtsen 2015-11-09 12:44:22 UTC
Created attachment 1091728 [details]
Updated working patch

Improved the patch and tested that it's working without any issues.

Comment 8 Oyvind Albrigtsen 2015-12-21 12:26:05 UTC
Before:
- Error about none-device not being supported, or error when running fsck if device is set to e.g. /dev/loop0.

After:
- Message that tmpfs is not supported.

Comment 9 Oyvind Albrigtsen 2015-12-24 13:09:41 UTC
Setup:
Add the following service to /etc/cluster/cluster.conf:
          <service autostart="0" domain="fodom" name="tmpfs">
               <fs device="/dev/loop0" fstype="tmpfs" mountpoint="/tmp/fstest" name="filesystem"/>
          </service>


Before:
# rpm -q resource-agents
resource-agents-3.9.5-24.el6_7.1.x86_64
# clusvcadm -e tmpfs
# tail -f /var/log/cluster/rgmanager.log
Dec 24 14:05:57 rgmanager Starting disabled service service:tmpfs
Dec 24 14:05:57 rgmanager [fs] start_filesystem: Creating mount point /tmp/fstest for device /dev/loop0
Dec 24 14:05:57 rgmanager [fs] Unknown file system type 'tmpfs' for device /dev/loop0.  Assuming fsck is required.
Dec 24 14:05:57 rgmanager [fs] Running fsck on /dev/loop0
Dec 24 14:05:57 rgmanager [fs] 'fsck -p /dev/loop0' failed, error=8; check /var/log/cluster/loop0.fsck.log for errors
Dec 24 14:05:57 rgmanager [fs] Invalidating buffers for /dev/loop0


After:
# rpm -q resource-agents
resource-agents-3.9.5-28.el6.x86_64
# clusvcadm -e tmpfs
# tail -f /var/log/cluster/rgmanager.log
Dec 24 14:06:57 rgmanager Starting disabled service service:tmpfs
Dec 24 14:06:57 rgmanager [fs] File system type tmpfs not supported

Comment 13 errata-xmlrpc 2016-05-10 19:12:59 UTC
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://rhn.redhat.com/errata/RHBA-2016-0735.html