The opa-bundle-builder-clusterrole is never bound

Ouverte
#823 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
52/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
Calme
Stack technique
helm, kubernetes, rust

Piste de recherche

Commencez par deploy/helm/opa-operator/templates/clusterrole-opa-builder.yaml, puis suivez la configuration du compte de service bundle-builder dans le code de l’opérateur. Vérifiez que le builder utilise son propre compte de service et que ce ClusterRole est lié, puis mettez à jour les commentaires associés de pull request #820. C’est terminé lorsque le builder n’hérite plus des permissions du compte de service de l’opérateur.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

type/bug
Affected Stackable version

Any (at least up until and including SDP 26.3)

Affected OpenPolicyAgent version

N/A

Current and expected behavior

Currently, the bundle-builder uses the operator service account, and therefore inherits operator permissions.

It should use it's own service account and be bound to the ClusterRole defined in deploy/helm/opa-operator/templates/clusterrole-opa-builder.yaml so it has only the necessary permissions to perform its function.

Possible solution
  1. Drop the unused ClusterRole
  2. Adjust the operator code so that the bundle-builder ClusterRole is used.

Option 2 is probably better, because it at least shows what is needed by the bundle-builder - and allows for architectural changes (like the bundle builder running outside of the OPA cluster pod if that was ever a possibility).

In either case, the comments in https://github.com/stackabletech/opa-operator/pull/820/changes/5dc06dbca2365f9c2957f74318d8f722e972e0d2 will need to be updated.

Additional context

Originally found here: https://github.com/stackabletech/opa-operator/pull/820#discussion_r3062985546

Environment

No response

Would you like to work on fixing this bug?

maybe

Langage dominant
Rust
Étoiles
21
Forks
5
Merge moyen
12 h 44 min
PR mergées (30 j)
11

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de stackabletech/opa-operator

Toutes les issues de stackabletech/opa-operator

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.