Skip to content

Modified the integtests to reflect changes in the names of the drunc_config "session" parameters#313

Merged
bieryAtFnal merged 3 commits intodevelopfrom
kbiery/integtest_session_name_changes
Apr 16, 2026
Merged

Modified the integtests to reflect changes in the names of the drunc_config "session" parameters#313
bieryAtFnal merged 3 commits intodevelopfrom
kbiery/integtest_session_name_changes

Conversation

@bieryAtFnal
Copy link
Copy Markdown
Contributor

@bieryAtFnal bieryAtFnal commented Apr 15, 2026

Description

These changes are intended to be included in fddaq-v5.7.0...

As described in DUNE-DAQ/integrationtest#153, I believe that there is value in making the names of the session-based parameters in our integrationtest drunc_config data structure more clear. The changes in this PR reflect the new names that are part of DUNE-DAQ/integrationtest#153. Please see the integrationtest PR for more details, including suggested instructions for testing the changes.

Two small additional changes that are part of this PR are the following:

  • I removed references to the currently-unused data_rate_slowdown_factor configuration parameter
  • I changed the configuration session name that should be used in a couple of integtests to better reflect the purpose of the test

Type of change

  • Optimization (non-breaking change that improves code/performance)

Testing checklist

  • Full set of integration tests pass (dunedaq_integtest_bundle.sh)

Kurt Biery added 3 commits March 8, 2026 12:28
…ig_session_name and renamed drunc_config session_name as daq_session_name to help make their meaning more clear. Also removed obsolete references to data_rate_slowdown_factor.
@bieryAtFnal bieryAtFnal merged commit 72a81fb into develop Apr 16, 2026
4 of 5 checks passed
@bieryAtFnal bieryAtFnal deleted the kbiery/integtest_session_name_changes branch April 16, 2026 20:51
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