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

Publish helm chart using github pages #97

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hemstreet
Copy link

@hemstreet hemstreet commented Dec 14, 2024

What kind of change does this PR introduce?

This change enables you to install this chart from terraform. All it require is a new branch named gh-pages. Currently the index.yaml inside the root of this project has invalid urls and is confusing to setup.

This solves the manual process of creating releases and will produce the output like so. All on a new tag creation, it will use the chart version vs the actual tag number.

With this change, it would then let you install this chart like so:

resource "helm_release" "suspabase" {
  name       = "supabase"
  repository = "https://github.com/supabase-community/supabase-kubernetes"
  chart      = "supabase"
  version    = "0.1.3"

  namespace        = "supabase"
  create_namespace = true
  ...
}

You will still need to set values, etc.

Bug fix, feature, docs update, ...

What is the current behavior?

When running helm install via terraform, You can try to use the raw content url but the tar.gz urls are invalid

https://raw.githubusercontent.com/supabase-community/supabase-kubernetes/refs/heads/main/index.yaml

Please link any relevant issues here.
#76

Fixes this draft from April https://github.com/supabase-community/supabase-kubernetes/pull/55/files

What is the new behavior?

uses github pages to host the helm charts

Feel free to include screenshots if it includes visual changes.

Additional context

Add any other context or screenshots.

Adding charts to gh-pages
@hemstreet hemstreet changed the title Create publish.yaml Publish helm chart using github pages Dec 14, 2024
@hemstreet
Copy link
Author

@mdluo i there anything pending on this? I saw you approved of it last week. I don't have merge access, Just trying to help push this over the finish line / be diligent about anything that is pending.

@jonathanmccracken
Copy link

+1 here - having this helm chart available on github pages would help a lot

@hemstreet
Copy link
Author

hemstreet commented Feb 3, 2025

@jonathanmccracken unfortunately this project might be abandoned. Not sure what else I can do to get this merged other than wait for someone to approve.

@mdluo any chance this can get merged? Or is there someone else that may be able to / review this?

@arpagon maybe? I see you approved / merged the last PR

@arpagon
Copy link
Member

arpagon commented Feb 3, 2025

Just let me take a look this morning, and I'll merge it.

@mdluo
Copy link

mdluo commented Feb 3, 2025

@jonathanmccracken unfortunately this project might be abandoned. Not sure what else I can do to get this merged other than wait for someone to approve.

@mdluo any chance this can get merged? Or is there someone else that may be able to / review this?

@arpagon maybe? I see you approved / merged the last PR

Unfortunately I'm not a member of this project and unable to merge it, but for some reason was able to review it.

But I forked this repo too: https://github.com/voltade/supabase-kubernetes, and did a few minor improvements:

Chart metadata file: https://voltade.github.io/supabase-kubernetes/index.yaml, feel free to try it out

@arpagon
Copy link
Member

arpagon commented Feb 7, 2025

Great work, @mdluo!
If you create the PR for this repo https://github.com/voltade/supabase-kubernetes,

I'd be happy to merge it. 🚀

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.

5 participants