DUNE-DAQ / DUNE-DAQ/confmodel

Should `Controller` be exclusive to `Segment`?

Open
#70 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
1
Forks
2
Avg merge
7h 30m
Merged PRs (30d)
3

Description

I wonder whether we could have a controller that is attached to more than one segment: is-exclusive="yes" on this line is implying that to be the case, but I think that could be relaxed.

This leads to code that doesn't look too good in daqsystemtest here and here, which are effectively the same controller.

Of course, doing that would allow you to use the same controller for all the segments (for example you wouldn't need df-controller, ru-controller, ... to be defined, just have one), which of course wouldn't work, unless we name the controller after their segment they control (relevant code in the run control: here for the place where we use names like root-controllerto grab the controller configuration and here where the name of the controller is planted into the command line).

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at schema/confmodel/dunedaq.schema.xml around line 354, then inspect the two daqsystemtest example-configs.data.xml references and the shared controller in ccm.data.xml. Read the linked drunc controller interface and oks_parser.py locations to understand how controller names select configuration. Done requires a decided design for shared controllers and coordinated updates across these configurations and consumers; no tests are named.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, python, xml
Domain
backend, tooling
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.