Isolate full-deployment Kafka and ZooKeeper from tenant action pods on 0.9.0
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 38/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- kubernetes, python
- Domaine
- infrastructure, security
Piste de recherche
Start by reading the operator code that applies NetworkPolicies and handles resume and Whisk updates; the payload does not name specific files or tests. Confirm how Kafka and ZooKeeper policies are reconciled, workloads are ordered, and legacy pods are restarted. Done means tenant action pods cannot reach either service while trusted clients, monitoring, and scheduled actions continue to work on a NetworkPolicy-enforcing CNI.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
In the 0.9.0 full deployment, ordinary tenant action pods can reach the shared Kafka and ZooKeeper endpoints. These are internal control-plane services and should not be accessible to tenant code. The deployment does not install a network boundary restricting those services to their intended clients.
This is an internal multi-tenant isolation problem. It does not establish an unauthenticated Internet-facing entry point, and slim installations without Kafka/ZooKeeper are outside this issue.
Attacker and risk
| Actor / position | Preconditions | Risk |
|---|---|---|
| Anonymous Internet client | Only the public HTTP ingress is exposed | No direct Kafka/ZooKeeper access demonstrated by these checks |
| Ordinary tenant/developer | Can deploy and invoke an action in their own namespace | Reachable unauthenticated shared messaging and coordination endpoints; potential cross-tenant confidentiality, integrity and availability impact |
| Host or Kubernetes administrator | Already controls the cluster | Outside the tenant-isolation boundary |
Expected behavior
Action pods must be denied access to Kafka and ZooKeeper by Service name, pod DNS name and pod IP. Controller/invoker traffic, broker coordination and monitoring must keep working.
Updating the operator on an existing full installation must install the protection automatically, including closing pre-existing messaging connections.
Proposed change
The accompanying operator PR targets 0.9.0 and adds narrow same-namespace ingress NetworkPolicies for Kafka and ZooKeeper. Trusted client selectors exclude every pod carrying user-action-pod, regardless of its value. Metrics access is limited to the installation's Prometheus pods.
The operator applies policies before workloads, rolls legacy messaging pods once, waits for completion, and reconciles enabled ZooKeeper then Kafka during resume and Whisk updates. Failures propagate; disabled components are skipped.
This requires a CNI that enforces NetworkPolicy. A policy object alone is not proof of isolation. The change preserves component versions, protocols and PVC contents; it does not add Kafka SASL/TLS or ZooKeeper authentication.
Validation
Local synthetic K3s lab:
- 16/16 tenant connection attempts were reachable in the legacy fixture and denied after automatic operator resume.
- Legitimate clients, Prometheus, action-labelled spoof checks and foreign-namespace checks passed.
- Repeating operator resume preserved messaging pod identities.
- Alpha/Beta application tests: 15/15; Redis Lua: 84/84; Streamer: 24/24.
- Scheduled actions continued working, and application source remained unchanged.
- Operator unit suite: 66/66, including 19 focused messaging tests.
The lab exercised operator image replacement/restart and the real Kopf resume handler. The exact ops update operator CLI command was not run with the unpublished local image.
Known limitation
Whisk uninstall still relies on owner garbage collection when the process-local saved component spec is absent. Policy-last teardown is therefore not guaranteed during uninstall. This was identified in the targeted review and accepted by the contributor as a non-blocking limitation of this proposal. It remains disclosed for maintainer assessment.
- Langage dominant
- Python
- Étoiles
- 577
- Forks
- 29
- Merge moyen
- 3 j 17 h
- PR mergées (30 j)
- 4
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de apache/openserverless
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
apache/openserverless#269 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
apache/openserverless#267 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
apache/openserverless#277 ·
-
Streamer 0.9.0 hardening: authenticated callbacks and backward-incompatible client migrationOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
apache/openserverless#274 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 8/100
apache/openserverless#273 · 2 commentaires ·
Toutes les issues de apache/openserverless
Issues similaires
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
fossasia/visdom-dev#77 ·
Les mainteneurs répondent en général sous 4 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
aws-cloudformation/cfn-lint#4890 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
volcengine/OpenViking#6521 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
AstrBotDevs/AstrBot#10720 ·
Les mainteneurs répondent en général sous 1 jour