Bug 1478073

Summary: Uncomment cephx: true as default value in all.yml.sample
Product: [Red Hat Storage] Red Hat Ceph Storage Reporter: khartsoe <khartsoe>
Component: DocumentationAssignee: Erin Donnelly <edonnell>
Status: CLOSED CURRENTRELEASE QA Contact: Vasishta <vashastr>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.0CC: adeza, aschoen, ceph-eng-bugs, edonnell, gmeno, hnallurv, kdreyer, nthomas, sankarshan, seb
Target Milestone: rc   
Target Release: 3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-12-26 03:38:42 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 khartsoe@redhat.com 2017-08-03 14:16:24 UTC 
Description of problem:
Currently, user is required to manually uncomment "cephx: true" when configuring ceph global settings. 

This requires an additional manual task during installation.

Version-Release number of selected component (if applicable):
2.x, etc.

How reproducible:
As root, create an all.yml file from the all.yml.sample file, uncomment "cephx: true"

Steps to Reproduce:
Step 8:
https://access.redhat.com/documentation/en-us/red_hat_ceph_storage/2/html-single/installation_guide_for_red_hat_enterprise_linux/#configuring_ceph_global_settings

Actual results:
all.yml.sample file default value "cephx: true" is commented out

Expected results:
all.yml.sample file default value "cephx: true" should be uncommented to prevent user action and therefore, reduce required user tasks during installation

Additional info:
https://docs.google.com/document/d/14c6cx2PY6xe2XC0N9sFY8Jc-VobAy_LjpTxJsrOL_q8/edit

Driven by RHCS-LT Process Improvement effort
Comment 2 seb 2017-08-03 14:50:35 UTC 
The doc needs to be reworked, this action is not needed.

Bara, can you move this to doc?
Thanks!
Comment 4 seb 2017-08-03 15:08:14 UTC 
Correct. It has always been set to true by default.
Thanks Bara.
Comment 6 seb 2017-08-04 16:41:20 UTC 
There is no need to uncomment it to give it its default value. You uncomment a parameter to **only** change its default value
Comment 9 Vasishta 2017-10-05 15:00:30 UTC 
Hi,

Verified in the doc that, redundant step asking user to set cephx to true has been removed and verified that by default cephx has been enabled.

$ cat group_vars/all.yml |grep cephx
#cephx: true
$ sudo ceph daemon /var/run/ceph/beta_2017-mon.magna009.asok config show |grep cephx
    "auth_client_required": "cephx, none",
    "auth_cluster_required": "cephx",
    "auth_service_required": "cephx",

Moving to VERIFIED state.

Regards,
Vasishta