Add Maintainers to Chart YAML to Pass CT Linter

[pierluigilenoci]

The current Komoplane Helm chart fails the ct linter due to the absence of the maintainers field in the Chart.yaml file.
The ct linter requires this field to ensure that charts have identifiable maintainers for support and accountability purposes.

Including the maintainers field in the Chart.yaml will ensure that the Helm chart passes the ct linter, adhering to Helm chart best practices and improving quality control.

For more information on the maintainers field, please refer to the Helm Chart Best Practices.

[undera]

I have read "Helm Chart Best Practices" section linked above, and there's no mention of maintainer fields there. Can it be that it's not part of Best Practices?

[pierluigilenoci]

@undera more refs:

• https://github.com/helm/chart-testing
• https://helm.sh/docs/topics/charts/#the-chartyaml-file
• https://github.com/helm/chart-testing/blob/main/doc/ct_lint.md

To get the "maintainers' error", run the command ct lint . in the Chart's root folder.

[undera]

I have added chart-testing lint step to our CI, it does not complain on maintainers: https://github.com/komodorio/komoplane/actions/runs/12213639829/job/34073479712

[pierluigilenoci]

@undera It's not just complaining because the charts/ct.yaml configuration is not correct and doesn't lint the chart.
If you look at the logs, it is clear.
There should have been something like:

------------------------------------------------------------------------------------------------------------------------
 Charts to be processed:
------------------------------------------------------------------------------------------------------------------------
 komoplane => (version: "0.1.5", path: "charts/komoplane")
------------------------------------------------------------------------------------------------------------------------

Linting chart "komoplane => (version: \"0.1.5\", path: \"charts/komoplane\")"
Validating /home/runner/work/komoplane/komoplane/charts/komoplane/Chart.yaml...
Validation success! 👍
Validating maintainers...
==> Linting charts/komoplane
[INFO] Chart.yaml: icon is recommended

1 chart(s) linted, 0 chart(s) failed

------------------------------------------------------------------------------------------------------------------------
 ✔︎ komoplane => (version: "0.1.5", path: "charts/komoplane")
------------------------------------------------------------------------------------------------------------------------
All charts linted successfully

[pierluigilenoci]

To quickly test run ct lint --all . inside the chart folder.

[undera]

I have added the maintainers field to the chart.yaml, but it still complains on some 404 error. I don't have more time to spend on this, sorry.