From b3fc2747dc714928aec4cecec0e9a34456d818aa Mon Sep 17 00:00:00 2001 From: Latha Sivakumar Date: Tue, 15 Dec 2020 19:15:53 -0500 Subject: [PATCH 1/3] (#54) Exception thrown whenn Nodes not in Ready state --- pre_install_report/README.md | 18 +++++++++++++++++- .../library/pre_install_utils.py | 2 +- 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/pre_install_report/README.md b/pre_install_report/README.md index a4059ca..d765daf 100644 --- a/pre_install_report/README.md +++ b/pre_install_report/README.md @@ -2,7 +2,7 @@ This tool compares your Kubernetes environment to the SAS Viya system requirements. It evaluates a number of items, such as memory, CPU cores, software versions, and permissions. The output is a web-viewable, HTML report with the results. -SAS recommends running the tool and resolving any reported issues before beginning a SAS Viya deployment in a Kubernetes cluster. +SAS recommends running the tool and resolving any reported issues before beginning a SAS Viya deployment in a Kubernetes cluster. ## Prerequisites - The tool should be run on a machine from which the Kubernetes command-line interface, `kubectl`, can access the Kubernetes cluster. @@ -20,6 +20,7 @@ Download the latest version of this tool and update required packages with every ## Usage **Note:** You must set your `KUBECONFIG` environment variable. `KUBECONFIG` must have administrator rights in the namespace where you intend to deploy your SAS Viya software. +To obtain a complete report use a `KUBECONFIG` with administrator rights in the cluster. After obtaining the latest version of this tool, cd to `/viya4-ark`. @@ -72,3 +73,18 @@ The tool generates the pre-install check report,`viya_pre_install_report_/viya4-ark/pre_install_report/viya_check_limit.properties file to alter the minimum and aggregate settings for CPU and memory on nodes. For more information, see the details in the file. + +## Known Issues +The following issue is known and may impact the performance and expected results of this tool. +- All Nodes in a cluster must be in READY state before running the tool. + - If all the Nodes are not in READY state, the tool takes longer to run. Wait for it to complete. + - Also, the tool may not be unable to clean up the pods and replicaset created in the specified namespace as shown below and must be manually deleted. + They will look similar to the resources shown below: +``` + NAME READY STATUS RESTARTS AGE + pod/hello-world-6665cf748b-5x2jq 0/1 Pending 0 115m + pod/hello-world-6665cf748b-tkq79 0/1 Pending 0 115m + + NAME DESIRED CURRENT READY AGE + replicaset.apps/hello-world-6665cf748b 2 2 0 115m +``` \ No newline at end of file diff --git a/pre_install_report/library/pre_install_utils.py b/pre_install_report/library/pre_install_utils.py index 4b0fe86..de9dfde 100644 --- a/pre_install_report/library/pre_install_utils.py +++ b/pre_install_report/library/pre_install_utils.py @@ -98,7 +98,7 @@ def get_rbac_group_cmd(self): role = data.get_api_group("Role") rolebinding = data.get_api_group("RoleBinding") except CalledProcessError as e: - self.logger.exception("get_rbac_group_cmd rc {}" + e.returncode) + self.logger.exception("get_rbac_group_cmd rc {}" + str(e.returncode)) return False if role is None: return False From 71ea02016b903883a1248b3bdb35fc81c76f409e Mon Sep 17 00:00:00 2001 From: FredPerrySAS <69848070+FredPerrySAS@users.noreply.github.com> Date: Wed, 16 Dec 2020 06:15:25 -0500 Subject: [PATCH 2/3] Edit README.md Before I edited this, the content said that that "the tool...must be manually deleted" if the output showed pods and replicaset had not been cleaned up. I changed this to say that the pods and replicaset had to be manually deleted. I'm pretty confident that this is correct, but I wanted to call it out to your attention to make sure it was the right thing to do. --- pre_install_report/README.md | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/pre_install_report/README.md b/pre_install_report/README.md index d765daf..e421c7f 100644 --- a/pre_install_report/README.md +++ b/pre_install_report/README.md @@ -75,10 +75,11 @@ The tool generates the pre-install check report,`viya_pre_install_report_/viya4-ark/pre_install_report/viya_check_limit.properties file to alter the minimum and aggregate settings for CPU and memory on nodes. For more information, see the details in the file. ## Known Issues -The following issue is known and may impact the performance and expected results of this tool. -- All Nodes in a cluster must be in READY state before running the tool. - - If all the Nodes are not in READY state, the tool takes longer to run. Wait for it to complete. - - Also, the tool may not be unable to clean up the pods and replicaset created in the specified namespace as shown below and must be manually deleted. + +The following issue may impact the performance and expected results of this tool. +- All Nodes in a cluster must be in the READY state before running the tool. + - If all the Nodes are not in the READY state, the tool takes longer to run. Wait for it to complete. + - Also, the tool may not be able to clean up the pods and replicaset created in the specified namespace as shown in the example output below. If that happens, the pods and replicaset must be manually deleted. They will look similar to the resources shown below: ``` NAME READY STATUS RESTARTS AGE @@ -87,4 +88,4 @@ The following issue is known and may impact the performance and expected results NAME DESIRED CURRENT READY AGE replicaset.apps/hello-world-6665cf748b 2 2 0 115m -``` \ No newline at end of file +``` From 4c94dbee108b2310a30be6bc9981623680d20572 Mon Sep 17 00:00:00 2001 From: Latha Sivakumar Date: Wed, 16 Dec 2020 10:25:31 -0500 Subject: [PATCH 3/3] (#54) Exception thrown when Nodes not in Ready state --- pre_install_report/README.md | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/pre_install_report/README.md b/pre_install_report/README.md index e421c7f..c8291ef 100644 --- a/pre_install_report/README.md +++ b/pre_install_report/README.md @@ -88,4 +88,9 @@ The following issue may impact the performance and expected results of this tool NAME DESIRED CURRENT READY AGE replicaset.apps/hello-world-6665cf748b 2 2 0 115m + + Suggested commands to delete resources before running the tool again: + kubectl -n delete replicaset.apps/hello-world-6665cf748b + kubectl -n delete pos/hello-world-6665cf748b-5x2jq + kubectl -n delete pod/hello-world-6665cf748b-tkq79 ```