Details
-
Bug
-
Resolution: Unresolved
-
Critical
-
None
-
8.0.0.Final, 9.0.0.Beta4
-
None
Description
Parameters attempts to set a restart level for add or remove based on the type of the handler for the op:
public Parameters setAddHandler(OperationStepHandler addHandler) { this.addHandler = addHandler; if (this.addRestartLevel == null) { this.addRestartLevel = restartLevelForAdd(addHandler); } return this; }
The problem is restartLevelForAdd(addHandler) doesn't get called because this.addRestartLevel and this.removeRestartLevel are initialized to non-null values.
The effect of this is SimpleResourceDefinition impls that moved from passing multiple params to the constructor to instead passing a Parameters object have had the restart-level description of their add and remove ops changed.
This is tricky because fixing this means some things may get changed back.