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

[maykinmedia/open-klant#299] Add nginx Open Klant charts #182

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

danielmursa-dev
Copy link
Contributor

@danielmursa-dev danielmursa-dev commented Feb 5, 2025

@danielmursa-dev danielmursa-dev force-pushed the feature/add-nginx-openklant branch from 5454f5f to cb41ead Compare February 6, 2025 09:51
@danielmursa-dev danielmursa-dev force-pushed the feature/add-nginx-openklant branch from 04da077 to d8cbe1b Compare February 6, 2025 13:20
@danielmursa-dev
Copy link
Contributor Author

Deployed with @Tiho0135 on the test cluster and it worked. I don't know what other tests we have to do to see if it works properly?

@danielmursa-dev danielmursa-dev changed the title Add nginx to Open Klant [maykinmedia/open-klant#299] Add nginx Open Klant charts Feb 6, 2025
Copy link
Collaborator

@SilviaAmAm SilviaAmAm left a comment

Choose a reason for hiding this comment

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

The CHANGELOG.md needs updating and the version of the chart needs to be bumped (

version: 1.5.0-beta.0
).

(Then the readme will also need to be regenerated with helm-docs!)

For the rest seems good

charts/openklant/templates/hpa.yaml Show resolved Hide resolved
@alextreme alextreme removed their request for review February 6, 2025 16:42
@danielmursa-dev danielmursa-dev force-pushed the feature/add-nginx-openklant branch 3 times, most recently from b173c11 to d53c4db Compare February 12, 2025 09:11
@danielmursa-dev
Copy link
Contributor Author

@SilviaAmAm updated CHANGELOG.md and generated README.md

@danielmursa-dev danielmursa-dev force-pushed the feature/add-nginx-openklant branch from d53c4db to b1e379e Compare February 12, 2025 09:19
@danielmursa-dev danielmursa-dev merged commit 2fcabd9 into main Feb 12, 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
3 participants