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

WIP: monitoring #15

Draft
Brooke wants to merge 12 commits from monitoring into main
pull from: monitoring
merge into: coop-cloud:main
Conversation 1 Commits 12 Files Changed 5 +68 -1
Brooke commented 2026-05-08 19:59:53 +00:00
Owner
Copy Link

#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
Some checks failed
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.
Some checks failed
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

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

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

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

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

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

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

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

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

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

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

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

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
Some checks failed
continuous-integration/drone/pr Build is failing
Details
3741560b13
Brooke changed title from monitoring to WIP: monitoring 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
Some checks failed
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
Some checks failed
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
Some checks failed
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
Some checks failed
continuous-integration/drone/pr Build is failing
Details
dfb679b1c0
Some checks failed
continuous-integration/drone/pr Build is failing
Details
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin monitoring:monitoring
git checkout monitoring
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 Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
3wordchant aadil abra-bot ammaratef45 amras Apfelwurm appletalk arjan basebuilder BornDeleuze Brooke carla cas codegod100 coopcloud cyrnel decentral1se dede devydave fauno flancian Frando iexos jade javielico jjsfunhouse jmakdah2 joe-irving kawaiipunk knoflook kolaente lambdabundesverband linnealovespie marlon mayel mirsal moosemower moritz nicksellen notplants oxaliq p4u1 pau pharaohgraphy PhiNatalie renovate-bot ripclap rix rscmbbng sef simon sixsmith stevensting tobias trav val vaznasty virtualboys wolcen wykwit xynosis yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/garage#15
No description provided.
Delete Branch "monitoring"

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?