Skip to content

Conversation

@lotharking
Copy link
Contributor

No description provided.

Signed-off-by: Andres Vallecilla <andresfelipe083195@hotmail.com>
Signed-off-by: Andres Vallecilla <andresfelipe083195@hotmail.com>
Signed-off-by: Andres Vallecilla <andresfelipe083195@hotmail.com>
Signed-off-by: Andres Vallecilla <andresfelipe083195@hotmail.com>
@lotharking lotharking requested a review from genaris June 3, 2025 14:20
…ic and updated docs

Signed-off-by: Gabriel Mata <gabrielmatau@gmail.com>
@gabrielmatau79
Copy link
Contributor

Hi @lotharking and @genaris 👋

I’ve improved the Helm chart by adding better handling for appCheckProxy:

  • Conditional rendering of sidecar, ports, and config entries

  • Centralized configuration in values.yaml

  • Updated README with full structure and usage

Let me know your thoughts

Signed-off-by: Gabriel Mata <gabrielmatau@gmail.com>
Signed-off-by: Gabriel Mata <gabrielmatau@gmail.com>
Signed-off-by: Gabriel Mata <gabrielmatau@gmail.com>
Copy link
Member

@genaris genaris left a comment

Choose a reason for hiding this comment

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

Some minor comments, especially about publishing Docker image

gabrielmatau79 and others added 3 commits June 25, 2025 16:20
Co-authored-by: Ariel Gentile <gentilester@gmail.com>
Co-authored-by: Ariel Gentile <gentilester@gmail.com>
Signed-off-by: Gabriel Mata <gabrielmatau@gmail.com>
@gabrielmatau79
Copy link
Contributor

@genaris @lotharking, All changes ready mates! please check and let me know ...

@genaris genaris merged commit b40c573 into main Jun 25, 2025
5 checks passed
@genaris genaris deleted the fix/add-helm-chart branch June 25, 2025 21:56
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.

4 participants