Skip to content

Add support for multiple namespaces #107

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Address review comments
- 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 <[email protected]>
  • Loading branch information
blink-so[bot] and matifali committed Jun 19, 2025
commit 9f8558d252e606458ea9e159b5a516cd47c4b3c6
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -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.
Expand Down
Binary file added coder-logstream-kube
Binary file not shown.
4 changes: 2 additions & 2 deletions helm/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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"]
Expand All @@ -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 }}
Expand Down
5 changes: 0 additions & 5 deletions helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Loading