-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Review eks documentation #137
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello,
tu ne rajoutes pas un mot sur l'annotation pour la sticky session en NGINX ? Avec, peut-etre 2 ingress differents pour DC et DS ?
platform/eks/README.md
Outdated
|
||
``` | ||
|
||
Reference: https://docs.aws.amazon.com/eks/latest/userguide/create-cluster.html | ||
> **Note** | ||
> Use the latest kubernetes version available in AWS. Refer to [Amazon EKS Kubernetes versions](https://docs.aws.amazon.com/eks/latest/userguide/kubernetes-versions.html). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I prefer to put something like that.
The tutorial has been tested with the Kubernetes version 1.26. Check the supported kubernetes version in the system requirement page.
platform/eks/README-NGINX.md
Outdated
|
||
- Get the [eks-rds-nginx-values.yaml](./eks-rds-nginx-values.yaml) file and replace the following keys: | ||
- `<RDS_DB_ENDPOINT>`: your database server endpoint (of the form: `<INSTANCE_NAME>.xxxxxxxx.<REGION>.rds.amazonaws.com`) | ||
- `<RDS_DATABASE_NAME>`: the initial database name defined when creating the RDS database | ||
|
||
```bash | ||
helm install mycompany ibmcharts/ibm-odm-prod --version 22.2.0 -f eks-rds-nginx-values.yaml | ||
helm install mycompany ibm-helm/ibm-odm-prod --version 22.2.0 -f eks-rds-nginx-values.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The helm chart version should be updated to 23.1.0.
platform/eks/README.md
Outdated
$ helm search repo ibm-odm-prod | ||
NAME CHART VERSION APP VERSION DESCRIPTION | ||
ibmcharts/ibm-odm-prod 22.2.0 8.11.1.0 IBM Operational Decision Manager | ||
ibm-helm/ibm-odm-prod 22.2.0 8.11.1.0 IBM Operational Decision Manager |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be updated to 23.1.0 8.12.0.0
platform/eks/README.md
Outdated
@@ -223,19 +228,21 @@ To install ODM with the AWS RDS PostgreSQL database created in [step 3](#3-creat | |||
- `<RDS_DATABASE_NAME>` is the initial database name defined when creating the RDS database | |||
|
|||
```bash | |||
helm install mycompany ibmcharts/ibm-odm-prod --version 22.2.0 -f eks-rds-values.yaml | |||
helm install mycompany ibm-helm/ibm-odm-prod --version 22.2.0 -f eks-rds-values.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TO be updated to 23.1.0
platform/eks/README.md
Outdated
> | ||
> - Get the [eks-values.yaml](./eks-values.yaml) file and replace the following key: | ||
> - `<AWS-AccountId>` is your AWS Account Id | ||
> | ||
>```bash | ||
>helm install mycompany ibmcharts/ibm-odm-prod --version 22.2.0 -f eks-values.yaml | ||
>helm install mycompany ibm-helm/ibm-odm-prod --version 22.2.0 -f eks-values.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to be updated to 23.1.0
No description provided.