Damans227 opened a new pull request, #14424: URL: https://github.com/apache/cloudstack/pull/14424
Fixes #14409 Deleting a CKS cluster on a VPC tier removed every allow rule for ports 6443 and 22 in the tier's ACL list, so other clusters using the same list lost their rules. The rules CKS added had nothing tying them to a cluster. Now each rule CKS adds gets the cluster's id in its reason, and delete only removes the rules with that id. For clusters made before this change, whose rules have no reason, delete removes one rule per port instead of all of them. ### Test: Added unit tests with three 6443 rules in one ACL list: one for cluster A, one for cluster B, one older rule with no reason. Then delete cluster A's rules.. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
