Repository navigation
Isolate Kafka and ZooKeeper from tenant action pods on 0.9.0 - #145
Merged
sciabarracom merged 2 commits intoOct 22, 2026
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tenant action pods in a full 0.9.0 installation can reach shared, unauthenticated Kafka and ZooKeeper endpoints. This change restricts their ingress to the required control-plane and monitoring pods while retaining existing component versions, protocols and PVC contents.
Closes apache/openserverless#276.
Changes
user-action-pod, regardless of its value.MESSAGING_SECURITY.md.Validation
The lab used an unpublished local image and exercised image replacement/restart plus the actual Kopf resume handler. The exact
ops update operatorCLI command and a new full installation were not repeated in this follow-up. The first rollout temporarily interrupted messaging while consumers/invoker health recovered.Deployment requirements and review notes
NetworkPolicy enforcement is required. Standard Kind networking without an enforcing policy provider does not establish this boundary. Administrator-added broad allow policies and compromised trusted control-plane pods remain outside the ordinary-tenant protection.
The targeted Daybreak diff review confirmed the automatic-upgrade correction and identified one remaining Medium teardown limitation: normal owner-backed Whisk deletion can bypass the explicit policy-last helper when saved process state is absent. Owner garbage collection does not guarantee policy deletion order. The contributor accepted this as non-blocking; this PR documents the limitation and does not fix it.
No release workflow, image pin, component upgrade, or application-code change is included. This branch is hosted on the contributor's personal fork.