coop-cloud/monitoring-ng
3
0
Fork 3
Code Issues 5 Pull Requests 10 Packages Projects Releases Wiki Activity

feat: access log coop-cloud/traefik#126 #27

Open
fauno wants to merge 2 commits from access-log into main
pull from: access-log
merge into: :main
Conversation 8 Commits 2 Files Changed 5
+11
fauno commented 2026-07-28 17:26:41 +00:00
Owner
Copy Link
Copy Source
No description provided.
fauno requested review from 3wordchant 2026-07-28 17:26:41 +00:00
fauno requested review from decentral1se 2026-07-28 17:26:41 +00:00
decentral1se left a comment
Owner
Copy Link
Copy Source

LGTM but I don't use this recipe!

LGTM but I don't use this recipe!
p4u1 commented 2026-09-02 16:14:24 +00:00
Owner
Copy Link
Copy Source

Traefik PR was merged and a new version released

Traefik PR was merged and a new version released
ammaratef45 changed target branch from alloy to main 2026-09-02 22:57:41 +00:00
fauno added 1 commit 2026-09-11 09:54:56 +00:00
feat: access log coop-cloud/traefik#126 f9820da529
fauno force-pushed access-log from 36ef86e8b7 to f9820da529 2026-09-11 09:54:56 +00:00 Compare
fauno commented 2026-09-11 09:55:14 +00:00
Author
Owner
Copy Link
Copy Source

ready for review!

ready for review!
dannygroenewegen requested changes 2026-09-17 10:25:01 +00:00
Dismissed
dannygroenewegen left a comment
Owner
Copy Link
Copy Source

Include a short line for the release notes.

Include a short line for the release notes.
.env.sample Outdated
@@ -128,1 +128,4 @@
#ALERT_NODE_MEMORY_USAGE=85
# Tell Traefik to keep access logs (could be very verbose)
ALLOY_ACCESS_LOGS=false
dannygroenewegen commented 2026-09-17 10:24:17 +00:00
Owner
Copy Link
Copy Source

In the labels, the default value is already set to false, so can these variables in .env.sample be commented out, like the other optional features?

In the labels, the default value is already set to false, so can these variables in .env.sample be commented out, like the other optional features?
simon commented 2026-09-17 13:35:03 +00:00
Owner
Copy Link
Copy Source

preferably: make it

#ALLOY_ACCESS_LOGS=true

that way you only have to comment in if you want to enable it.

preferably: make it ``` #ALLOY_ACCESS_LOGS=true ``` that way you only have to comment in if you want to enable it.
fauno commented 2026-09-25 14:05:45 +00:00
Author
Owner
Copy Link
Copy Source

done

done
simon commented 2026-09-25 12:23:53 +00:00
Owner
Copy Link
Copy Source

I don't really get why all services get their own access-log flag. But I think I generally don't fully understand how the traefik-access-log architecture for the co-op cloud is supposed to work, yet. Will it require all services to set a label via the .env or ist this something I'll configure via traefik centrally? A bit of documentation would be the bare minimum I guess :)

I don't really get why all services get their own access-log flag. But I think I generally don't fully understand how the traefik-access-log architecture for the co-op cloud is supposed to work, yet. Will it require all services to set a label via the .env or ist this something I'll configure via traefik centrally? A bit of documentation would be the bare minimum I guess :)
fauno added 1 commit 2026-09-25 14:04:20 +00:00
fix: logging is opt-in d7bb30d346
fauno commented 2026-09-25 14:05:33 +00:00
Author
Owner
Copy Link
Copy Source

I don't really get why all services get their own access-log flag. But I think I generally don't fully understand how the traefik-access-log architecture for the co-op cloud is supposed to work, yet. Will it require all services to set a label via the .env or ist this something I'll configure via traefik centrally? A bit of documentation would be the bare minimum I guess :)

sure! i had already forgot how it worked xD

> I don't really get why all services get their own access-log flag. But I think I generally don't fully understand how the traefik-access-log architecture for the co-op cloud is supposed to work, yet. Will it require all services to set a label via the .env or ist this something I'll configure via traefik centrally? A bit of documentation would be the bare minimum I guess :) sure! i had already forgot how it worked xD
fauno requested review from dannygroenewegen 2026-09-25 14:05:50 +00:00
fauno commented 2026-10-11 23:36:58 +00:00
Author
Owner
Copy Link
Copy Source

i've been testing this for some time and i still find prometheus on the access logs 🤔

i've been testing this for some time and i still find prometheus on the access logs 🤔
You are not authorized to merge this pull request.
This pull request can be merged automatically.
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 access-log:access-log
git checkout access-log
Sign in to join this conversation.
Reviewers
No Reviewers
3wordchant
decentral1se
dannygroenewegen
Labels
No items
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) BornDeleuze (Born Deleuze) Brooke cas (Cassowary) coopcloud dannygroenewegen (Danny Groenewegen) decentral1se (d1) fauno (fauno) jjsfunhouse kawaiipunk (KawaiiPunk) knoflook moosemower moritz notplants oxaliq (sorrel) p4u1 (p4u1) renovate-bot (Comrade Renovate Bot) simon stevensting (stevensting) trav (Trav Fryer) yksflip
No Assignees
5 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/monitoring-ng#27