k8s-mediaserver-operator icon indicating copy to clipboard operation
k8s-mediaserver-operator copied to clipboard

Draft: Separate Helm Charts

Open reedjosh opened this issue 1 year ago • 7 comments

Breaks the helm charts out into individual charts with a superchart pattern.

generally by pulling this branch down it can be used in the same manner it always was via:

helm template k8s-mediaserver ./

When changes are made to the sub charts the changes can be pulled in via:

helm dep update

and in the future we can add a GitHub pages helm index for the charts and reference them via that.

reedjosh avatar Dec 30 '23 17:12 reedjosh

@kubealex I'm leaving this in draft state atm, but I'd love if you could do a brief review to make sure you like the direction and to get your overall feedback.

reedjosh avatar Dec 30 '23 21:12 reedjosh

@kubealex been on hold for the Jellyfin PR https://github.com/kubealex/k8s-mediaserver-operator/pull/72 for a month now.

Want me to just toss this all together?

Think with quick PR reviews I could get this PR done with Jellyfin within a week.

I'll start the work either way. 🙂 Probably have an update by tomorrow.

reedjosh avatar Jan 28 '24 15:01 reedjosh

image

Got a few minutes not buried under kiddos. Had to modify my current fork slightly, but back into business with Jellyfin.

Will try to clean this up further, and prove that it gens the same stuff. Hopefully then we can just merge this PR all at once to get Jellyfin and individual helm charts.

reedjosh avatar Jan 28 '24 23:01 reedjosh

Jellyfin has been merged @reedjosh, can you double check if this in any way creates any clash?

kubealex avatar Feb 08 '24 08:02 kubealex

I will begin cleanup on this branch. Hopefully I'll have a non-draft PR within the next week. : )

reedjosh avatar Feb 08 '24 15:02 reedjosh

@reedjosh Any idea how much cleanup you still want to do? Anywhere I can assist?

NigelVanHattum avatar Mar 15 '24 22:03 NigelVanHattum

@NigelVanHattum I'll try to get back to this sometime soon.

Unfortunately, it will need quite a bit still as the upstream changes will need incorporating and my PR was still not polished even before that.

I do still want this done though, so I'll try to get to it in the next week or so.

reedjosh avatar Mar 19 '24 03:03 reedjosh