This is an update to the metrics section of the values.yaml. This PR covers:
adds optionally setting NEXTCLOUD_SERVER via the metrics.server parameter, instead of allowing the chart to template it out from the k8s service. If not provided, we still use the service name
adds setting NEXTCLOUD_INFO_APPS via the metrics.info.apps parameter, which "Enables gathering of apps-related metrics". Defaults to false as it does in the actual container.
adds metrics.replicaCount to the README, as it was missing
formats the tables in the README (because my linter did it by default 😅 )
Benefits
This allows users to get more granular when playing with the metrics exporter container.
Possible drawbacks
none that I can see immediately. open to chatting as usual :)
Pull Request
Description of the change
This is an update to the
metricssection of the values.yaml. This PR covers:adds optionally setting
NEXTCLOUD_SERVERvia themetrics.serverparameter, instead of allowing the chart to template it out from the k8s service. If not provided, we still use the service nameadds setting
NEXTCLOUD_INFO_APPSvia themetrics.info.appsparameter, which "Enables gathering of apps-related metrics". Defaults to false as it does in the actual container.adds
metrics.replicaCountto the README, as it was missingformats the tables in the README (because my linter did it by default 😅 )
Benefits
This allows users to get more granular when playing with the metrics exporter container.
Possible drawbacks
none that I can see immediately. open to chatting as usual :)
Applicable issues
not sure we have any issues on this yet?
Additional information
See docs here: https://github.com/xperimental/nextcloud-exporter?tab=readme-ov-file#environment-variables
Checklist
Chart.yamlaccording to semver.