fix(gitea) enable prometheus metrics on main service (#13477)
**Description** The servicemonitor deployed with Gitea chart is bugged with the current helm chart because metrics are not enabled on Gitea properly. ![image](https://github.com/truecharts/charts/assets/6644855/c803c7b5-8838-411e-a75d-9c92e60f9201) The helm chart common.yaml template is looking for `metrics.enabled: true` but it should be `metrics.main.enabled: true` as per the current values. ![image](https://github.com/truecharts/charts/assets/6644855/87459c66-dc74-4d01-9c8c-26096af9098d) This bug currently results in `app.ini` to have an empty value: ![image](https://github.com/truecharts/charts/assets/6644855/677575c6-8dc6-4ce6-aaca-53d3c42a221b) **⚙️ Type of change** - [ ] ⚙️ Feature/App addition - [x] 🪛 Bugfix - [ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] 🔃 Refactor of current code **🧪 How Has This Been Tested?** Deploy the helm chart with the change **📃 Notes:** <!-- Please enter any other relevant information here --> **✔️ Checklist:** - [x] ⚖️ My code follows the style guidelines of this project - [x] 👀 I have performed a self-review of my own code - [ ] #️⃣ I have commented my code, particularly in hard-to-understand areas - [ ] 📄 I have made corresponding changes to the documentation - [ ] ⚠️ My changes generate no new warnings - [ ] 🧪 I have added tests to this description that prove my fix is effective or that my feature works - [x] ⬆️ I increased versions for any altered app according to semantic versioning **➕ App addition** If this PR is an app addition please make sure you have done the following. - [ ] 🪞 I have opened a PR on [truecharts/containers](https://github.com/truecharts/containers) adding the container to TrueCharts mirror repo. - [ ] 🖼️ I have added an icon in the Chart's root directory called `icon.png` --------- Signed-off-by: Julien Nicolas de Verteuil <6644855+jndeverteuil@users.noreply.github.com>
This commit is contained in:
parent
24ff2f9035
commit
af62a7d763
|
@ -30,7 +30,7 @@ sources:
|
||||||
- https://gitea.com/gitea/helm-chart
|
- https://gitea.com/gitea/helm-chart
|
||||||
- https://github.com/go-gitea/gitea
|
- https://github.com/go-gitea/gitea
|
||||||
type: application
|
type: application
|
||||||
version: 14.0.2
|
version: 14.0.3
|
||||||
annotations:
|
annotations:
|
||||||
truecharts.org/catagories: |
|
truecharts.org/catagories: |
|
||||||
- GIT
|
- GIT
|
||||||
|
|
|
@ -44,7 +44,7 @@ secret:
|
||||||
{{- end }}
|
{{- end }}
|
||||||
|
|
||||||
[metrics]
|
[metrics]
|
||||||
ENABLED = {{ .Values.metrics.enabled }}
|
ENABLED = {{ .Values.metrics.main.enabled }}
|
||||||
{{- range $catindex, $catvalue := .Values.customConfig }}
|
{{- range $catindex, $catvalue := .Values.customConfig }}
|
||||||
{{- if eq $catvalue.name "metrics" }}
|
{{- if eq $catvalue.name "metrics" }}
|
||||||
{{- range $index, $value := $catvalue.keys }}
|
{{- range $index, $value := $catvalue.keys }}
|
||||||
|
|
|
@ -37,6 +37,9 @@ metrics:
|
||||||
main:
|
main:
|
||||||
enabled: true
|
enabled: true
|
||||||
type: "servicemonitor"
|
type: "servicemonitor"
|
||||||
|
endpoints:
|
||||||
|
- port: main
|
||||||
|
path: /metrics
|
||||||
ldap:
|
ldap:
|
||||||
enabled: false
|
enabled: false
|
||||||
# name:
|
# name:
|
||||||
|
|
Loading…
Reference in New Issue