community icon indicating copy to clipboard operation
community copied to clipboard

HIP to include repository URL and tarball digest in release object under chart metadata

Open lucasreed opened this issue 3 years ago • 5 comments

Signed-off-by: Luke Reed [email protected]

Opening this HIP after conversation with @mattfarina in this PR comment section: https://github.com/helm/helm/pull/10369

This would help solve this longstanding issue: https://github.com/helm/helm/issues/4256

lucasreed avatar Nov 24 '21 14:11 lucasreed

A potential scope increase to also include the tar sha256sum in the release metadata might be nice as well. If installed with a local tarball, this could still be populated where repoURL would not get populated in that case. I can extend the proposal with this.

This came out of a conversation in this PR: https://github.com/FairwindsOps/nova/pull/61

I'm also planning on attending the helm dev meeting today in case there is time to talk about this HIP.

lucasreed avatar Dec 02 '21 15:12 lucasreed

@technosophos can you look at the latest changes?

mattfarina avatar Jan 06 '22 16:01 mattfarina

will this MR be merged? look forward for this function:)

jiangfoxi avatar Mar 02 '22 04:03 jiangfoxi

Also very interested to see this move forward. Is there anything we can do to assist?

sudermanjr avatar May 02 '22 20:05 sudermanjr

I am also looking forward for having this functionality. What needs to be done next to push this forward?

kekscode avatar May 30 '22 15:05 kekscode

Closing this in lieu of @sudermanjr opening a newer one here: https://github.com/helm/community/pull/269 Thank you, and sorry I haven't had the time to pursue this further!

lucasreed avatar Aug 22 '22 13:08 lucasreed