Skip to content

Conversation

SajidDeo
Copy link
Contributor

@SajidDeo SajidDeo commented Oct 10, 2025

Main purpose of this document is to provide evidence that we "sign-off" on third party helm charts. Pre-requisite section covers this

Copy link
Contributor

@sylus sylus left a comment

Choose a reason for hiding this comment

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

This is a great baseline doc upon which to build from so LGTM!


Before adding any third-party helm charts to the Aurora platform do the following:

- Test the Helm chart in a local cluster and validate the rendered YAML manifests.
Copy link
Contributor

Choose a reason for hiding this comment

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

Rather than testing in a local cluster, rendering the template is substantially easier:

$ helm template <name> ./path/to/chart --values=<example values.yaml>

We should also do things like check for pods/containers that we don't expect, resources we wouldn't want, etc in the rendered output.


Bump the version number specified in the `Chart.yaml` file located under `aurora-platform`.

### 9. Deploy & test
Copy link
Contributor

Choose a reason for hiding this comment

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

We need to have a step here that is the PR review and approval process, to show that a second person takes a look before a change to the platform occurs

@SajidDeo SajidDeo merged commit 4638b6a into main Oct 14, 2025
1 check passed
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