Bug 1375350 - [DOCS] Problems with the Swap section in the documentation
Summary: [DOCS] Problems with the Swap section in the documentation
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Documentation
Version: 3.2.1
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: ---
Assignee: brice
QA Contact: Xiaowei Li
Vikram Goyal
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-12 18:59 UTC by Eric Jones
Modified: 2019-12-16 06:43 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-11-22 00:36:24 UTC
Target Upstream Version:


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.