-
Notifications
You must be signed in to change notification settings - Fork 888
docs: add workspace process logging doc #9002
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Docs LGTM. I only have basic feedback!
We provide working example templates for Kubernetes, and Kubernetes with | ||
`envbox` (for [Docker support in workspaces](./docker-in-workspaces.md)). You | ||
can view these templates in the | ||
[exectrace repo](https://github.com/coder/exectrace/tree/main/enterprise/templates). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any ideas on how to avoid drift between these example templates and the code snippets in our docs? Perhaps we add a comment to both places informing users to update the other place?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added comments here and I'll add comments in the exectrace repo in a sec
docs/templates/process-logging.md
Outdated
### View logs in AWS EKS | ||
|
||
If you're using AWS' Elastic Kubernetes Service, you can [configure your | ||
cluster][eks-cloudwatch] to send logs to CloudWatch. This allows you to view the | ||
logs for a specific user or workspace. | ||
|
||
To view your logs, go to the CloudWatch dashboard (which is available on the | ||
**Log Insights** tab) and run a query similar to the following: | ||
|
||
```text | ||
fields @timestamp, log_processed.fields.cmdline | ||
| sort @timestamp asc | ||
| filter kubernetes.container_name="exectrace" | ||
| filter log_processed.fields.labels.username="zac" | ||
| filter log_processed.fields.labels.workspace_name="code" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice addition!
Very similar to v1 but contains a section explaining how to add to existing templates.
Closes #8838