Closed darkowlzz closed 3 weeks ago
Thanks for submitting this! I've approved the workflow run, but do not have time to test this further, so I will add some other PR reviewers :)
Here's a quick fix for those (like me) that can't wait for the chart to be fixed. This file's values will override those in config/config.php.
nextcloud:
configs: # I've taken the standard trusted domains and added one that's necessary for the exporter to work
trusted-domains.config.php: |-
<?php
$CONFIG = [
'trusted_domains' =>
array (
0 => 'localhost',
1 => 'nextcloud.example.com',
2 => 'nextcloud.nextcloud.svc.cluster.local'
),
];
?>
Pull Request
Description of the change
When nextcloud-exporter is used by enabling metrics, it fails to fetch the serverinfo with 400 response code.
nextcloud-exporter log
nextcloud log
This is related to the switch from ingress to local service endpoint for nextcloud-exporter in https://github.com/nextcloud/helm/pull/388.
To fix this, in addition to
.Values.nextcloud.host
, the nextcloud service also needs to be included in the trusted domains. In config/config.phpwhere
nextcloud
is the service name.This solution is based on https://github.com/xperimental/nextcloud-exporter/issues/14 , an upstream issue for the same problem.
Benefits
The nextcloud-exporter will be able to fetch and export the metrics properly.
Possible drawbacks
None
Applicable issues
None
Additional information
Without this, the metrics exporter exported metrics about itself only:
Once the issue is fixed, it exports all the nextcloud app metrics:
Checklist
Chart.yaml
according to semver.