Skip to content

Conversation

mariot8
Copy link
Contributor

@mariot8 mariot8 commented Aug 6, 2025

Describe the purpose of your pull request

  • Provide a clear and concise description of the changes.
    Add configcat-proxy helm chart and release pipeline

Related issues (only if applicable)

How to test? (only if applicable)

  • What part of the application was affected by the changes? What should be tested?
    helm install chart and see if the pods are running

Security (only if applicable)

  • Describe any potential risks that could affect the component in terms of security.

Requirement checklist (only if applicable)

  • I have covered the applied changes with automated tests.
  • I have executed the full automated test set against my changes.
  • I have validated my changes against all supported platform versions.
  • I have read and accepted the contribution agreement.

@mariot8 mariot8 requested a review from a team as a code owner August 6, 2025 10:28
@z4kn4fein
Copy link
Member

Hey @mariot8, thank you for the contribution! We'll review and test your PR soon

Copy link
Member

@z4kn4fein z4kn4fein left a comment

Choose a reason for hiding this comment

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

Hey @mariot8, I finally had the chance to try the chart out. Overall, it's working great, thank you! I have some minor suggestions, could you please check them? Thanks!

@mariot8
Copy link
Contributor Author

mariot8 commented Aug 14, 2025

Thank you for the review. These are small change requests, but I’ll be able to address them next week.

@mariot8
Copy link
Contributor Author

mariot8 commented Aug 29, 2025

I know this may not be ideal yet - cache backends other than Redis are still missing, etc.
But I hope the community or ConfigCat will turn it into a gem.

@z4kn4fein
Copy link
Member

z4kn4fein commented Sep 5, 2025

I think it's a good start, so I'm going to merge this. Later, I'll add support for the new proxy profiles and see how it goes. :)
Thank you again for your contribution!

@z4kn4fein z4kn4fein merged commit 1f3c691 into configcat:main Sep 5, 2025
2 of 3 checks 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.

2 participants