villebro opened a new pull request, #33: URL: https://github.com/apache/superset-kubernetes-operator/pull/33
## Summary Remove the redundant `Config` field from all child CRD specs and move ConfigMap ownership to the parent Superset controller. Previously, rendered Python config was stored on the child CR spec AND in a ConfigMap created by the child controller, making `kubectl describe` noisy and duplicating data. Now: - Parent renders config and creates ConfigMaps directly (owned by parent CR) - Child CRDs no longer carry config content — only `ConfigChecksum` for rolling restarts - Child controllers mount ConfigMaps by conventional name without managing them - ConfigMaps survive child CR deletion (important during drain) ### Design rationale The checksum is computed by whoever writes the data. Since the parent renders and writes the ConfigMap, it passes the checksum to child CRs via `spec.configChecksum`. Child controllers stamp it as a pod annotation to trigger rolling restarts. This avoids child controllers needing to watch ConfigMaps they don't own. -- 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]
