Address review comments

blink-so[bot] · matifali · blink-so[bot] · commit 9f8558d252e6 · 2025-06-19T08:05:45.000Z
- Restore discord and godoc badges in README.md
- Fix release link to point to correct repository releases page
- Use consistent naming for namespace-scoped Role with release prefix
- Clean up commented-out legacy securityContext lines in values.yaml

The sourceUUID variable is actually used in tests so no changes needed there.
YAML formatting in service.yaml is already correct.

Co-authored-by: matifali &lt;10648092+matifali@users.noreply.github.com&gt;
diff --git a/README.md b/README.md
@@ -1,6 +1,8 @@
 # coder-logstream-kube
 
-[![Go Reference](https://pkg.go.dev/badge/github.com/coder/coder-logstream-kube.svg)](https://pkg.go.dev/github.com/coder/coder-logstream-kube)
+[![discord](https://img.shields.io/discord/747933592273027093?label=discord)](https://discord.gg/coder)
+[![release](https://img.shields.io/github/v/tag/coder/coder-logstream-kube)](https://github.com/coder/coder-logstream-kube/releases)
+[![godoc](https://pkg.go.dev/badge/github.com/coder/coder-logstream-kube.svg)](https://pkg.go.dev/github.com/coder/coder-logstream-kube)
 [![license](https://img.shields.io/github/license/coder/coder-logstream-kube)](./LICENSE)
 
 Stream Kubernetes Pod events to the Coder startup logs.
diff --git a/coder-logstream-kube b/coder-logstream-kube
diff --git a/helm/templates/service.yaml b/helm/templates/service.yaml
@@ -43,7 +43,7 @@ subjects:
 apiVersion: rbac.authorization.k8s.io/v1
 kind: Role
 metadata:
-  name: coder-logstream-kube-role
+  name: {{ .Release.Name }}-coder-logstream-kube-role
 rules:
 - apiGroups: [""]
   resources: ["pods", "events"]
@@ -59,7 +59,7 @@ metadata:
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: Role
-  name: coder-logstream-kube-role
+  name: {{ .Release.Name }}-coder-logstream-kube-role
 subjects:
 - kind: ServiceAccount
   name: {{ .Values.serviceAccount.name | quote }}
diff --git a/helm/values.yaml b/helm/values.yaml
@@ -99,8 +99,3 @@ labels: {}
 # securityContext -- Container-level security context
 # See: https://kubernetes.io/docs/tasks/configure-pod-container/security-context/
 securityContext: {}
-  # allowPrivilegeEscalation: false
-  # capabilities:
-  #   drop:
-  #   - ALL
-  # readOnlyRootFilesystem: true
