bec/csaxs_bec
0
0
Fork 1
Code Issues 16 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

WIP: Fix Eiger to respect message_severity of JFJ broker status #199

Draft
appel_c wants to merge 1 commits from feat/add-error-handling-to-on-complete-eiger into main
pull from: feat/add-error-handling-to-on-complete-eiger
merge into: bec:main
Conversation 1 Commits 1 Files Changed 1
+17 -2
appel_c commented 2026-04-28 10:16:18 +02:00
Owner
Copy Link
Copy Source

This PR adds additional logic to the complete method of the Eiger called at the end of a scan. It now checks more metadata from the JungfrauJoch to improve error handling of backend issues and forward information to the users.

  • Fix tests from Eiger.
This PR adds additional logic to the complete method of the Eiger called at the end of a scan. It now checks more metadata from the JungfrauJoch to improve error handling of backend issues and forward information to the users. - [ ] Fix tests from Eiger.
appel_c added 1 commit 2026-04-28 10:16:18 +02:00
fix(eiger): Fix on_complete from eiger to respect message_severity of JFJ broker status
CI for csaxs_bec / test (push) Failing after 2m44s
Details
CI for csaxs_bec / test (pull_request) Failing after 1m59s
Details
0f4e8b2867 
Co-authored-by: Copilot <copilot@github.com>
appel_c requested review from diaz 2026-04-28 10:16:34 +02:00
appel_c requested review from holler 2026-04-28 10:16:34 +02:00
appel_c requested review from menzel 2026-04-28 10:16:34 +02:00
appel_c requested review from wakonig_k 2026-04-28 10:16:34 +02:00
appel_c requested review from wyzula_j 2026-04-28 10:16:34 +02:00
appel_c marked the pull request as work in progress 2026-04-28 10:17:12 +02:00
diaz commented 2026-04-28 16:43:26 +02:00
Member
Copy Link
Copy Source

Thank you, Christian. I cannot judge much here...

Thank you, Christian. I cannot judge much here...
Some required checks failed
CI for csaxs_bec / test (push) Failing after 2m44s
Details
CI for csaxs_bec / test (pull_request) Failing after 1m59s
Required
Details
This pull request is marked as a work in progress.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/add-error-handling-to-on-complete-eiger:feat/add-error-handling-to-on-complete-eiger
git checkout feat/add-error-handling-to-on-complete-eiger
Sign in to join this conversation.
Reviewers
No Reviewers
wakonig_k
wyzula_j
diaz
holler
menzel
Labels
Clear labels
issue::not confirmed
Used if an issue has not yet been confirmed by a developer
 issue::not reproducible
Used if an issue is not reproducible
 issue::reproducible
Used if a developer was able to reproduce the issue
 priority::critical
Should be done as soon as possible
 priority::needed
An important update to the code base
 priority::nice to have
Not really needed but a good improvement
 scope::backend
An issue regarding the backend services
 scope::frontend
An issue regarding the user-facing services
 stage::in development
This issue is currently being worked on
 stage::on hold
Something is blocking the development
 stage::pre_release
For issues that have been merged into a pre-release branch and are waiting for their release.
stage::ready for review
For issues that have been resolved and are ready for review
 stage::selected for development
This issue has been selected for development but has not been scheduled yet
 stage::weekly backlog
This issue has been selected for development for the next development cycle
 type::bug
It's a bug!
 type::discussion
A discussion is wanted/needed
 type::doc
Improvements to the documentation
 type::feature
A new feature
 type::improvement
Neither a feature nor a bug but an improvement to the code base
 type::test
Missing tests or improvements to the test suite
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
acerbo_a (acerbo_a) appel_c (appel_c) bec-gitea celcer_t (celcer_t) debenjak_l (debenjak_l) diaz (diaz) divall_e (divall_e) guizar_m (guizar_m) haghta_s (haghta_s) holler (holler) lombosi_c (lombosi_c) menzel (menzel) perl_d (perl_d) usov_i (usov_i) wakonig_k (wakonig_k) wang_x1 (wang_x1) wyzula_j (wyzula_j) zamofing_t (zamofing_t)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bec/csaxs_bec#199
Delete Branch "feat/add-error-handling-to-on-complete-eiger"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?