8000 feat(site): display devcontainer start error by DanielleMaywood · Pull Request #18637 · coder/coder · GitHub
[go: up one dir, main page]
Skip to content

feat(site): display devcontainer start error #18637

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 11 commits into from
Jun 30, 2025
Merged

feat(site): display devcontainer start error #18637

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
e094e22
feat(site): surface devcontainer start error
DanielleMaywood Jun 27, 2025
8989dbb
Merge branch 'main' into danielle/surface-devcontainer-errors
DanielleMaywood Jun 30, 2025
220f25f
test: add specific test (and fix bug)
DanielleMaywood Jun 30, 2025
a3cc8e7
chore: appease formatter and linter
DanielleMaywood Jun 30, 2025
e147688
test: skip on windows
DanielleMaywood Jun 30, 2025
59bdade
chore: add error story
DanielleMaywood Jun 30, 2025
ece1fc0
chore: a devcontainer with an error will not have an agent
DanielleMaywood Jun 30, 2025
525337f
chore: feedback
DanielleMaywood Jun 30, 2025
08576b1
chore: also surface errors on 'devcontainer up'
DanielleMaywood Jun 30, 2025
9412417
chore: comment
DanielleMaywood Jun 30, 2025
c0b9d12
chore: appease linter
DanielleMaywood Jun 30, 2025
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
chore: appease linter
  • Loading branch information
@DanielleMaywood
DanielleMaywood committed Jun 30, 2025
commit c0b9d12ba5ce83de42f76d72751b44e95d2ab0a6
3 changes: 1 addition & 2 deletions agent/agentcontainers/api_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ package agentcontainers_test
import (
"context"
"encoding/json"
"errors"
"fmt"
"math/rand"
"net/http"
Expand Down Expand Up @@ -1717,7 +1716,7 @@ func TestAPI(t *testing.T) {
require.Equal(t, http.StatusAccepted, rec.Code)

// Given: We simulate an error running `devcontainer up`
simulatedError := errors.New("simulated error")
simulatedError := xerrors.New("simulated error")
testutil.RequireSend(ctx, t, fDCCLI.upErrC, simulatedError)

nowRecreateErrorTrap.MustWait(ctx).MustRelease(ctx)
Expand Down
Loading
0