[enterprise-logs] Mark chart as deprecated

[jkroepke]

No description provided.

[jkroepke]

The chart never works, the minio version seems never exists,

[jkroepke]

@zalegrala I may need your assistance here, because the chart is broken anyways (due invalid dependencies), I cant merge this.

[zalegrala]

Hmm, I can't force-merge this either with the failing CI. So to deprecate it, do we need to fix the dependencies here?

[Sheikh-Abubaker]

@jkroepke what if we remove the minio dependency completely or set the version to something that actually exists ? anyways the chart is being deprecated so I don't really think it matters ?

[jkroepke]

Maybe @dehaansa can help here.

(ref #3982)

[dehaansa]

Maybe @dehaansa can help here.

(ref #3982)

I'm not confident enough to merge this myself (though it makes sense to me!), but I'll try to get some eyes on it.

Oops, doing this on mobile I missed some context, I can't merge with the failing tests either.

[Sheikh-Abubaker]

@jkroepke the tempo-distributed chart also includes minio (the version that actually exists) as one of its dependency, how about if could you try with the same version here in order to pass CI check:

helm-charts/charts/tempo-distributed/Chart.yaml

Lines 27 to 32 in 0db5800

	dependencies:
	- name: minio
	alias: minio
	version: 4.0.12
	repository: https://charts.min.io/
	condition: minio.enabled