Bug 1679862

Summary: Document steps for HEKETI_ZONE_CHECKING RFE
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: Rachael <rgeorge>
Component: doc-Container_Native_Storage_with_OpenShiftAssignee: Chandrakanth Pai <chpai>
Status: CLOSED CURRENTRELEASE QA Contact: Rachael <rgeorge>
Severity: high Docs Contact:
Priority: unspecified    
Version: ocs-3.11CC: asriram, chpai, jmulligan, knarra, puebele, rhs-bugs, rtalur, sankarshan, storage-doc
Target Milestone: ---Keywords: ZStream
Target Release: OCS 3.11.z Batch Update 2   
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: 2019-04-01 06:08:34 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:
Bug Depends On: 1635204    
Bug Blocks: 1672844    
Attachments:
Description Flags
PDF version of the OCS 3.11 Deployment Guide
none
PDF version of the OCS 3.11 Deployment Guide
none
PDF version of the OCS 3.11 Deployment Guide
none
PDF version of the OCS 3.11 Deployment Guide
none
PDF version of the OCS 3.11 Deployment Guide none

Comment 5 RamaKasturi 2019-03-05 10:52:42 UTC
Hello John,

   Can we also mention about what is the recommended way for customers or users to set this option ? I see that is missing in the above comment.

Thanks
kasturi

Comment 7 RamaKasturi 2019-03-05 17:40:10 UTC
sorry if i was not clear the way i asked the question.

I was expecting us to add a line in the document saying "though there are two ways of enabling this feautre at 3.11.2, recommended way of doing would be via storage class options" . I was looking for the above statement as well in the doc inputs you provided.

Hope i am clear now.

Comment 8 John Mulligan 2019-03-08 19:39:46 UTC
(In reply to RamaKasturi from comment #7)
> sorry if i was not clear the way i asked the question.
> 
> I was expecting us to add a line in the document saying "though there are
> two ways of enabling this feautre at 3.11.2, recommended way of doing would
> be via storage class options" . I was looking for the above statement as
> well in the doc inputs you provided.
> 
> Hope i am clear now.

Sure. That sounds OK to me. I'm still not 100% on documenting the latter but I'm not dead set against it.

Comment 11 Chandrakanth Pai 2019-03-19 10:49:47 UTC
Hi John,

The changes suggested by Rachael involve adding the glusterfs_zone variable in the examples shown for the glusterfs and glusterfs_registry section in the inventory file (Chapters 4 and 5 of the Deployment Guide). I have made this change as per Rachael's inputs in the bug.

She has also requested to add a separate note/section on how to use the heketi zone checking feature and any pre-requisites required for the same.
As per my understanding, you have suggested not to document all the ways to configure the storageclass. Kasturi suggested adding a line "There are two ways of enabling this feature at 3.11.2, recommended way of doing would be via storage class options" and you have disagreed with the latter part of her suggestion.

Therefore, could you please let me know your thoughts.
It would be helpful if you can provide your inputs at the earliest as I need to address this bug by today as we are nearing the GA. 

Thanks,
Chandrakanth

Comment 12 John Mulligan 2019-03-19 14:11:53 UTC
(In reply to Chandrakanth Pai from comment #11)
> Hi John,
> 
> The changes suggested by Rachael involve adding the glusterfs_zone variable
> in the examples shown for the glusterfs and glusterfs_registry section in
> the inventory file (Chapters 4 and 5 of the Deployment Guide). I have made
> this change as per Rachael's inputs in the bug.

Sounds good.

> 
> She has also requested to add a separate note/section on how to use the
> heketi zone checking feature and any pre-requisites required for the same.
> As per my understanding, you have suggested not to document all the ways to
> configure the storageclass. Kasturi suggested adding a line "There are two
> ways of enabling this feature at 3.11.2, recommended way of doing would be
> via storage class options" and you have disagreed with the latter part of
> her suggestion.
> 
> Therefore, could you please let me know your thoughts.
> It would be helpful if you can provide your inputs at the earliest as I need
> to address this bug by today as we are nearing the GA. 
> 

Sorry if I was unclear. I _do_ want to document the method of enabling strict zone checking using the storage class. I _do not_ want to document enabling strict zone checking using the ZONE_CHECKING environment variable at this time.

Please let me know if this response is sufficient to answer your questions or you need more details.

Comment 16 Chandrakanth Pai 2019-03-21 05:16:54 UTC
Created attachment 1546307 [details]
PDF version of the OCS 3.11 Deployment Guide

Comment 19 Chandrakanth Pai 2019-03-22 04:51:20 UTC
Created attachment 1546761 [details]
PDF version of the OCS 3.11 Deployment Guide

Comment 23 Chandrakanth Pai 2019-03-22 11:33:06 UTC
Created attachment 1546826 [details]
PDF version of the OCS 3.11 Deployment Guide

Comment 27 Chandrakanth Pai 2019-03-23 00:39:50 UTC
Created attachment 1547176 [details]
PDF version of the OCS 3.11 Deployment Guide

Comment 28 Chandrakanth Pai 2019-03-24 22:31:51 UTC
Created attachment 1547532 [details]
PDF version of the OCS 3.11 Deployment Guide