Bug 1680608

Summary: SELinux labels are missing on the gluster bricks created using gluster-ansible-roles
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: SATHEESARAN <sasundar>
Component: gluster-ansibleAssignee: Sachidananda Urs <surs>
Status: CLOSED NOTABUG QA Contact: SATHEESARAN <sasundar>
Severity: high Docs Contact:
Priority: unspecified    
Version: rhgs-3.4CC: rhs-bugs, sabose, sankarshan, sasundar
Target Milestone: ---   
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:
: 1680609 (view as bug list) Environment:
Last Closed: 2019-02-26 12:48:18 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
inventory_file none

Description SATHEESARAN 2019-02-25 12:09:25 UTC 
Description of problem:
-----------------------
Gluster bricks ( XFS ) created using gluster ansible roles are missing the selinux labels.

Version-Release number of selected component (if applicable):
-------------------------------------------------------------
gluster-ansible-role-1.0.4-3

How reproducible:
-----------------
Always

Steps to Reproduce:
-------------------
1. Create bricks using gluster ansible roles

Actual results:
---------------
SELinux labels are missing on the gluster bricks

Expected results:
-----------------
SELinux labels should be available on the gluster bricks

Additional info:
----------------
This bug was already showed up earlier and fixed with gluster-ansible-roles-1.0.3.( https://bugzilla.redhat.com/show_bug.cgi?id=1638940 ) Marking this bug as 'Regression' for that reason
Comment 1 SATHEESARAN 2019-02-25 12:12:21 UTC 
Created attachment 1538420 [details]
inventory_file
Comment 2 SATHEESARAN 2019-02-26 12:48:18 UTC 
gluster-ansible roles already provides the param 'gluster_set_selinux_labels: true' which is missed in the construction of the playbook.
When this param is used, selinux labels are added to the brick paths