villebro opened a new pull request, #47: URL: https://github.com/apache/superset-kubernetes-operator/pull/47
## Summary Fixes issues identified in post-merge review of #46: - **Port mismatch (high):** When the web-server container port is customized (e.g., 9090), the maintenance nginx was still hardcoded to listen on 8088. Traffic switched to maintenance pods would hit a port where nothing is listening. Now `renderNginxConf()` and the maintenance Deployment both use the resolved web-server container port. - **Return-path failure window (medium):** If maintenance resource deletion failed after `MaintenanceActive` was set false, the Service could still select maintenance pods whose Deployment was already gone. Fixed by reordering: the Service selector now switches *before* maintenance resources are deleted. Deletion is best-effort since the resources are parent-owned (GC handles them regardless). - **Stale docs (medium):** Updated AGENTS.md, architecture/internals, architecture/overview, user-guide/lifecycle, and index.md to reflect the new parent-owned Service design, removal of `SupersetMaintenancePage` CRD, and three-task lifecycle model. - **Missing regression tests (low):** Added tests for custom port propagation, selector switching based on `MaintenanceActive`, and port resolution edge cases. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
