[bug]: v0.0.12 helm chart now requires that you pass in your own namespace

Offen
  • Stimmt etwas nicht?
#126 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
35/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
helm, kubernetes
Bereich
authorization, devops

Rechercherichtung

Überprüfe PR #124 und die Helm-Chart-Templates, die Namespaces, RBAC-Ressourcen und Umgebungsvariablen definieren. Vergleiche Namespace-bezogene und Cluster-bezogene Deployments und überprüfe anschließend, dass ein nicht gesetztes oder explizites rbacScope das entsprechende Verhalten von Role/RoleBinding oder ClusterRole/ClusterRoleBinding erzeugt, ohne Werte-Dateien pro Namespace zu erfordern.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

In https://github.com/coder/coder-logstream-kube/pull/124 you now must pass in your own namespace as a parameter to the helm chart if you don't want to create a ClusterRole and ClusterRoleBinding. This is rough from a GitOps perspective, because it means that you have two choices (in an environment where applications are not allowed to create cluster-scoped resources):

  1. You create values.<something>.yaml files where you set namespaces: [ my-coder-ns ] and launch the application with custom values files for each namespace you put it in.
  2. When you launch the application you pass in --set namespaces=...

In my opinion, if namespaces is not set, then there should be a simple setting of rbacScope: <namespace|cluster> ... and if it is set to namespace then the RBAC permissions created with Role/RoleBindings (and NAMESPACES=${{ .Release.Namespace}} is set as an env variable). If it is set to cluster, then you create ClusterRole/ClusterRoleBindings.

Vorherrschende Sprache
Go
Sterne
28
Forks
9
Ø Merge
7 T. 11 Std.
Gemergte PRs (30 T.)
6

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus coder/coder-logstream-kube

Alle Issues in coder/coder-logstream-kube

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.