Skip to content

Commit feb2a7f

Browse files
authored
Minor fixes (#280)
* remove aemm from sqs tests * align helm install instructs
1 parent f00d2c6 commit feb2a7f

File tree

5 files changed

+3
-70
lines changed

5 files changed

+3
-70
lines changed

config/helm/aws-node-termination-handler/README.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,9 @@ Install AWS Node Termination Handler:
1616
To install the chart with the release name aws-node-termination-handler and default configuration:
1717

1818
```sh
19-
helm install --name aws-node-termination-handler \
20-
--namespace kube-system eks/aws-node-termination-handler
19+
helm upgrade --install aws-node-termination-handler \
20+
--namespace kube-system \
21+
eks/aws-node-termination-handler
2122
```
2223

2324
To install into an EKS cluster where the Node Termination Handler is already installed, you can run:

test/e2e/asg-lifecycle-sqs-test

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -40,22 +40,6 @@ set -x
4040
helm "${localstack_helm_args[@]}"
4141
set +x
4242

43-
aemm_helm_args=(
44-
upgrade
45-
--install
46-
"$CLUSTER_NAME-aemm"
47-
"$AEMM_DL_URL"
48-
--namespace default
49-
--set servicePort="$IMDS_PORT"
50-
--wait
51-
)
52-
[[ ${#common_helm_args[@]} -gt 0 ]] &&
53-
aemm_helm_args+=("${common_helm_args[@]}")
54-
55-
set -x
56-
retry 5 helm "${aemm_helm_args[@]}"
57-
set +x
58-
5943
sleep 10
6044

6145
RUN_INSTANCE_CMD="awslocal ec2 run-instances --private-ip-address ${WORKER_IP} --region ${AWS_REGION}"
@@ -79,7 +63,6 @@ anth_helm_args=(
7963
"$CLUSTER_NAME-acth"
8064
"$SCRIPTPATH/../../config/helm/aws-node-termination-handler/"
8165
--namespace kube-system
82-
--set instanceMetadataURL="${INSTANCE_METADATA_URL:-"http://$AEMM_URL:$IMDS_PORT"}"
8366
--set image.repository="$NODE_TERMINATION_HANDLER_DOCKER_REPO"
8467
--set image.tag="$NODE_TERMINATION_HANDLER_DOCKER_TAG"
8568
--set awsAccessKeyID=foo

test/e2e/ec2-state-change-sqs-test

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -40,22 +40,6 @@ set -x
4040
helm "${localstack_helm_args[@]}"
4141
set +x
4242

43-
aemm_helm_args=(
44-
upgrade
45-
--install
46-
"$CLUSTER_NAME-aemm"
47-
"$AEMM_DL_URL"
48-
--namespace default
49-
--set servicePort="$IMDS_PORT"
50-
--wait
51-
)
52-
[[ ${#common_helm_args[@]} -gt 0 ]] &&
53-
aemm_helm_args+=("${common_helm_args[@]}")
54-
55-
set -x
56-
retry 5 helm "${aemm_helm_args[@]}"
57-
set +x
58-
5943
sleep 10
6044

6145
RUN_INSTANCE_CMD="awslocal ec2 run-instances --private-ip-address ${WORKER_IP} --region ${AWS_REGION}"

test/e2e/rebalance-recommendation-sqs-test

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -40,23 +40,6 @@ set -x
4040
helm "${localstack_helm_args[@]}"
4141
set +x
4242

43-
aemm_helm_args=(
44-
upgrade
45-
--install
46-
"$CLUSTER_NAME-aemm"
47-
"$AEMM_DL_URL"
48-
--namespace default
49-
--set servicePort="$IMDS_PORT"
50-
--wait
51-
)
52-
53-
[[ ${#common_helm_args[@]} -gt 0 ]] &&
54-
aemm_helm_args+=("${common_helm_args[@]}")
55-
56-
set -x
57-
retry 5 helm "${aemm_helm_args[@]}"
58-
set +x
59-
6043
sleep 10
6144

6245
RUN_INSTANCE_CMD="awslocal ec2 run-instances --private-ip-address ${WORKER_IP} --region ${AWS_REGION}"
@@ -80,7 +63,6 @@ anth_helm_args=(
8063
"$CLUSTER_NAME-acth"
8164
"$SCRIPTPATH/../../config/helm/aws-node-termination-handler/"
8265
--namespace kube-system
83-
--set instanceMetadataURL="${INSTANCE_METADATA_URL:-"http://$AEMM_URL:$IMDS_PORT"}"
8466
--set image.repository="$NODE_TERMINATION_HANDLER_DOCKER_REPO"
8567
--set image.tag="$NODE_TERMINATION_HANDLER_DOCKER_TAG"
8668
--set awsAccessKeyID=foo

test/e2e/spot-interruption-sqs-test

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -40,22 +40,6 @@ set -x
4040
helm "${localstack_helm_args[@]}"
4141
set +x
4242

43-
aemm_helm_args=(
44-
upgrade
45-
--install
46-
"$CLUSTER_NAME-aemm"
47-
"$AEMM_DL_URL"
48-
--namespace default
49-
--set servicePort="$IMDS_PORT"
50-
--wait
51-
)
52-
[[ ${#common_helm_args[@]} -gt 0 ]] &&
53-
aemm_helm_args+=("${common_helm_args[@]}")
54-
55-
set -x
56-
retry 5 helm "${aemm_helm_args[@]}"
57-
set +x
58-
5943
sleep 10
6044

6145
RUN_INSTANCE_CMD="awslocal ec2 run-instances --private-ip-address ${WORKER_IP} --region ${AWS_REGION}"
@@ -79,7 +63,6 @@ anth_helm_args=(
7963
"$CLUSTER_NAME-acth"
8064
"$SCRIPTPATH/../../config/helm/aws-node-termination-handler/"
8165
--namespace kube-system
82-
--set instanceMetadataURL="${INSTANCE_METADATA_URL:-"http://$AEMM_URL:$IMDS_PORT"}"
8366
--set image.repository="$NODE_TERMINATION_HANDLER_DOCKER_REPO"
8467
--set image.tag="$NODE_TERMINATION_HANDLER_DOCKER_TAG"
8568
--set awsAccessKeyID=foo

0 commit comments

Comments
 (0)