Skip to content

fix: update log level to debug for in progress events sent to Kubernetes/Coder#198

Merged
rowansmithau merged 1 commit into
mainfrom
rowan/k8s-events-log-level
Jun 17, 2026
Merged

fix: update log level to debug for in progress events sent to Kubernetes/Coder#198
rowansmithau merged 1 commit into
mainfrom
rowan/k8s-events-log-level

Conversation

@rowansmithau

@rowansmithau rowansmithau commented Jun 16, 2026

Copy link
Copy Markdown
Member

closes #145

This corrects the behaviour of events sent to Kubernetes to ensure Coder shows them with a blue border instead of a green, thus not implying the workspace is ready.

builds on the changes implemented in #168

With a test build, an example of image pull failure staying blue:

blue.mov

With a test build, an example of image pull staying blue (image has to be pulled, does not exist locally):

blue-image-pull-delay.mov

@rowansmithau rowansmithau self-assigned this Jun 16, 2026
@rowansmithau rowansmithau changed the title fix: update log level to debug fix: update log level to debug for in progress events sent to Kubernetes/Coder Jun 16, 2026
@rowansmithau rowansmithau marked this pull request as ready for review June 16, 2026 06:43

@matifali matifali left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. Not sure if there are any side affcts of this change in behaviour..

@matifali matifali requested a review from ethanndickson June 16, 2026 07:24

@ethanndickson ethanndickson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This corrects the behaviour of events sent to Kubernetes to ensure Coder shows them with a green border instead of a blue, thus not implying the workspace is ready.

FWICT the workspace agent border is not influenced by the logs, it's only influenced by the workspace agent lifecycle state machine. That aligns with my prior understanding.

What this PR does change, however, is it adds the blue background around the debug log lines. That appears to be the only thing it changes.

@rowansmithau rowansmithau merged commit 4308889 into main Jun 17, 2026
4 checks passed
@rowansmithau rowansmithau deleted the rowan/k8s-events-log-level branch June 17, 2026 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: kubernetes events sent from coder-logstream-kube to coder should not be displayed with a green border

3 participants