Skip to content

Modified the 3ru_3df_multirun_test to make use of the 'start-run' command... - #335

Merged
bieryAtFnal merged 1 commit into
developfrom
kbiery/use_start_run_in_3ru_3df
Aug 20, 2026
Merged

Modified the 3ru_3df_multirun_test to make use of the 'start-run' command...#335
bieryAtFnal merged 1 commit into
developfrom
kbiery/use_start_run_in_3ru_3df

Conversation

@bieryAtFnal

Copy link
Copy Markdown
Contributor

…for one of its three runs. This will validate the use of that command going forward.

Description

At today's SWI&T meeting, Michal mentioned that he noticed that the start-run command in run control was running boot and conf when those transitions had already been run in the current DAQ session.

Surprisingly, we don't seem to have an integration test that uses start-run, but we certainly should.

This PR makes use of that command in one of the three data-taking runs that are part of the 3ru_3df_multirun_test.

The modified version of this integtest will (and should) fail until the bug is found and fixed in run control.

Type of change

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

Testing checklist

  • Full set of integration tests pass (dunedaq_integtest_bundle.sh)
    • NB, this test will fail initially and that is a good thing. The failure demonstrates a problem in run control.

… control command for one of its three runs. This will validate the use of that command going forward.
@bieryAtFnal
bieryAtFnal requested a review from MRiganSUSX August 18, 2026 21:58

@MRiganSUSX MRiganSUSX left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank @bieryAtFnal.

Can confirm that:

  • running 3ru_3df_multirun_test the errors are caught
Image Image Image

(although not directly mentioned in this Failures end block (?))

  • running with drunc @ DUNE-DAQ/drunc#1015 which is the fix targetting this, there are no more errors
Image

@bieryAtFnal

Copy link
Copy Markdown
Contributor Author

@MRiganSUSX , thanks for your review. I'm going to go ahead and merge these changes to the develop branch, and I've created an Issue in the integrationtest repo to request that we make the text in the 'failure end block' more useful (DUNE-DAQ/integrationtest#170).

@bieryAtFnal
bieryAtFnal merged commit 6d0a098 into develop Aug 20, 2026
6 checks passed
@bieryAtFnal
bieryAtFnal deleted the kbiery/use_start_run_in_3ru_3df branch August 20, 2026 13:36
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