Update github clone link in readme and clean up old comments

README.md

@@ -60,7 +60,7 @@ SageMaker HyperPod CLI currently supports start training job with:
 1. Clone and install the sagemaker-hyperpod-cli package.
 
     ```
-    git clone git@github.com:aws/sagemaker-hyperpod-cli.git
+    git clone https://github.com/aws/sagemaker-hyperpod-cli.git
     cd sagemaker-hyperpod-cli
     pip install .
     ```

helm_chart/HyperPodHelmChart/charts/neuron-device-plugin/templates/k8s-neuron-device-plugin.yaml

@@ -38,22 +38,6 @@ spec:
         nodeAffinity:
           requiredDuringSchedulingIgnoredDuringExecution:
             nodeSelectorTerms:
-              # Uncomment following matchExpressions if using k8s 1.16 or lower
-              #- matchExpressions:
-              #    - key: "beta.kubernetes.io/instance-type"
-              #      operator: In
-              #      values:
-              #        - inf1.xlarge
-              #        - inf1.2xlarge
-              #        - inf1.6xlarge
-              #        - inf1.24xlarge
-              #        - inf2.xlarge
-              #        - inf2.8xlarge
-              #        - inf2.24xlarge
-              #        - inf2.48xlarge
-              #        - trn1.2xlarge
-              #        - trn1.32xlarge
-              #        - trn1n.32xlarge
               - matchExpressions:
                   - key: "node.kubernetes.io/instance-type"
                     operator: In

helm_chart/HyperPodHelmChart/values.yaml

@@ -172,11 +172,6 @@ nvidia-device-plugin:
               - ml.p5.48xlarge
               - ml.p5e.48xlarge
               - ml.p5en.48xlarge
-          # nvidia plugin needs at least one node selector. Below label exists for all hyperpod nodes
-          - key: kubernetes.io/os
-            operator: In
-            values:
-            - "linux"
   tolerations:
     - key: nvidia.com/gpu
       operator: Exists