dheeraj12347 commented on PR #109: URL: https://github.com/apache/cloudstack-kubernetes-provider/pull/109#issuecomment-6150115240
Hi @vishesh92 , @DaanHoogland , I reviewed the changes for per-Service Network ACL ownership. The ownership and cleanup approach looks worth testing carefully, particularly when multiple Services share a port. One additional improvement is to use `%w` when wrapping underlying errors so callers can inspect the original errors. I also noticed the E2E helper `requireNoACLRules` checks only exact start/end port matches, so it may miss a leftover rule whose port range includes the requested port. The existing Copilot comment flags this as well. -- 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]
