Task #2052: Add check for events, lrts and sync in notification status - extras - IdStory Identity Manager

Actions

Copy link

Task #2052

open

Add check for events, lrts and sync in notification status

Added by Patrik Stloukal almost 5 years ago. Updated about 4 years ago.

Status:

In Progress

Priority:

Normal

Assignee:

Peter Štrunc

Target version:

Start date:

02/12/2020

Due date:

% Done:

90%

Estimated time:

Owner:

Description

To CzechIdm status notification add check for:
- many created events, which could be waiting in queue
- running lrt (lrt could be stuck and still running in the morning)
- running time of synchronizations (synchronization running time could take too much time due to some circumstances)

Actions

Copy link

Updated by Patrik Stloukal almost 5 years ago

% Done changed from 0 to 50

I added 2 parameters for lrt CzechIdmNotificationStatus.
maximum number of created events, if in time of running this task created events would be more then this parameter number, it will be added to notification.
second parameter is number of hours. If some lrt will be running in time of this notification, it will check duration of this task.
But this parameter is even used in checking of synchronizations. It will check the duration of the last run of each sync and if it will exceed hour value of this parameter, it will be mentioned in notification.

For now it is compatible with new version 10.0.0

commit: https://github.com/bcvsolutions/czechidm-extras/commit/299f29f080d0bcc5a06c60448024dca4c336f781

Actions

Copy link

Updated by Patrik Stloukal almost 5 years ago

% Done changed from 50 to 70

In original version only last synchronization log is checked, because back then there was not attribute from in filter.
Now I changed behaviour and all sync logs from last notification run are checked.

commit: https://github.com/bcvsolutions/czechidm-extras/commit/e5b149174d35b7c4c694d819116713ab027c0153

Actions

Copy link

Updated by Ondřej Kopr almost 5 years ago

Together we made first part of review:

code for version 10.0.0 is OK,
code for version 9.7.x contains ZonedDateTime, this class was moved from version 10, please change this class back to DateTime (joda time),
changelog information missing for both version,
documentation missing,
tests missing (only for new feature).

After you update all these things we will continue with check this feature via frontend.

After consultation we also check sonar.

Actions

Copy link