Project

General

Profile

Actions

Task #1034

closed

Remove role request from automatic roles

Added by Ondřej Kopr about 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Ondřej Kopr
Category:
Profiling
Target version:
Start date:
03/16/2018
Due date:
% Done:

100%

Estimated time:
Owner:

Description

Remove role request from all automatic roles (role tree node, automatic role) Roles will be assigned directly via identity role with new skip check authorities.


Related issues

Related to IdStory Identity Manager - Task #1032: Profiling CzechIdM (synchronization, recalculation, hr process)ClosedOndřej Kopr03/16/2018

Actions
Actions #1

Updated by Ondřej Kopr about 6 years ago

  • Related to Task #1032: Profiling CzechIdM (synchronization, recalculation, hr process) added
Actions #2

Updated by Ondřej Kopr about 6 years ago

  • % Done changed from 0 to 90

I removed role request from role tree node and automatic role attribute:

commit (branch hotfix/7.8.4):
https://github.com/bcvsolutions/CzechIdMng/commit/59da6148278b162d24fd9685cc94e305cad81e71
https://github.com/bcvsolutions/CzechIdMng/commit/6dc334f90f3d0b644b53cc80c73febcea4ff34cf

Any test wasn't changed.

Actions #3

Updated by Ondřej Kopr about 6 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Ondřej Kopr to Radek Tomiška

Please Radek could you made a review? Test result from load test can be found in ticket #1032

Thank you :)

Actions #4

Updated by Radek Tomiška about 6 years ago

  • Assignee changed from Radek Tomiška to Ondřej Kopr

I did test and refiew, it works, thx.

Note:
I improved LRT for adding and removing automatic role by tree structure:
https://github.com/bcvsolutions/CzechIdMng/commit/d2a007d2200d06a61b4d88afbeb957f1928ae5ee
Improve 'ProcessAutomaticRoleByAttributeTaskExecutor' the same way if needed, or close this ticket if not.

Actions #5

Updated by Ondřej Kopr about 6 years ago

  • Assignee changed from Ondřej Kopr to Radek Tomiška

Thank you for feedback and sorry for some confusing methods.

I added behavior with failed entities and throw exception in LRT.

Example LRT result wit failed message:

eu.bcvsolutions.idm.core.api.exception.ResultCodeException: Automatic role [72dea55a-3963-4036-b92f-caf689a24c18] was not process correctly, failed contracts add: [], failed contracts remove: [ac655c2b-656e-4309-bd2a-621fd8c10712].
    at eu.bcvsolutions.idm.core.scheduler.task.impl.ProcessAutomaticRoleByAttributeTaskExecutor.process(ProcessAutomaticRoleByAttributeTaskExecutor.java:134)
    at eu.bcvsolutions.idm.core.scheduler.task.impl.ProcessAutomaticRoleByAttributeTaskExecutor.process(ProcessAutomaticRoleByAttributeTaskExecutor.java:1)
    at eu.bcvsolutions.idm.core.scheduler.api.service.AbstractLongRunningTaskExecutor.call(AbstractLongRunningTaskExecutor.java:171)
    at java.util.concurrent.FutureTask.run(FutureTask.java:266)
    at org.springframework.security.concurrent.DelegatingSecurityContextRunnable.run(DelegatingSecurityContextRunnable.java:80)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
    at java.lang.Thread.run(Thread.java:748)

Commit: https://github.com/bcvsolutions/CzechIdMng/commit/5fc1cf590d9c62997a42f2d5d5d02ab0549b1d07 (branch: hotfix/7.8.4)

Actions #6

Updated by Radek Tomiška about 6 years ago

  • Status changed from Needs feedback to Closed
  • Assignee changed from Radek Tomiška to Ondřej Kopr
  • % Done changed from 90 to 100

Thx, it looks nice!

Actions

Also available in: Atom PDF