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 k8s manifests and Helm Chart #1

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

add k8s manifests and Helm Chart #1

wants to merge 11 commits into from

Conversation

cmdoret
Copy link
Member

@cmdoret cmdoret commented Jun 19, 2023

This PR defines a helm chart to deploy the Gimie API to a kubernetes cluster.

Among other things, this allows to:

  • Deploy multiple replicas (3 by default) accessible at the same address with automatic load balancing.
  • Automatically restart container in case of problems.
  • Deploy rolling updates without downtime.

Usage instructions are available in gimie-api/README.md.

Caveats of the current setup:

  • The service uses the personal token provided by the person who deploys it.
  • It needs a (public) docker image, currently cmdoret/gimie-api-container.

@cmdoret cmdoret self-assigned this Jun 19, 2023
@cmdoret cmdoret requested review from caviri and marftn June 19, 2023 13:28
@caviri
Copy link
Contributor

caviri commented Jun 21, 2023

Soo Cool @cmdoret, I'm checking it. Do you think that docker image can be modified by one at ghcr.io?

@cmdoret
Copy link
Member Author

cmdoret commented Feb 21, 2024

Update: Took in updates from main +

  • use ~= (compatible release operator, PEP440) when specifying dependencies in Dockerfile
    • it locks only the major version to allow security patches and bugfixes, but not beaking backward-compatibility (assuming SemVer).
  • update github namespace throughout the repo (SDSC-ORD -> sdsc-ordes)
  • upload image to github container registry and refer to it in helm chart

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