You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Tests ran on: WHAT HOSTNAME from release RELEASE_NAME
Unit tests - some tests can't be ran on the CI. This is documented. If this PR checks a feature that can't be tested with CI, this has been marked appropriately.
Integration tests - the daqsystemtest_integtest_bundle requires a lot of resources, and connections to the EHN1 infrastructure. Check the cross referenced list if you can't run these. The developer needs to run at least the .
Unit tests (pytest --marker) passed
With relevant marker
Without marker
Integration tests passed
Only daqsystemtest_integtest_bundle.sh -k minimal_system_quick_test.py
Full daqsystemtest_integtest_bundle.sh
Testing skipped as there are no core code changes in this PR, this only relates to documentation/CI workflows
Once the above boxes are checked, the PR(s) can be merged following the steps below.
Prior to merging
Choose one of the following an complete all substeps
Changes only affect the Run Control, are in a single repository, and do not affect the end user.
Changes are documented in docstrings and code comments
Wiki has been updated if architectural or endpoint changes
Otherwise
Workflow changes demonstrated in the Change Log (if necessary)
Wiki has been updated (if necessary)
#dunedaq-integration Slack channel notified (see below)
Once completed, the reviewer can merge the PR.
Notification message for a Slack channel
Note - this should be to #dunedaq-integration for general workflow that isn't during a release candidate period, and to #daq-release-prep otherwise.
For an single merge that changes the user workflow
The CCM WG has an isolated PR ready to merge that affects user workflows. The PR is:
_URL_
I will leave time for any comments, otherwise will merge these at the end of the work day _Insert your time zone_.
For co-ordinated merge
The CCM WG has a set of co-ordinated merges ready to merge. The PRs are:
_URL_
_URL_
I will leave time for any comments, otherwise will merge these at the end of the day.
When running the option --pytest-options "--no-integtest-connsvc" with integration tests, I first noticed the following
[2026/07/08 15:48:11 UTC] WARNING process_manager_driver.py:602 drunc.process_manager_driver Connectivity service client (csc) is not available. Using static OKS configuration only.
To investigate the cause of this further, I added some CRITICAL logs to see what is being manipulated with the connectivity service client. This results in
++++++++++ DRUNC Session BEGIN ++++++++++
[2026/07/08 15:48:10 UTC] INFO shell.py:186 drunc.unified_shell Setting up to use the process manager with configuration ssh-standalone and configuration id "minimal" from
oksconflibs:/tmp/pytest-of-pplesnia/pytest-192/config0/integtest-session-resolved.data.xml
[2026/07/08 15:48:10 UTC] INFO shell.py:208 drunc.unified_shell Starting process manager
[2026/07/08 15:48:10 UTC] INFO process_manager.py:111 drunc.process_manager process_manager communicating through address 10.73.136.70:41779
[2026/07/08 15:48:10 UTC] CRITICAL shell.py:277 drunc.unified_shell Investigating the session DAL's connectivity service: None
[2026/07/08 15:48:10 UTC] INFO shell.py:537 drunc.unified_shell unified_shell ready with process_manager and controller commands
Which indicates that the connectivity service is not present in the session DAL. I am unclear as to whether this is the intent behind --pytest-options "--no-integtest-connsvc", but this block in the integrationtest repository suggests whether the connectivity service should be disabled or not. As far as I am aware, and following this issue, it seems to me that as of now the connectivity service is a requirement of the session, but this should be discussed further, and seems like a motivation to develop towards addressing #397 .
I leave this open for discussion with @bieryAtFnal as he is the original poster of issue #909
PawelPlesniak
changed the title
Investigation started
Failures of integration tests with disabled connectivity services
Jul 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes issue #909
Initial PR for investigating the cause of failures of tests that use the
"--no-integtest-connsvc"option in integration tests.Type of change
List of required branches from other repositories
N/A
Change log
Only logs for now
Suggested manual testing checklist
Developer checklist
Prior to marking this as "Ready for Review"
Tests ran on: WHAT HOSTNAME from release RELEASE_NAME
Unit tests - some tests can't be ran on the CI. This is documented. If this PR checks a feature that can't be tested with CI, this has been marked appropriately.
Integration tests - the
daqsystemtest_integtest_bundlerequires a lot of resources, and connections to the EHN1 infrastructure. Check the cross referenced list if you can't run these. The developer needs to run at least the .pytest --marker) passeddaqsystemtest_integtest_bundle.sh -k minimal_system_quick_test.pydaqsystemtest_integtest_bundle.sh./scripts/drunc_integtest_bundle.sh)Final checklist prior to marking this as "Ready for Review"
Reviewer checklist
src/daqsystemtest_integtest_bundle.sh -k minimal_system_quick_test.pyifdaqsystemtest_integtest_bundle.shdruncare in the log filesdruncfailure appears:scripts/drunc_integtest_bundle.sh)Once the above boxes are checked, the PR(s) can be merged following the steps below.
Choose one of the following an complete all substepsPrior to merging
Once completed, the reviewer can merge the PR.
Notification message for a Slack channel
Note - this should be to #dunedaq-integration for general workflow that isn't during a release candidate period, and to #daq-release-prep otherwise.
For an single merge that changes the user workflow
For co-ordinated merge