Addressing an Overlooked Test Case: Reevaluating Pre-operation Checks in Recent Patch
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
senlin |
Fix Released
|
Undecided
|
Nguyen Ngoc Hieu |
Bug Description
I missed a test case in my recent patch. It happens when ignoring pre_op LB if the cluster is already at the minimum size [https:/
- The pre_op check actions include: CLUSTER_DEL_NODES, CLUSTER_SCALE_IN, CLUSTER_RESIZE, NODE_DELETE, CLUSTER_
- The actions among the above will skip the pre_op check if the cluster is already at the minimum size: CLUSTER_DEL_NODES, CLUSTER_SCALE_IN, CLUSTER_RESIZE, NODE_DELETE.
- The exception action is CLUSTER_
However, I quickly realized that the CLUSTER_RESIZE action should be an exception for the following reasons:
- Resizing a cluster that already has desired_capacity = min_size is still entirely valid.
- CLUSTER_RESIZE allows simultaneous changes to both desired_capacity and min_size, and they have been validated in the conductor [https:/
Changed in senlin: | |
assignee: | nobody → Nguyen Ngoc Hieu (zitechdev201) |
status: | New → In Progress |
Fix proposed to branch: master /review. opendev. org/c/openstack /senlin/ +/905429
Review: https:/