Project

General

Profile

Actions

Defect #1005

closed

Dry run starts common run on some scheduled tasks

Added by Marcel Poul about 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
High
Assignee:
Radek Tomiška
Category:
Synchronization
Target version:
Start date:
02/28/2018
Due date:
% Done:

90%

Estimated time:
Affected versions:
Owner:

Description

Even if scheduled task (e.g. Synchronization task) does not implement dry run, it is started. This can have really bad consequences, if e.g. admin wanted to try some changes on defined synchronization and do not want to save them in IdM. Affected version 7.8.2

How to reproduce: Click the blue dry run button on scheduled tasks agenda

Expected behaviour: do not show blue dru run button, if task does not implement it. Definitely do run task as regular task, if started as dry run.

Actions #1

Updated by Marcel Poul about 6 years ago

  • Tracker changed from Task to Defect
Actions #2

Updated by Marcel Poul about 6 years ago

  • Subject changed from Dry run starts common on some scheduled tasks to Dry run starts common run on some scheduled tasks
Actions #3

Updated by Radek Tomiška about 6 years ago

  • Status changed from New to In Progress
  • Target version set to Garnet (7.8.3)
Actions #4

Updated by Radek Tomiška about 6 years ago

  • 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 fixed issues:
- check for dry run is supported by LRT - FE and BE too
- button is disabled, when dry run mode is not supported
- dependent task is executed in dry run mode, if parent task was executed in dry run mode.

Commit: https://github.com/bcvsolutions/CzechIdMng/commit/c7404ce773c3895e964387b36f4ec7880bf56dfc

Could you do a feedback, pls?

Actions #5

Updated by Vít Švanda about 6 years ago

  • Status changed from Needs feedback to Closed
  • Assignee changed from Vít Švanda to Radek Tomiška

I did review and test. I cannot start dry run for sync LRT now.
Thanks for that.

Actions

Also available in: Atom PDF