Task #2461
closedEmpty provisioning activates provisioning break for UPDATE
100%
Description
Provisioning break counts all provisioning operations including empty provisioning.
However empty provisioning does effectively nothing on the system. The break generates false-positive warnings or even disables update operation when there is no real need to do so.
Provisioning break should ignore empty provisioning operation when checking limits for warning or operation disabling.
Updated by Vít Švanda over 3 years ago
- Priority changed from Normal to High
- Target version set to 12.0.0
Updated by Vít Švanda about 3 years ago
- Assignee changed from Vít Švanda to Ondrej Husník
Updated by Radek Tomiška about 3 years ago
- Target version changed from 12.0.0 to 12.1.0
Updated by Roman Kučera about 3 years ago
- Target version changed from 12.1.0 to 12.2.0
Updated by Peter Štrunc almost 3 years ago
- Target version changed from 12.2.0 to 13.0.0
Updated by Roman Kučera over 2 years ago
- Status changed from New to In Progress
- Assignee changed from Ondrej Husník to Roman Kučera
Updated by Roman Kučera over 2 years ago
- % Done changed from 0 to 60
Implemented in https://github.com/bcvsolutions/CzechIdMng/tree/kucerar/2461-ignore-empty-provisioning-in-break
Empty provisioning is not counted in provisioning break.
If system is already blocked, all new provisionings will be in queue as previously.
I'll check tests and eventually write some new one for this behavior.
Updated by Roman Kučera over 2 years ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Roman Kučera to Tomáš Doischer
- % Done changed from 60 to 80
Test fixed + added new one.
PR https://github.com/bcvsolutions/CzechIdMng/pull/233
@doischert Can you please do a review?
Updated by Tomáš Doischer over 2 years ago
- Sprint set to Sprint 12.3-1 (srp 03 - srp 17)
Updated by Tomáš Doischer over 2 years ago
- Status changed from Needs feedback to Resolved
- Assignee changed from Tomáš Doischer to Roman Kučera
- % Done changed from 80 to 100
LGTM, merged to develop.
Updated by Tomáš Doischer almost 2 years ago
- Status changed from Resolved to Closed