Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xroot: handle haproxy and checksum command #7686

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

DmitryLitvintsev
Copy link
Member

Motivation:

When using xrootd doors behind an HAProxy w/
xrootd.enable.proxy-protocol=true it has been discovered that

xrdcp --cksum adler32:<value> <source> <destiation>

hangs after upload has completed and then eventually fails after a timeout. This is due to xrootd door repoting actual door address to the client.

Modification:

Return destination address (that is haproxy address) if xrootd.enable.proxy-protocol=true is set.

Result:

xrdcp --cksum adler32:<value> <source> <destiation>

works as expected (and likely many other similar commands)

Target: trunk
Request: 10.*
Request: 9.2
Patch: https://rb.dcache.org/r/14338/
Acked-by: Tigran
Require-book: no
Require-notes: yes
Signed-off-by: Dmitry Litvintsev [email protected]
(cherry picked from commit e98ab94)

Motivation:
----------
When using xrootd doors behind an HAProxy w/
`xrootd.enable.proxy-protocol=true` it has been discovered
that
```
xrdcp --cksum adler32:<value> <source> <destiation>
```
hangs after upload has completed and then eventually fails
after a timeout. This is due to xrootd door repoting actual
door address to the client.

Modification:
-------------
Return destination address (that is haproxy address) if
`xrootd.enable.proxy-protocol=true` is set.

Result:
-------
```
xrdcp --cksum adler32:<value> <source> <destiation>
```
works as expected (and likely many other similar commands)

Target: trunk
Request: 10.*
Request: 9.2
Patch: https://rb.dcache.org/r/14338/
Acked-by: Tigran
Require-book: no
Require-notes: yes
Signed-off-by: Dmitry Litvintsev <[email protected]>
(cherry picked from commit e98ab94)
@lemora lemora merged commit 93c5074 into dCache:10.2 Oct 29, 2024
1 check passed
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.

2 participants