Skip to content
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

Add deployment helm chart. #45

Merged
merged 3 commits into from
Feb 11, 2025
Merged

Add deployment helm chart. #45

merged 3 commits into from
Feb 11, 2025

Conversation

pantierra
Copy link
Collaborator

@pantierra pantierra commented Feb 7, 2025

This PR adds a helm chart and basic instructions to deploy it on a (local) kubernetes cluster.

Related to: #33

@pantierra pantierra force-pushed the feature/helm-chart branch 7 times, most recently from 40dee09 to a2d5ca5 Compare February 10, 2025 14:25
@pantierra pantierra marked this pull request as ready for review February 10, 2025 14:27
Copy link
Collaborator

@emmanuelmathot emmanuelmathot left a comment

Choose a reason for hiding this comment

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

Some image name and tags changes and version check. The rest LGTM

@pantierra pantierra force-pushed the feature/helm-chart branch 2 times, most recently from 0665f28 to 5180a2a Compare February 11, 2025 09:00
@pantierra
Copy link
Collaborator Author

Many thanks for your reviews! I suggest to add the version check in a second step and move these charts in first.

@emmanuelmathot emmanuelmathot merged commit 8596a60 into main Feb 11, 2025
5 checks passed
@emmanuelmathot emmanuelmathot deleted the feature/helm-chart branch February 11, 2025 14:40
@pantierra
Copy link
Collaborator Author

Thanks again. Here is the promised follow-up draft PR for the version checks: #49

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