Skip to content

Avoid SIGHUP error message in connectivity server log file#113

Merged
bieryAtFnal merged 2 commits intodevelopfrom
kbiery/SIGHUP_fix
Nov 5, 2025
Merged

Avoid SIGHUP error message in connectivity server log file#113
bieryAtFnal merged 2 commits intodevelopfrom
kbiery/SIGHUP_fix

Conversation

@bieryAtFnal
Copy link
Copy Markdown
Collaborator

Description

We have noticed occasional error messages like the following in regression test logfiles:

pytest-9439/run4/log_biery_lr-session_local-connection-server.txt:[2025-11-04 10:59:16 -0600] [3574082] [ERROR] Worker (pid:3574083) was sent SIGHUP!

Primarily these messages appear in the connectivity server log file, but occasionally they appear in the root controller log file.

Claudia provided a fix for the connectivity server occurrences of the problem in DUNE-DAQ/daqsystemtest#259. That fix slightly changes the commands that we use to start up and shut down the connectivity server.

This PR in the listrev repo copies those changes to the ConnSrvr startup/shutdown to the listrev_test.

To these these changes, one would simply run the listrev_test.py and confirm that it completes without any complaints.

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature or enhancement (non-breaking change which adds functionality)
  • Optimization (non-breaking change that improves code/performance)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Testing checklist

  • Unit tests pass (e.g. dbt-build --unittest)
  • Minimal system quicktest passes (pytest -s minimal_system_quick_test.py)
  • Full set of integration tests pass (daqsystemtest_integtest_bundle.sh)
  • Python tests pass if applicable (e.g. python -m pytest)
  • Pre-commit hooks run successfully if applicable (e.g. pre-commit run --all-files)

Further checks

Kurt Biery added 2 commits November 4, 2025 11:10
…s.data.xml to avoid spurious SIGHUP messages (copying what Claudia did in daqsystemtest).
… exclusions in the listrev_test, now that we should no longer see those messages.
@bieryAtFnal bieryAtFnal requested a review from wanyunSu November 4, 2025 17:37
Copy link
Copy Markdown

@wanyunSu wanyunSu left a comment

Choose a reason for hiding this comment

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

LGTM

@bieryAtFnal bieryAtFnal merged commit ec3bb5d into develop Nov 5, 2025
4 checks passed
@bieryAtFnal bieryAtFnal deleted the kbiery/SIGHUP_fix branch November 5, 2025 15:12
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.

3 participants