Bug 1375350

Summary: [DOCS] Problems with the Swap section in the documentation
Product: OpenShift Container Platform Reporter: Eric Jones <erjones>
Component: DocumentationAssignee: brice <bfallonf>
Status: CLOSED CURRENTRELEASE QA Contact: Xiaowei Li <xiaoli>
Severity: high Docs Contact: Vikram Goyal <vigoyal>
Priority: high    
Version: 3.2.1CC: aos-bugs, erjones, jokerman, mmccomas
Target Milestone: ---   
Target Release: ---   
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: 2016-11-22 00:36:24 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:

Description Eric Jones 2016-09-12 18:59:52 UTC
Document URL: 
https://docs.openshift.com/enterprise/3.2/admin_guide/overcommit.html#disabling-swap-memory

Section Number and Name: 
Cluster Administration
Overcommitting
Disabling Swap Memory

Describe the issue: 
This section is contradictory and is actually incorrect after discussing with engineering.

Suggestions for improvement: 
Work on this PR and then accept it:
https://github.com/openshift/openshift-docs/pull/2690

This line specifically (wrt this issue)
https://github.com/openshift/openshift-docs/pull/2690/files#diff-4d823c0a78f1894abddc191328c3efa2R340

Comment 1 brice 2016-10-11 05:15:31 UTC
PR created:

https://github.com/openshift/openshift-docs/pull/3023

Waiting on comments from Dev

Comment 2 brice 2016-10-19 01:12:13 UTC
After talking to Derek in my followup PR above, there's talks to maybe change the swappoff behaviour for the next release.

Eric, as the reporter of this BZ, is that enough? If you have any comments on my rewrites of the section, go ahead. Thanks!

Comment 3 Eric Jones 2016-10-24 20:29:22 UTC
My apologies for the delay, I was OoO. 

If Derek has approved the changes then I have no issue with them either.

I did check them briefly to be safe though and LGTM.

Thanks

Comment 4 openshift-github-bot 2016-11-08 04:53:55 UTC
Commit pushed to master at https://github.com/openshift/openshift-docs

https://github.com/openshift/openshift-docs/commit/611370addbc1cdc58133c5d4d9e512a9c2a1d333
Merge pull request #3023 from bfallonf/swap-1375350

Bug 1375350 Edits to OOR info