Project

General

Profile

Actions

Task #2461

closed

Empty provisioning activates provisioning break for UPDATE

Added by Vladimír Kotýnek over 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
High
Assignee:
Roman Kučera
Category:
Provisioning
Target version:
Start date:
08/27/2020
Due date:
% Done:

100%

Estimated time:
Owner:

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.

Actions #2

Updated by Vít Švanda over 2 years ago

  • Priority changed from Normal to High
  • Target version set to 12.0.0
Actions #3

Updated by Vít Švanda over 2 years ago

  • Assignee changed from Vít Švanda to Ondrej Husník
Actions #4

Updated by Radek Tomiška over 2 years ago

  • Target version changed from 12.0.0 to 12.1.0
Actions #6

Updated by Roman Kučera over 2 years ago

  • Target version changed from 12.1.0 to 12.2.0
Actions #7

Updated by Peter Štrunc about 2 years ago

  • Target version changed from 12.2.0 to 13.0.0
Actions #8

Updated by Roman Kučera almost 2 years ago

  • Status changed from New to In Progress
  • Assignee changed from Ondrej Husník to Roman Kučera
Actions #9

Updated by Roman Kučera almost 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.

Actions #10

Updated by Roman Kučera almost 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?

Actions #11

Updated by Tomáš Doischer over 1 year ago

  • Sprint set to Sprint 12.3-1 (srp 03 - srp 17)
Actions #12

Updated by Tomáš Doischer over 1 year 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.

Actions #13

Updated by Tomáš Doischer about 1 year ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF