minio / operator

Simple Kubernetes Operator for MinIO clusters :computer:
https://min.io/docs/minio/kubernetes/upstream/index.html
GNU Affero General Public License v3.0
1.19k stars 450 forks source link

Add port names to container ports #2326

Open filipdutescu opened 6 days ago

filipdutescu commented 6 days ago

Is your feature request related to a problem? Please describe. The Prometheus PodMetricsEndpoint spec has deprecated the targetPort field, which would be able to take an Integer. Since the minio/tenant chart does not set names to port 9000 (which is used for metrics), I cannot move away from the deprecated field.

Unfortunately, minio appears to be moving away from that field for good, so the easiest option might be my proposal below.

Describe the solution you'd like Define a name for the port in question. It doesn't need to be customizable via Helm, it can be static, but I would really appreciate the option to reference it by name.

Describe alternatives you've considered Provide a PodMonitor in the Helm chart, which takes care of all of the setup, but that might be too much in terms of maintenance burden. Leaving it here as an option though.

Additional context