Task #2597
closed
Event: set EXECUTED event result state when task will continue asynchronously (when AcceptedException is returned, e.g. Role recalculation - change type of result code when 2 LRTs run simultaneously)
Added by Ondrej Husník about 4 years ago.
Updated about 4 years ago.
Description
When 2 LRTs, which both recalculate roles, are supposed to be run simultaneously, an error occurs in event audit displaying "task can't be run 2 times". LRTs are rescheduled on background so they run sequentially and behavior is correct though. The error message uses too severe level and causes that result of the IdM status is "FAILED" which distresses customers. Please could you mitigate the level of result code. It could be rather info than error.
Files
- Category set to Event
- Priority changed from Normal to Low
- Target version set to 10.7.0
- Status changed from New to In Progress
- Subject changed from Role recalculation - change type of result code when 2 LRTs run simultaneously to Event: set EXECUTED event result state when task will continue asynchronously (when AcceptedException is returned, e.g. Role recalculation - change type of result code when 2 LRTs run simultaneously)
- Related to Defect #2524: Adding multiple automatic roles for organizations with many users fails, the second automatic role doesn't exist added
- Related to Defect #1538: Business role: Show cyclic role in tree, removing business sub role from business role definition, remove account from sub role added
- Status changed from In Progress to Needs feedback
- Assignee changed from Radek Tomiška to Vít Švanda
- % Done changed from 0 to 90
I improved common event processing to transform accepted exception to EXECUTED state => event is executed from the event point of view (~ task will be processed asynchronously on background).
Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/494bd8c23a18807989a01781515cdfc4ecbee1c4
Could you provide me a feedback, please?
Note: Cannot be reproduced easy way in develop. UC from dexcription was improved in #1538 (10.6.0) and in #2529 (new @IdmCheckConcurrentExecution annotation was introduced).
- Status changed from Needs feedback to Resolved
- % Done changed from 90 to 100
I did review. Thanks for this improvement.
- Assignee changed from Vít Švanda to Radek Tomiška
- Status changed from Resolved to Closed
Also available in: Atom
PDF