From 6a6835f5db9497f2b5867652e1f4d9883b32638c Mon Sep 17 00:00:00 2001
From: Christopher McGrath <cmcgrath@oteemo.com>
Date: Wed, 6 Oct 2021 15:20:38 -0400
Subject: [PATCH] #809, bugfix: sudo missing multiple places in step 4

---
 .../guides/deployment_scenarios/quickstart.md | 36 ++++++++-----------
 1 file changed, 14 insertions(+), 22 deletions(-)

diff --git a/docs/guides/deployment_scenarios/quickstart.md b/docs/guides/deployment_scenarios/quickstart.md
index 7d583d4a0f..1f696fe4c5 100644
--- a/docs/guides/deployment_scenarios/quickstart.md
+++ b/docs/guides/deployment_scenarios/quickstart.md
@@ -264,37 +264,29 @@ Note: This guide follows the DevOps best practice of left-shifting feedback on m
 
     ```shell
     # [ubuntu@Ubuntu_VM:~]
-    # ECK implementation of ElasticSearch needs the following or will see OOM errors
+    # Needed for ECK to run correctly without OOM errors
     sudo sysctl -w vm.max_map_count=524288
+    # Alternatively can use: 
+    # echo 'vm.max_map_count=524288' | sudo tee -a /etc/sysctl.d/vm-max_map_count.conf
 
-    # SonarQube host OS pre-requisites
+    # Needed by Sonarqube
     sudo sysctl -w fs.file-max=131072
+    # Alternatively can use:  
+    # echo 'fs.file-max=131072' | sudo tee -a /etc/sysctl.d/fs-file-max.conf
+
+    # Also Needed by Sonarqube
     ulimit -n 131072
     ulimit -u 8192
 
-    # Needed for ECK to run correctly without OOM errors
-    echo 'vm.max_map_count=524288' > /etc/sysctl.d/vm-max_map_count.conf
-
-    # Needed by Sonarqube
-    echo 'fs.file-max=131072' > /etc/sysctl.d/fs-file-max.conf
-
     # Load updated configuration
-    sysctl --load
-
-    # Alternative form of above 3 commands:
-    # sudo sysctl -w vm.max_map_count=524288
-    # sudo sysctl -w fs.file-max=131072
-
-    # Needed by Sonarqube
-    ulimit -n 131072
-    ulimit -u 8192
+    sudo sysctl --load
 
-    # Preload kernel modules required by istio-init, required for SELinux enforcing instances using istio-init
-    modprobe xt_REDIRECT
-    modprobe xt_owner
-    modprobe xt_statistic
+    # Preload kernel modules, required by istio-init running on SELinux enforcing instances
+    sudo modprobe xt_REDIRECT
+    sudo modprobe xt_owner
+    sudo modprobe xt_statistic
 
-    # Persist modules after reboots
+    # Persist kernel modules settings after reboots
     printf "xt_REDIRECT\nxt_owner\nxt_statistic\n" | sudo tee -a /etc/modules
 
     # Kubernetes requires swap disabled
-- 
GitLab