fix: cleanup and linting of metric resources

[wrenix]

Pull Request

Additional information

Split of #478

1. Move the metrics-deployment related resources into dedicated folder templates/metrics.
   make a fee cleanups and lints for easier readable:

   • indent of list
   • use nindent instatt of indent
   • use with instatt of if
   • merge if with using and (and could handle more then two parameters)

2. Add namespaceSelector on servicemonitor

3. add ci/*-values.yaml for chart-testing

Checklist

• DCO has been signed off on the commit.
• Chart version bumped in Chart.yaml according to semver.

[jessebot]

Looks OK, but please add a description of what you're doing and why to the PR description.

[wrenix]

Done

[jessebot]

Awesome, thank you! Have you tested installing this with metrics enabled?

[wrenix]

Nope, but if you like i will (and then i will also reimplement #349 ).

[jessebot]

Yes, please test parts of the code you are modifying. That's probably a good habit generally, but at least until we figure out the helm unit tests (I still haven't had the time to sit down and play with it after the most recent PR was added) or add more CI jobs to do more than the default install.

[wrenix]

• I test it
• add namespaceSelector Fix namespace selector #349
• i add value files for chart-testing:

New chart version: 4.5.3
Chart version ok.
Validating /home/wrenix/src/github.com/nextcloud/helm/charts/nextcloud/Chart.yaml...
Validation success! 👍
Validating maintainers...

Linting chart with values file "charts/nextcloud/ci/ct-all-enabled-values.yaml"...

==> Linting charts/nextcloud

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

Linting chart with values file "charts/nextcloud/ci/ct-specials-values.yaml"...

==> Linting charts/nextcloud

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

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

[wrenix]

rebased

[jessebot]

Thanks! Approved the workflow.

add namespaceSelector #349

Can we put a little "Coauthored by" on a commit here to credit Richardds for the other PR so that when we merge this one, we can close #349 at the same time and say thank you?

[jessebot]

Please bump the chart version a minor version as we will be adding the functionality of the namespace selector here :)

[wrenix]

done:

• rebase
• new version
• co-authored of @Richardds

[wrenix]

rebased and version dump again

[wrenix]

rebased and version dump again.

ping @jessebot

---

that is my fourth version dump, without any new review.

So, please say something or i will close this PR ....
(PS: of course i could remove the ci/ct-*-values.yaml commit if we want discuss it later or still here #480 (comment))

[wrenix]

another conflict ... - so i stop supporting this PR

[provokateurin]

@wrenix we are happy to review improvements, I just think that these huge PRs make it hard to review and currently nobody seems to really have time for it. If could split your changes into smaller PRs then they can be reviewed sooner.

[wrenix]

that is already a split into parts ... and you did not asked in this pr again ....

you could take the rebased version/commit from here:
https://github.com/wrenix/nextcloud-helm/commits/main/

current it is:

• wrenix@f23c107
• wrenix@8868682 (this is from Fix namespace selector #349