Skip to content

Commit

Permalink
appmesh-controller: v1.12.7
Browse files Browse the repository at this point in the history
  • Loading branch information
eks-bot committed Dec 13, 2023
1 parent 2ca1cb4 commit 6d20cd8
Show file tree
Hide file tree
Showing 6 changed files with 24 additions and 6 deletions.
4 changes: 2 additions & 2 deletions stable/appmesh-controller/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
apiVersion: v1
name: appmesh-controller
description: App Mesh controller Helm chart for Kubernetes
version: 1.12.5
appVersion: 1.12.5
version: 1.12.7
appVersion: 1.12.7
home: https://github.com/aws/eks-charts
icon: https://raw.githubusercontent.com/aws/eks-charts/master/docs/logo/aws.png
sources:
Expand Down
19 changes: 19 additions & 0 deletions stable/appmesh-controller/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,25 @@ helm upgrade -i appmesh-controller eks/appmesh-controller \

The [configuration](#configuration) section lists the parameters that can be configured during installation.

**Note**
If you want to start the controller in the EKS private cluster, enable the app mesh and service discovery VPC endpoints to the linked private subnet first. Also accountId is a required field now as `--set accountId=$AWS_ACCOUNT_ID`.
If you want to enable X-ray tracing in private cluster, enable the X-ray VPC endpoint. Also, ECR VPC endpoint [does not support public repository](https://docs.aws.amazon.com/AmazonECR/latest/userguide/vpc-endpoints.html). Controller uses `public.ecr.aws/xray/aws-xray-daemon:latest` by default, so you need to pull this image to local and [push it into your personal ECR repository](https://docs.aws.amazon.com/AmazonECR/latest/userguide/docker-push-ecr-image.html). Set it when deploying the controller like:
```
helm upgrade -i appmesh-controller eks/appmesh-controller \
--namespace appmesh-system \
--set region=$AWS_REGION \
--set serviceAccount.create=false \
--set serviceAccount.name=appmesh-controller \
--set accountId=$AWS_ACCOUNT_ID \
--set log.level=debug \
--set tracing.enabled=true \
--set tracing.provider=x-ray \
--set xray.image.repository={your-account-id}.dkr.ecr.{your-region}.amazonaws.com/{your-repository} \
--set xray.image.tag={your-xray-daemon-image-tag}
```
Verify if the X-ray daemon being injected successfully when binding application deployment with virtual node/gateway.
More troubleshooting please see: https://docs.aws.amazon.com/eks/latest/userguide/private-clusters.html

**Note**
Make sure that the Envoy proxies have the following IAM policies attached for the Envoy to authenticate with AWS App Mesh and fetch it's configuration
- https://raw.githubusercontent.com/aws/aws-app-mesh-controller-for-k8s/master/config/iam/envoy-iam-policy.json
Expand Down
2 changes: 1 addition & 1 deletion stable/appmesh-controller/ci/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,5 @@ accountId: 123456789
region: us-west-2
image:
repository: public.ecr.aws/appmesh/appmesh-controller
tag: v1.12.5
tag: v1.12.7
pullPolicy: IfNotPresent
1 change: 0 additions & 1 deletion stable/appmesh-controller/crds/crds.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3914,7 +3914,6 @@ spec:
type: object
required:
- action
- match
type: object
required:
- name
Expand Down
2 changes: 1 addition & 1 deletion stable/appmesh-controller/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ useAwsFIPSEndpoint: false

image:
repository: 840364872350.dkr.ecr.us-west-2.amazonaws.com/amazon/appmesh-controller
tag: v1.12.5
tag: v1.12.7
pullPolicy: IfNotPresent

sidecar:
Expand Down
2 changes: 1 addition & 1 deletion stable/appmesh-controller/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ useAwsFIPSEndpoint: false

image:
repository: 840364872350.dkr.ecr.us-west-2.amazonaws.com/amazon/appmesh-controller
tag: v1.12.5
tag: v1.12.7
pullPolicy: IfNotPresent

sidecar:
Expand Down

0 comments on commit 6d20cd8

Please sign in to comment.