coop-cloud/monitoring-ng
2
0
Fork 2
Code Issues 7 Pull Requests 4 Packages Projects Releases Wiki Activity

chore: update image tags #10

Merged
ammaratef45 merged 4 commits from upgrade-recipe into main 2026-04-09 04:36:05 +00:00
Conversation 16 Commits 4 Files Changed 9 +17 -18
ammaratef45 commented 2026-03-03 05:34:14 +00:00
Owner
Copy Link

Needs some testding

Needs some testding
❤️ 1
ammaratef45 added 1 commit 2026-03-03 05:34:14 +00:00
chore: update image tags 423b9bcfb2
p4u1 commented 2026-03-13 15:51:23 +00:00
Owner
Copy Link

did you test the upgrade or did you deploy a new app? I'm a little scared if this breaks any existing deployment

did you test the upgrade or did you deploy a new app? I'm a little scared if this breaks any existing deployment
ammaratef45 commented 2026-03-13 19:36:04 +00:00
Author
Owner
Copy Link

@p4u1 I'm testing this and some things need to change (for example the loki config template will be updated because config values have changed)

But without upgrading, the recipe as is works only for older docker version

Example error:

�2026-03-03T05:27:21.190753654Z level=error ts=2026-03-03T05:27:21.097480418Z caller=refresh.go:99 component=docker_discovery discovery=docker config=docker/unix:///var/run/docker.sock:80 msg="Unable to refresh target groups" err="error while listing containers: Error response from daemon: client version 1.42 is too old. Minimum supported API version is 1.44, please upgrade your client to a newer version"
@p4u1 I'm testing this and some things need to change (for example the loki config template will be updated because config values have changed) But without upgrading, the recipe as is works only for older docker version Example error: ``` �2026-03-03T05:27:21.190753654Z level=error ts=2026-03-03T05:27:21.097480418Z caller=refresh.go:99 component=docker_discovery discovery=docker config=docker/unix:///var/run/docker.sock:80 msg="Unable to refresh target groups" err="error while listing containers: Error response from daemon: client version 1.42 is too old. Minimum supported API version is 1.44, please upgrade your client to a newer version" ```
p4u1 commented 2026-03-13 19:40:29 +00:00
Owner
Copy Link

Ah damn, yeah the upgrade is long overdue. If you need help testing, I have some time next friday

Ah damn, yeah the upgrade is long overdue. If you need help testing, I have some time next friday
ammaratef45 commented 2026-03-13 19:47:09 +00:00
Author
Owner
Copy Link

Yeah testing the upgrade would be great as I'm focused on testing making the recipe deployment work for new apps (I have no current stacks with this recipe deployed)

Yeah testing the upgrade would be great as I'm focused on testing making the recipe deployment work for new apps (I have no current stacks with this recipe deployed)
ammaratef45 commented 2026-03-13 19:48:32 +00:00
Author
Owner
Copy Link

I can spin a new vm and test the deploying with an older docker so I can do the upgrade as well, given how big this recipe is it might be a heavy lift so I'll take all the help I can get haha

I can spin a new vm and test the deploying with an older docker so I can do the upgrade as well, given how big this recipe is it might be a heavy lift so I'll take all the help I can get haha
p4u1 commented 2026-03-20 09:17:42 +00:00
Owner
Copy Link

I just check and we did not deploy loki yet. I wonder if anyone is actually using it if it is not even working with a recent docker version. Will test the other updates later

I just check and we did not deploy loki yet. I wonder if anyone is actually using it if it is not even working with a recent docker version. Will test the other updates later
p4u1 reviewed 2026-03-20 10:46:21 +00:00
compose.prometheus.yml
@ -3,3 +3,3 @@
services:
prometheus:
image: prom/prometheus:v2.55.1
image: prom/prometheus:v3.10.0
p4u1 commented 2026-03-20 10:46:21 +00:00
Owner
Copy Link

Breaking changes look okay to me: https://prometheus.io/docs/prometheus/latest/migration/

Breaking changes look okay to me: https://prometheus.io/docs/prometheus/latest/migration/
ammaratef45 marked this conversation as resolved
p4u1 reviewed 2026-03-20 10:47:34 +00:00
compose.matrix-alertmanager-receiver.yml
@ -3,3 +3,3 @@
services:
matrix-alertmanager-receiver:
image: metio/matrix-alertmanager-receiver:2025.2.9
image: metio/matrix-alertmanager-receiver:2026.2.25
p4u1 commented 2026-03-20 10:47:34 +00:00
Owner
Copy Link

Looks good to me

Looks good to me
ammaratef45 marked this conversation as resolved
p4u1 reviewed 2026-03-20 10:47:57 +00:00
compose.pushgateway.yml
@ -3,3 +3,3 @@
services:
pushgateway:
image: prom/pushgateway:v1.10.0
image: prom/pushgateway:v1.11.2
p4u1 commented 2026-03-20 10:47:57 +00:00
Owner
Copy Link

looks good

looks good
ammaratef45 marked this conversation as resolved
p4u1 reviewed 2026-03-20 10:48:07 +00:00
compose.yml
@ -4,3 +4,3 @@
services:
app:
image: prom/node-exporter:v1.8.1
image: prom/node-exporter:v1.10.2
p4u1 commented 2026-03-20 10:48:07 +00:00
Owner
Copy Link

looks good

looks good
ammaratef45 marked this conversation as resolved
p4u1 reviewed 2026-03-20 10:50:41 +00:00
compose.yml
@ -44,3 +44,3 @@
cadvisor:
image: gcr.io/cadvisor/cadvisor:v0.49.2
image: gcr.io/cadvisor/cadvisor:v0.55.1
p4u1 commented 2026-03-20 10:50:41 +00:00
Owner
Copy Link

This was the only upgrade not I could find in the release notes, so looks good to me:

v0.55: docker versions older than 25.0 are no longer supported

This was the only upgrade not I could find in the release notes, so looks good to me: v0.55: docker versions older than 25.0 are no longer supported
ammaratef45 marked this conversation as resolved
p4u1 reviewed 2026-03-20 11:59:16 +00:00
compose.grafana.yml
@ -3,3 +3,3 @@
services:
grafana:
image: grafana/grafana:10.4.14
image: grafana/grafana:12.4.0
p4u1 commented 2026-03-20 11:59:16 +00:00
Owner
Copy Link

Needed to migrate how we import alerts: #16

Needed to migrate how we import alerts: https://git.coopcloud.tech/coop-cloud/monitoring-ng/pulls/16
ammaratef45 marked this conversation as resolved
ammaratef45 added 1 commit 2026-03-27 21:10:38 +00:00
update loki config file cdc0133c31
dannygroenewegen commented 2026-03-29 12:31:11 +00:00
Owner
Copy Link

Deployed the previous recipe two weeks ago, then updated cadvisor after a day to support reading labels on my Docker version, the image tags in this PR fixed that. Upgrade worked, but this was without Loki and only with one day of data.

This week I got Loki running. Wasn't really sure yet of any side effects of what I had to change in the config, I now see it's nearly identical to the last commit of @ammaratef45 . Except I kept schema v11 initially. I've now upgraded to cdc0133c31, so upgrading to scheme v13 in Loki also works without losing log history.

Deployed the previous recipe two weeks ago, then updated cadvisor after a day to support reading labels on my Docker version, the image tags in this PR fixed that. Upgrade worked, but this was without Loki and only with one day of data. This week I got Loki running. Wasn't really sure yet of any side effects of what I had to change in the config, I now see it's nearly identical to the last commit of @ammaratef45 . Except I kept schema v11 initially. I've now upgraded to cdc0133c31, so upgrading to scheme v13 in Loki also works without losing log history.
❤️ 1
dannygroenewegen reviewed 2026-03-29 12:37:58 +00:00
loki.yml.tmpl Outdated
@ -108,3 +104,4 @@
retention_enabled: true
retention_delete_delay: 2h
retention_delete_worker_count: 150
delete_request_store: filesystem
dannygroenewegen commented 2026-03-29 12:33:53 +00:00
Owner
Copy Link

Changing this line to

{{ if eq (env "LOKI_STORAGE_FILESYSTEM") "1" }}
  delete_request_store: filesystem
{{ end }}
{{ if eq (env "LOKI_STORAGE_S3") "1" }}
  delete_request_store: aws
{{ end }}

to support the optional S3 storage in the config.

Changing this line to ``` {{ if eq (env "LOKI_STORAGE_FILESYSTEM") "1" }} delete_request_store: filesystem {{ end }} {{ if eq (env "LOKI_STORAGE_S3") "1" }} delete_request_store: aws {{ end }} ``` to support the optional S3 storage in the config.
❤️ 1
ammaratef45 added 1 commit 2026-04-07 02:03:47 +00:00
delete_request_store based on env variable 23b13cb855
ammaratef45 added 1 commit 2026-04-07 02:14:26 +00:00
Merge branch 'main' into upgrade-recipe 0ca5bd55b3
ammaratef45 changed title from WIP: chore: update image tags to chore: update image tags 2026-04-07 03:58:16 +00:00
ammaratef45 requested review from dannygroenewegen 2026-04-07 03:58:37 +00:00
ammaratef45 requested review from p4u1 2026-04-07 03:58:52 +00:00
ammaratef45 commented 2026-04-07 03:59:13 +00:00
Author
Owner
Copy Link

Okay I'm happy with this so far, @dannygroenewegen @p4u1 wdyt?

Okay I'm happy with this so far, @dannygroenewegen @p4u1 wdyt?
👍 1
dannygroenewegen approved these changes 2026-04-07 10:06:14 +00:00
ammaratef45 merged commit 9cb997b25a into main 2026-04-09 04:36:04 +00:00
ammaratef45 deleted branch upgrade-recipe 2026-04-09 04:36:05 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
p4u1
dannygroenewegen
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
3wordchant aadil abra-bot ammaratef45 Apfelwurm appletalk arjan basebuilder Brooke carla cas codegod100 coopcloud cyrnel dannygroenewegen decentral1se devydave fauno flancian Frando iexos jade javielico jjsfunhouse jmakdah2 joe-irving kawaiipunk knoflook kolaente lambdabundesverband linnealovespie marlon mayel mirsal moosemower moritz nicksellen notplants p4u1 pau pharaohgraphy PhiNatalie renovate-bot ripclap rix rscmbbng sef simon sixsmith stevensting tobias trav val vaznasty virtualboys wolcen wykwit xynosis yksflip
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/monitoring-ng#10
No description provided.
Delete Branch "upgrade-recipe"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?