infrastructure/k8s-cluster
1
0
Fork 2
Code Issues 35 Pull requests 6 Projects Releases Packages Wiki Activity Actions

fix: do not route data.forgejo.org to code.forgejo.org for hetzner01 #741

Closed
earl-warren wants to merge 1 commit from refs/pull/753/headc8deb80c142292fb86669c1aab23e894d3594bc1 into main82e4a30788b6d02a51223613882e77073debd28d
pull from: refs/pull/753/headc8deb80c142292fb86669c1aab23e894d3594bc1
merge into: infrastructure:main82e4a30788b6d02a51223613882e77073debd28d
Conversation 3 Commits 1 Files changed 1 +2 -2
earl-warren commented 2025-12-08 13:14:31 +00:00
First-time contributor
Copy link

it is meant to allow hetzner01 and other trusted hosts to bypass rate limits but it has the unfortunate side effect or having a higher priority than the routes defined by git-mirror.yaml

it is meant to allow hetzner01 and other trusted hosts to bypass rate limits but it has the unfortunate side effect or having a higher priority than the routes defined by git-mirror.yaml
fix: do not route data.forgejo.org to code.forgejo.org for hetzner01
Some checks failed
build / lint (pull_request) Has been cancelled
Details
/ test (pull_request) Has been cancelled
Details
build / lint (push) Has been cancelled
Details
c8deb80c14
 
it is meant to allow hetzner01 and other trusted hosts to bypass rate limits
but it has the unfortunate side effect or having a higher priority than
the routes defined by git-mirror.yaml
earl-warren closed this pull request 2025-12-08 13:14:31 +00:00
earl-warren commented 2025-12-08 13:14:31 +00:00
Author
First-time contributor
Copy link

infrastructure/k8s-cluster@82e4a30788/flux/apps/forgejo-code/forgejo-route.yaml (L45-L57)

The first one is a blocker, yes. But the second one is not.

https://invisible.forgejo.org/infrastructure/k8s-cluster/src/commit/82e4a30788b6d02a51223613882e77073debd28d/flux/apps/forgejo-code/forgejo-route.yaml#L45-L57 The first one is a blocker, yes. But the second one is not.
viceice reviewed 2025-12-08 13:14:32 +00:00
viceice left a comment
First-time contributor
Copy link

those are only the crawler blocker, I don't think that's the right place

those are only the crawler blocker, I don't think that's the right place
viceice approved these changes 2025-12-08 13:14:32 +00:00
viceice left a comment
First-time contributor
Copy link

ah, the second is for the runners

ah, the second is for the runners
Some checks failed
build / lint (pull_request) Has been cancelled
Details
/ test (pull_request) Has been cancelled
Details
build / lint (push) Has been cancelled
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
viceice
Labels
Clear labels   
bug

Something is not working

  
cleanup

   
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
forgefriends

https://code.forgejo.org/infrastructure/k8s-cluster/src/branch/main/.forgejo/workflows/sync.yml

  
help wanted

Need some help

  
hetzner

Hetzner related problems and work items.

  
invalid

Something is wrong

  
label workflow

a workflow is associated with this issue and will trigger when labels are updated

  
need more info

   
question

More information is needed

  
refactor

No functional change, reorganization

  
static-site

https://code.forgejo.org/infrastructure/k8s-cluster/src/branch/main/.forgejo/workflows/sync.yml

  
sync

https://code.forgejo.org/infrastructure/k8s-cluster/src/branch/main/.forgejo/workflows/build.yml

  
wontfix

This won't be fixed

No labels
bug
cleanup
duplicate
enhancement
forgefriends
help wanted
hetzner
invalid
label workflow
need more info
question
refactor
static-site
sync
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
infrastructure/k8s-cluster!741
No description provided.