Skip to content

Conversation

@sabban
Copy link
Contributor

@sabban sabban commented Dec 1, 2025

add values parameter reference

@aws-amplify-eu-west-1
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-939.d1to60jd2gb6y6.amplifyapp.com

@sabban
Copy link
Contributor Author

sabban commented Dec 1, 2025

Add the helm's values.yaml reference in the documentation.

I'll add the right branch (main) right after approval.


| Name | Description | Value |
| ------------------- | ------------------------------------------------------------- | -------- |
| `container_runtime` | [string] for raw logs format: json or cri (docker|containerd) | `docker` |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to escape the | as it breaks rendering

@buixor
Copy link
Contributor

buixor commented Jan 2, 2026

thx, I think this is a good step towards better doc experience. I guess next steps are:

  • referencing this in the getting started
  • maybe find a way to highlight the important options and what they allow to achieve

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants