-
Notifications
You must be signed in to change notification settings - Fork 7
Comparing changes
Open a pull request
base repository: coder/coder-logstream-kube
base: b45e9d5
head repository: coder/coder-logstream-kube
compare: 9f8558d
- 8 commits
- 7 files changed
- 4 contributors
Commits on Jun 3, 2025
-
Add support for multiple namespaces
- Allow CODER_NAMESPACE to accept comma-separated list of namespaces - Support watching all namespaces when CODER_NAMESPACE is empty - Automatically use ClusterRole/ClusterRoleBinding for multi-namespace or all-namespace scenarios - Update Helm chart to support both namespace-scoped and cluster-wide RBAC - Add comprehensive documentation for multi-namespace usage - Maintain backward compatibility with single namespace deployments Fixes #5
Configuration menu - View commit details
-
Copy full SHA for d2d6be2 - Browse repository at this point
Copy the full SHA d2d6be2View commit details -
Configuration menu - View commit details
-
Copy full SHA for efe2bac - Browse repository at this point
Copy the full SHA efe2bacView commit details
Commits on Jun 10, 2025
-
Fix compilation errors and update to current agentsdk API
- Replace deprecated AgentLogWriter with agentsdk.Client and PatchLogs - Fix struct field names in tests (namespace -> namespaces) - Add missing isEmpty method to tokenCache - Update setPodToken and setReplicaSetToken to return values - Fix agentLog struct literals in tests - Add PatchLogs endpoint to fake API for tests - Update test expectations to match current log message format Co-authored-by: kylecarbs <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for ac97429 - Browse repository at this point
Copy the full SHA ac97429View commit details -
Improve async log handling and fix test compatibility
- Add proper context management to logger goroutines - Fix log level conversion in fake API PatchLogs method - Improve goroutine lifecycle with cancellation support - Use ticker-based approach for more reliable log sending - Continue on PatchLogs errors instead of exiting goroutine Co-authored-by: kylecarbs <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for f6b43a0 - Browse repository at this point
Copy the full SHA f6b43a0View commit details -
Improve async log handling and fix test compatibility
- Fix Test_logQueuer to use real clock and proper timing - Add clock advances and sleeps to handle async log processing - Improve test reliability for TestPodEvents and TestReplicaSetEvents - Use quartz.NewTicker for proper mock clock integration - Simplify test expectations for better CI compatibility Co-authored-by: kylecarbs <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 8700cc8 - Browse repository at this point
Copy the full SHA 8700cc8View commit details
Commits on Jun 19, 2025
-
Fix test timeouts by making tick interval configurable
- Add configurable tickInterval to logQueuer struct - Update Test_logQueuer to use real clock with fast tick interval (10ms) - Update TestPodEvents and TestReplicaSetEvents to use real clock - Replace mock clock advances with real time waits - Fix Test_logQueuer test which now passes consistently The new ticker-based logging implementation requires different timing than the original synchronous approach. Tests now use fast tick intervals (10ms) with real clocks instead of mock clocks to avoid timing conflicts. Co-authored-by: matifali <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 68df1af - Browse repository at this point
Copy the full SHA 68df1afView commit details -
Configuration menu - View commit details
-
Copy full SHA for e95d523 - Browse repository at this point
Copy the full SHA e95d523View commit details -
- 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]>
Configuration menu - View commit details
-
Copy full SHA for 9f8558d - Browse repository at this point
Copy the full SHA 9f8558dView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff b45e9d5...9f8558d