coop-cloud/garage
2
0
Fork 0
Code Issues 6 Pull Requests 1 Projects 1 Releases Activity

monitoring #15

Merged
Brooke merged 13 commits from monitoring into main 2026-05-21 18:12:46 +00:00
Conversation 1 Commits 13 Files Changed 5
+68 -1
Brooke commented 2026-05-08 19:59:53 +00:00
Owner
Copy Link
Copy Source

#8

https://git.coopcloud.tech/coop-cloud/garage/issues/8
Brooke added 4 commits 2026-05-08 19:59:53 +00:00
add secrets 7e8571306a
add monitoring conditional and secrets to .env 7d1b4ddc77
setup conditional for garage.toml, set require metrics token to true aeb6675be4
update readme with basic configuration
continuous-integration/drone/pr Build is failing
Details
ff778eab62
Brooke added 1 commit 2026-05-08 20:10:23 +00:00
use compose_file env instead of a conditional in the .env file.
continuous-integration/drone/pr Build is failing
Details
46ba380e45
fauno requested changes 2026-05-08 20:34:25 +00:00
fauno left a comment
Owner
Copy Link
Copy Source

thanks!

thanks!
README.md Outdated
@@ -56,0 +64,4 @@
>SECRET_METRICS_TOKEN_SECRET_VERSION=v1 # length=64 charset=hex
>
> MONITORING="true"
> ```
fauno commented 2026-05-08 20:34:06 +00:00
Owner
Copy Link
Copy Source

also add instructions on which address and port to add to prometheus/alloy

also add instructions on which address and port to add to prometheus/alloy
Brooke commented 2026-05-08 20:56:00 +00:00
Author
Owner
Copy Link
Copy Source

Good point, instructions are definitely not finished, let me add the wip flag to the pr

Good point, instructions are definitely not finished, let me add the wip flag to the pr
Brooke commented 2026-05-09 00:54:37 +00:00
Author
Owner
Copy Link
Copy Source

The pr is a wip so I'll wait on review until the instructions are finalized.

The pr is a wip so I'll wait on review until the instructions are finalized.
Brooke marked this conversation as resolved
compose.monitoring.yml Outdated
@@ -0,0 +4,4 @@
services:
app:
environment:
- MONITORING_ENABLED=true
fauno commented 2026-05-08 20:29:53 +00:00
Owner
Copy Link
Copy Source

i recently discovered the env var is not needed here because the template is rendered locally

i recently discovered the env var is not needed here because the template is rendered locally
Brooke commented 2026-05-09 00:49:27 +00:00
Author
Owner
Copy Link
Copy Source

Where should this be placed to trigger the conditional in the config? I wasn't sure if it should go in the .env or in the compose file and chose the compose with the idea that it'd be simpler to manage in the config.

Where should this be placed to trigger the conditional in the config? I wasn't sure if it should go in the .env or in the compose file and chose the compose with the idea that it'd be simpler to manage in the config.
fauno commented 2026-05-11 16:45:30 +00:00
Owner
Copy Link
Copy Source

as i understand, if it's user-customizable it has to go on .env and only defined on compose files if it's needed during container runtime. if it's for templates it's ok for it to only be on .env.

as i understand, if it's user-customizable it has to go on `.env` and only defined on compose files if it's needed during container runtime. if it's for templates it's ok for it to only be on `.env`.
👍 1
fauno commented 2026-05-11 16:46:42 +00:00
Owner
Copy Link
Copy Source

everything else looks ok, thanks!

everything else looks ok, thanks!
Brooke marked this conversation as resolved
compose.monitoring.yml Outdated
@@ -0,0 +9,4 @@
labels:
- "traefik.tcp.routers.garage-monitoring.rule=HostSNI(`*`)"
- "traefik.tcp.routers.garage-monitoring.entrypoints=garage-monitoring"
- "traefik.tcp.services.garage-monitoring.loadbalancer.server.port=3903"
fauno commented 2026-05-08 20:33:17 +00:00
Owner
Copy Link
Copy Source

this means the port is proxied but not protected by https, right? wouldn't it be better to proxy it through the domain name? i'm not sure if you can do this on coopcloud, maybe the paths need to be listed

this means the port is proxied but not protected by https, right? wouldn't it be better to proxy it through the domain name? i'm not sure if you can do this on coopcloud, maybe the paths need to be listed
Brooke commented 2026-05-08 20:53:22 +00:00
Author
Owner
Copy Link
Copy Source

True, I didn't think of this. Going through a domain name would be easiest, because If I understand the monitoring endpoint correctly it's just served as a plaintext result.

True, I didn't think of this. Going through a domain name would be easiest, because If I understand the monitoring endpoint correctly it's just served as a plaintext result.
Brooke marked this conversation as resolved
compose.yml Outdated
@@ -52,0 +58,4 @@
external: true
metrics_token:
name: ${STACK_NAME}_metrics_token_${SECRET_METRICS_TOKEN_SECRET_VERSION}
external: true
fauno commented 2026-05-08 20:28:46 +00:00
Owner
Copy Link
Copy Source

i think you can move these secrets to compose.monitoring.yml so they're only needed when monitoring is enabled

i think you can move these secrets to compose.monitoring.yml so they're only needed when monitoring is enabled
Brooke commented 2026-05-08 20:53:39 +00:00
Author
Owner
Copy Link
Copy Source

Ah, thanks for catching this

Ah, thanks for catching this
Brooke marked this conversation as resolved
Brooke added 1 commit 2026-05-08 20:54:37 +00:00
move secrets to compose.monitoring.yml
continuous-integration/drone/pr Build is failing
Details
3741560b13
Brooke marked the pull request as work in progress 2026-05-08 20:56:24 +00:00
Brooke added 1 commit 2026-05-08 20:58:45 +00:00
no idea if this actually works, pushing so that I can test later
continuous-integration/drone/pr Build is failing
Details
baa4167a89
Brooke added 1 commit 2026-05-09 00:54:18 +00:00
set unique stack name
continuous-integration/drone/pr Build is failing
Details
88d46b767e
Brooke added 3 commits 2026-05-15 20:11:55 +00:00
move monitoring_enabled to .env file 97e96ed1d3
scrap monitoring domain as it was too much of a headache to configure, use /metrics endpoint which seems fairly standard(?) 04463dffe8
update readme for clarity
continuous-integration/drone/pr Build is failing
Details
37dbb6caa8
Brooke requested review from fauno 2026-05-15 20:11:57 +00:00
Brooke added 1 commit 2026-05-15 20:32:13 +00:00
Merge branch 'main' into monitoring
continuous-integration/drone/pr Build is failing
Details
dfb679b1c0
Brooke marked the pull request as ready for review 2026-05-21 18:04:40 +00:00
Brooke removed review request for fauno 2026-05-21 18:05:51 +00:00
Brooke added 1 commit 2026-05-21 18:12:16 +00:00
Merge branch 'main' into monitoring
continuous-integration/drone/pr Build is failing
Details
b484236fea
Brooke merged commit 796ee6c2d4 into main 2026-05-21 18:12:46 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
fauno
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Help wanted
3
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
3wordchant aadil (Aadil Ayub) abra-bot (Abra Bot) ammaratef45 amras (Sarma) Apfelwurm appletalk arjan basebuilder BornDeleuze Brooke carla cas (Cassowary) codegod100 coopcloud cyrnel decentral1se (d1) dede devydave fauno (fauno) flancian Frando iexos jade (Jade Ambrose) javielico (Javielico) jjsfunhouse jmakdah2 (Jackie Makdah) joe-irving (Joe Irving) kawaiipunk (KawaiiPunk) knoflook kolaente lambdabundesverband linnealovespie (April) marlon (marlon) mayel mirsal moosemower moritz nicksellen (Nick Sellen) notplants oxaliq (sorrel) p4u1 pau pharaohgraphy (Andrew 🐦‍🔥❤️‍🔥✴️) PhiNatalie renovate-bot (Comrade Renovate Bot) ripclap rix rscmbbng sef (sef) simon sixsmith (Sixsmith) stevensting tobias trav (Trav Fryer) val (val (he/him)) vaznasty virtualboys wolcen (Chris Thompson) wykwit xynosis yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/garage#15