(#62) Pre_Install_Check: Add additional messages to report when All o…

…r Some Node(s) are not in READY state
sassoftware · Jan 28, 2021 · c0d15b2 · c0d15b2
1 parent f8c5b59
commit c0d15b2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pre_install_report/README.md b/pre_install_report/README.md
@@ -58,7 +58,7 @@ $ export INGRESS_HOST=externalIP=$(kubectl -n <ingress-namespace> get service <n
 $ export INGRESS_HTTP_PORT=$(kubectl -n <ingress-namespace> get service <nginx-ingress-controller-name> -o jsonpath='{.spec.ports[?(@.name=="http")].port}')
 $ export INGRESS_HTTPS_PORT=$(kubectl -n <ingress-namespace> get service <nginx-ingress-controller-name> -o jsonpath='{.spec.ports[?(@.name=="https")].port}')
 ```
-The command to determine the Ingress Host may be slightly different with Amazon EKS:
+The command to determine the Ingress Host may be slightly different with Amazon Elastic Kubernetes Service(EKS):
 ```
 $ export INGRESS_HOST=externalIP=$(kubectl -n <ingress-namespace> get service <nginx-ingress-controller-name> -o jsonpath='{.status.loadBalancer.ingress[*].hostname}')
 ```

diff --git a/pre_install_report/library/pre_install_check.py b/pre_install_report/library/pre_install_check.py
@@ -739,7 +739,7 @@ def evaluate_nodes(self, nodes_data, global_data, cluster_info, quantity_):
             alloc_memory = str(node['allocatableMemory'])
             total_memory = total_memory + quantity_(str(node['memory']))
             total_allocatable_memory = total_allocatable_memory + quantity_(alloc_memory)
-            # nodestatus = False
+
             try:
                 nodeReady = str(node['Ready'])
                 if nodeReady == "True":