Skip to content

feat(codersdk/agentsdk): add StartupLogsSender and StartupLogsWriter #8129

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.

Sign up for GitHub

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

Merged
merged 12 commits into from
Jun 22, 2023
Merged

feat(codersdk/agentsdk): add StartupLogsSender and StartupLogsWriter #8129

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
5e48ee1
feat(agentsdk): add `StartupLogsSender` and `StartupLogsWriter`
mafredri Jun 21, 2023
d078203
add test for cr
mafredri Jun 21, 2023
c80755e
improve documentatiaon
mafredri Jun 21, 2023
cdb64e2
simplify log writer in favor of documentation
mafredri Jun 21, 2023
867a66e
minor correctness fix for writer
mafredri Jun 21, 2023
0c148ca
fix typo
mafredri Jun 21, 2023
e247bb4
prevent a potential race condition on select during send
mafredri Jun 21, 2023
e98b274
fix use of wrong context in error
mafredri Jun 21, 2023
9ac5d60
fix startup log level streaming via websocket
mafredri Jun 21, 2023
27923a3
Update codersdk/agentsdk/logs_test.go
mafredri Jun 22, 2023
02bea6e
Merge branch 'main' into mafredri/feat-add-startup-logs-sender-writer…
mafredri Jun 22, 2023
b2b4993
Merge branch 'main' into mafredri/feat-add-startup-logs-sender-writer…
mafredri Jun 22, 2023
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
Next Next commit
add test for cr
  • Loading branch information
@mafredri
mafredri committed Jun 21, 2023
commit d078203f3592d7e5ee23ce391d393c681793183f
20 changes: 20 additions & 0 deletions codersdk/agentsdk/logs_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,26 @@ func TestStartupLogsWriter_Write(t *testing.T) {
},
},
},
{
name: "removes carraige return when grouped with newline",
ctx: context.Background(),
level: codersdk.LogLevelInfo,
writes: []string{"hello world\r\n", "\r\r\n", "goodbye world\n"},
want: []agentsdk.StartupLog{
{
Level: codersdk.LogLevelInfo,
Output: "hello world",
},
{
Level: codersdk.LogLevelInfo,
Output: "\r",
},
{
Level: codersdk.LogLevelInfo,
Output: "goodbye world",
},
},
},
{
name: "cancel context",
ctx: canceledCtx,
Expand Down