Session Manager should not return `EXECUTED_SUCCESSFULLY` when config files fail to parse
Open
@PawelPlesniak is already working on this.
Since Jul 23, 2026.
Session Manager
- Dominant language
- Python
- Stars
- 3
- Forks
- 5
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 10
Description
Currently, in session_manager.py, the list_all_configs method returns ResponseFlag.EXECUTED_SUCCESSFULLY even when configuration files are missing or fail to parse. While in these scenarios the errors are raised by conffwk.Configuration, the methods still return EXECUTED_SUCCESSFULLY, which could lead to confusion or misleading behaviour.
Expected Behaviour:
Return ResponseFlag.FAILED or a more appropriate status when:
DUNEDAQ_DB_PATHis not set- No config files are found
- All config files fail to parse
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.