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

feat: add helm chart #450

Conversation

stevefan1999-personal
Copy link

Describe your changes

Related Issues / Projects

Checklist

  • Get official Docker repo and image
  • Find a way to integrate go-migrate because it fits well with init container (which is done by Gitea Helm chart)
  • Fix all the shit template
  • Make sure that the chart values README are generated (that means documenting the values)
  • Deploy a test pipeline using vcluster
  • Get a Github page to publish the charts (that means adding the chart-releaser action)

@NiceAesth NiceAesth changed the title add helm chart feat: add helm chart Jul 19, 2023
@NiceAesth NiceAesth added the enhancement New feature or request label Jul 19, 2023
@cmyui
Copy link
Member

cmyui commented Sep 24, 2023

Hey, this is neat the work you've done here but I don't think this is particularly useful for most users of b.py - at least in it's current state - I think it would make most sense as a repo you build as a fork of this project.

I think there are quite a few roadblocks before b.py will be worthy of throwing on k8s - notably making it fully stateless

I'd be open to making an official helm chart in the future when the time is appropriate & the project is in the right state for it - but not sure if it should even live in this repo (it's already quite bloated imo)

Ty for the PR!

@cmyui cmyui closed this Sep 24, 2023
@cmyui
Copy link
Member

cmyui commented Sep 24, 2023

I think there are quite a few roadblocks before b.py will be worthy of throwing on k8s - notably making it fully stateless

made a ticket for this for @tsunyoku and i

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants