search

flux-framework / flux-accounting

bank/accounting interface for the Flux resource manager
https://flux-framework.readthedocs.io/projects/flux-accounting/en/latest/index.html
GNU Lesser General Public License v3.0
3 stars 10 forks source link

plugin: add callback prefixes to exception messages #499

Closed cmoussa1 closed 1 month ago

cmoussa1 commented 1 month ago

Problem

The priority plugin uses the same exception message for jobspec updates in more than one callback because there is more than one instance where that message would be appropriate. However, these exception messages are not prefixed with the callback the exception is being raised from, which can make it hard to debug when looking at a job's eventlog.

This PR just adds a prefix of the name of the callback the exception message is coming from.

cmoussa1 commented 1 month ago

Thanks @jameshcorbett! Setting MWP here