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 Helm Chart and make skaffold compatible with Helm #26

Merged
merged 15 commits into from
Apr 11, 2023

Conversation

kemurayama
Copy link
Collaborator

@kemurayama kemurayama commented Apr 4, 2023

This PR enables you to install Space Agon application with Helm.

Helm is the package manager tool for kubernetes that simplifies the deployment and management.
It is also used in Agones and Open Match.

In addition, I also added support for Skaffold with Helm.

Skaffold is a tool for building, pushing and deploying your application. It enables you to iterate quickly and deploy changes.

You can also test Helm Chart changes with skaffold.

Close #9

@kemurayama kemurayama requested a review from govargo April 4, 2023 08:29
templates/helm_template_values.yaml Outdated Show resolved Hide resolved
install/helm/space-agon/templates/dedicated.yaml Outdated Show resolved Hide resolved
install/helm/space-agon/templates/dedicated.yaml Outdated Show resolved Hide resolved
install/helm/space-agon/templates/dedicated.yaml Outdated Show resolved Hide resolved
install/helm/space-agon/templates/dedicated.yaml Outdated Show resolved Hide resolved
install/helm/space-agon/templates/mmf.yaml Outdated Show resolved Hide resolved
install/helm/space-agon/templates/mmf.yaml Outdated Show resolved Hide resolved
install/helm/space-agon/templates/mmf.yaml Outdated Show resolved Hide resolved
install/helm/space-agon/templates/mmf.yaml Outdated Show resolved Hide resolved
install/helm/space-agon/templates/mmf.yaml Outdated Show resolved Hide resolved
@kemurayama
Copy link
Collaborator Author

Hi @govargo, thank you for reviewing. I added helper templates.

@kemurayama
Copy link
Collaborator Author

Let me override parameters for GKE. I'll update later

Extracted Git tags from `build.sh`.

Add new command called `make install-local`
@kemurayama
Copy link
Collaborator Author

Hi @govargo, it's ready to review.

@kemurayama kemurayama requested a review from govargo April 7, 2023 04:16
@govargo
Copy link
Collaborator

govargo commented Apr 10, 2023

@kemurayama maybe, there are some reviews existing in frontend part.

{{ .Values.frontend.name }}{{ template "frontend.name" .}}

It looks there still be {{ .Values.frontend.name }}.

Others look good for me.

@kemurayama
Copy link
Collaborator Author

@govargo
Thanks. I updated.

Copy link
Collaborator

@govargo govargo left a comment

Choose a reason for hiding this comment

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

Thank you very much @kemurayama!
LGTM

@govargo govargo merged commit 43833c3 into googleforgames:main Apr 11, 2023
@kemurayama kemurayama deleted the helm_chart branch April 11, 2023 04:38
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.

Add Helm Chart
2 participants