apache/celeborn

[BUG] Syntax error in helm charts file: prometheus-podmonitor.yaml

Closed this issue · 1 comments

What is the bug(with logs or screenshots)?

A clear and concise description of what the bug is.
charts/celeborn/templates/prometheus-podmonitor.yaml has two syntax errors that will be reported when installed directly using helm, as follows.
image
and the reason is just an extra space.
image
It's easy to fix, but I don't have a jira account until now.
(btw, I am curious why the bug is ignored in test while it is important and easy to find)

How to reproduce the bug?

Steps to reproduce the bug.
just use helm to deploy, you will get the error
/cc @who-need-to-know

/assign @who-can-solve-this-bug

Hi @every-breaking-wave , thanks for your feedback, I've approved your jira request and you should received the welcome mail.
We have to admit that currently the UT coverage is relatively low, especially for k8s environment, that's one of the important things we need to improve.