8000 fix: avoid missed logs when streaming startup logs by mafredri · Pull Request #8029 · coder/coder · GitHub
[go: up one dir, main page]
Skip to content

fix: avoid missed logs when streaming startup logs #8029

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 16, 2023
Merged

fix: avoid missed logs when streaming startup logs #8029

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
624e367
fix: Send startup log EOF and prevent lost messages
mafredri Jun 14, 2023
f35acd0
Merge branch 'main' into mafredri/fix-coderd-startup-log-eof
mafredri Jun 15, 2023
6fe8045
Ensure startup logs are closed on lifecycle state change
mafredri Jun 16, 2023
033262e
fix(codersdk): Fix startup log channel corruption
mafredri Jun 16, 2023
96c1ccd
Merge branch 'main' into mafredri/fix-coderd-startup-log-eof
mafredri Jun 16, 2023
1b58971
fix(site): Unshow the eof log line from WebUI
mafredri Jun 16, 2023
bcbec3e
Fix lint
mafredri Jun 16, 2023
eb7a7a3
Make EOF during lifecycle change mandatory
mafredri Jun 16, 2023
0dcfd73
Fix
mafredri Jun 16, 2023
a1a40f3
Fix after qp
mafredri Jun 16, 2023
9a49172
Merge branch 'main' into mafredri/fix-coderd-startup-log-eof
mafredri Jun 16, 2023
71f476a
Fix
mafredri Jun 16, 2023
File filter

Filter by extension

Filter by extension
8000
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
Fix after qp
  • Loading branch information
@mafredri
mafredri committed Jun 16, 2023
commit a1a40f37f72ee37ddb808ed7024815ad099b4cd0
9 changes: 3 additions & 6 deletions
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -434,19 +434,16 @@ func (api *API) workspaceAgentStartupLogs(rw http.ResponseWriter, r *http.Reques
if afterRaw != "" {
var err error
after, err = strconv.ParseInt(afterRaw, 10, 64)
if err != nil {
if err != nil || after < 0 {
httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{
Message: "Query param \"after\" must be an integer.",
Message: "Query param \"after\" must be an integer greater than zero.",
Validations: []codersdk.ValidationError{
{Field: "after", Detail: "Must be an integer"},
{Field: "after", Detail: "Must be an integer greater than zero"},
},
})
return
}
}
if after < 0 {
after = 0
}

logs, err := api.Database.GetWorkspaceAgentStartupLogsAfter(ctx, database.GetWorkspaceAgentStartupLogsAfterParams{
AgentID: workspaceAgent.ID,
Expand Down
0