Add jmx-exporter monitoring functionality with servicemonitor

[azisi]

No description provided.

[valeriano-manassero]

Hi, Ty for the PR. Pls increment minor version in Chart.yaml and regenerate README with helm-docs to pass tests.

[azisi]

Oops sorry about that, this just a draft.
I wanted to open the PR in my repo first to clean it up before I open it here 😄

[valeriano-manassero]

No worries and take your time, will merge it when ready 😄

[azisi]

No worries and take your time, will merge it when ready 😄

done!
btw, i think the lint step in ci.yaml is broken. There is no v1.12.0 helm-docs version.

[valeriano-manassero]

i think the lint step in ci.yaml is broken. There is no v1.12.0 helm-docs version.

Good catch, would you mind to change this line with 1.11.0 too pls?

helm-charts/.github/helm-docs.sh

Line 4 in 549224c

HELM_DOCS_VERSION="1.12.0"

[valeriano-manassero]

I suggest to change version to 2.7.0 instead of 2.6.2.
This is because some new parameter is added so it's not just a patch.
wdyt?

[azisi]

Should we update trino version to the latest also, which is 406 from 380 that is now?

[valeriano-manassero]

Should we update trino version to the latest also, which is 406 from 380 that is now?

I have no idea if new version is compatible with the chart.
I suggest just to make this PR to pas tests (helm-docs needs to be run again). Then you can open another PR for a new version.