Skip to content

Prevent redirect for readiness probes#323

Open
maxnoe wants to merge 2 commits intomasterfrom
avoid-probe-redirect
Open

Prevent redirect for readiness probes#323
maxnoe wants to merge 2 commits intomasterfrom
avoid-probe-redirect

Conversation

@maxnoe
Copy link
Contributor

@maxnoe maxnoe commented Jan 25, 2026

In our deployment we observe these logs on the server:

│ rucio-server [2026-01-25 11:27:15]    10.244.0.51    _gateway    -    aXX-EwYUDVckA2Fh8qI1MAAAAQA    308    1711    245    3283572    "GET /ping HTTP/1.1"    "-"    "kube-probe/1.34"   │
│ rucio-server [2026-01-25 11:27:15]    10.244.0.51    _gateway    -    aXX-E6xq_YR1ydXMBK0lHQAAAYA    308    1711    245    2722894    "GET /ping HTTP/1.1"    "-"    "kube-probe/1.34"   │
│ rucio-server [2026-01-25 11:27:18]    10.244.0.51    _gateway    -    aXX-FgYUDVckA2Fh8qI1MQAAAQI    200    1751    21    3256241    "GET /ping/ HTTP/1.1"    "-"    "kube-probe/1.34"   │
│ rucio-server [2026-01-25 11:27:18]    10.244.0.51    _gateway    -    aXX-FgYUDVckA2Fh8qI1MgAAAQM    200    1751    21    3255756    "GET /ping/ HTTP/1.1"    "-"    "kube-probe/1.34"

So the initial probe gets a 308 redirect from /ping to /ping/ and only then the ping succeeds.

Directly using /ping/ should avoid these redirects.

In our deployment we observe these logs on the server:

```
│ rucio-server [2026-01-25 11:27:15]    10.244.0.51    _gateway    -    aXX-EwYUDVckA2Fh8qI1MAAAAQA    308    1711    245    3283572    "GET /ping HTTP/1.1"    "-"    "kube-probe/1.34"   │
│ rucio-server [2026-01-25 11:27:15]    10.244.0.51    _gateway    -    aXX-E6xq_YR1ydXMBK0lHQAAAYA    308    1711    245    2722894    "GET /ping HTTP/1.1"    "-"    "kube-probe/1.34"   │
│ rucio-server [2026-01-25 11:27:18]    10.244.0.51    _gateway    -    aXX-FgYUDVckA2Fh8qI1MQAAAQI    200    1751    21    3256241    "GET /ping/ HTTP/1.1"    "-"    "kube-probe/1.34"   │
│ rucio-server [2026-01-25 11:27:18]    10.244.0.51    _gateway    -    aXX-FgYUDVckA2Fh8qI1MgAAAQM    200    1751    21    3255756    "GET /ping/ HTTP/1.1"    "-"    "kube-probe/1.34"
```

So the initial probe gets a 308 redirect from `/ping` to `/ping/` and the the ping succeeds.

Directly using `/ping/` should avoid these redirects.
@maxnoe maxnoe requested a review from lobis January 25, 2026 11:28
@maxnoe maxnoe requested review from bari12 and removed request for lobis January 29, 2026 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant