Skip to content

Increase the timeout when waiting for the xrootd server to start#89

Merged
bbockelm merged 1 commit intoPelicanPlatform:mainfrom
ellert:test-timeout
Mar 16, 2025
Merged

Increase the timeout when waiting for the xrootd server to start#89
bbockelm merged 1 commit intoPelicanPlatform:mainfrom
ellert:test-timeout

Conversation

@ellert
Copy link
Contributor

@ellert ellert commented Feb 2, 2025

10 seconds is not always sufficient.
Here is a test run on ricci.debian.org, the Debian riscv64 porter box with an increased timeout:

    22: xrootd daemon PID: 4157210
    22: XRootD logs are available at /.../xrootd-s3-http-0.2.1/obj-riscv64-linux-gnu/tests/basic/server.log
    22: Waiting for xrootd to start (2 seconds so far) ...
    22: Waiting for xrootd to start (3 seconds so far) ...
    22: Waiting for xrootd to start (4 seconds so far) ...
    [ ... ]
    22: Waiting for xrootd to start (25 seconds so far) ...
    22: Waiting for xrootd to start (26 seconds so far) ...
    22: Waiting for xrootd to start (27 seconds so far) ...
    22: xrootd started at https://ricci:39891/

The test log and the server log from the test are attached.

testlog.txt
server.log

10 seconds is not always sufficient.
Here is a test run on ricci.debian.org, the Debian riscv64 porter box:

22: xrootd daemon PID: 4157210
22: XRootD logs are available at /.../xrootd-s3-http-0.2.1/obj-riscv64-linux-gnu/tests/basic/server.log
22: Waiting for xrootd to start (2 seconds so far) ...
22: Waiting for xrootd to start (3 seconds so far) ...
22: Waiting for xrootd to start (4 seconds so far) ...
[ ... ]
22: Waiting for xrootd to start (25 seconds so far) ...
22: Waiting for xrootd to start (26 seconds so far) ...
22: Waiting for xrootd to start (27 seconds so far) ...
22: xrootd started at https://ricci:39891/
Copy link
Collaborator

@bbockelm bbockelm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@bbockelm bbockelm merged commit e3e5683 into PelicanPlatform:main Mar 16, 2025
3 checks passed
@ellert ellert deleted the test-timeout branch March 17, 2025 05:58
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