Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: Add workspace agent lifecycle state reporting #5785

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
mafredri merged 24 commits into from
Jan 24, 2023
Merged

feat: Add workspace agent lifecycle state reporting #5785

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
40e9c87
feat: Add new migrations
mafredri Jan 19, 2023
5abf555
Generate database changes
mafredri Jan 19, 2023
a84e467
feat: Add agent state reporting
mafredri Jan 19, 2023
8119e71
Generate docs
mafredri Jan 19, 2023
515e342
WIP
mafredri Jan 19, 2023
1e53635
Rename state -> lifecycle state
mafredri Jan 19, 2023
534f954
Add tests, improve lifecycle reporting
mafredri Jan 23, 2023
90192e5
Fix lint
mafredri Jan 23, 2023
aaf19bc
test: Update terraform tests for provider v0.6.7
mafredri Jan 23, 2023
bd1a87f
test(site): Update agent mocks
mafredri Jan 23, 2023
df9944e
Add new columns to workspace agent table
mafredri Jan 23, 2023
60f414f
Fix plumbing
mafredri Jan 23, 2023
087070e
Run OverrideVSCodeConfigs even on error
mafredri Jan 23, 2023
c6928e9
Use consistent Seconds terminology
mafredri Jan 23, 2023
c132d4e
Fix s/continue/break/
mafredri Jan 23, 2023
903e850
Fix git auth override order
mafredri Jan 23, 2023
034a850
Fix nit
mafredri Jan 23, 2023
f76247f
Publish workspace update
mafredri Jan 23, 2023
2ccdcac
Hide report lifecycle endpoint from apidocs
mafredri Jan 24, 2023
830f9f1
Set all existing agents to ready
mafredri Jan 24, 2023
2b1569f
Add debug logging to lifecycle states
mafredri Jan 24, 2023
71706e0
Add note about error during vscode git auth override
mafredri Jan 24, 2023
21a9d28
Fix typo in tests
mafredri Jan 24, 2023
da67c73
Merge branch 'main' into mafredri/feat-add-workspace-agent-ready-status
mafredri Jan 24, 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 note about error during vscode git auth override
  • Loading branch information
@mafredri
mafredri committed Jan 24, 2023
commit 71706e0e170bfe41c338f91a7e281c961a4ed67e
3 changes: 3 additions & 0 deletions agent/agent.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,9 @@ func (a *agent) run(ctx context.Context) error {
// concurrently with the startup script to avoid conflicts between
// them.
if metadata.GitAuthConfigs > 0 {
// If this fails, we should consider surfacing the error in the
// startup log and setting the lifecycle state to be "start_error"
// (after startup script completion), but for now we'll just log it.
err := gitauth.OverrideVSCodeConfigs(a.filesystem)
if err != nil {
a.logger.Warn(ctx, "failed to override vscode git auth configs", slog.Error(err))
Copy link
Copy Markdown
Member

@mtojek mtojek Jan 24, 2023

Choose a reason for hiding this comment

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

nit: should the agent quit here?

Copy link
Copy Markdown
Member Author

@mafredri mafredri Jan 24, 2023

Choose a reason for hiding this comment

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

We probably shouldn't, this could fail for a multitude of reasons, like a user setting chmod 0000 on a folder. The worst that can happen is a degraded user experience (git auth in vscode not working). Ultimately we might want to:

  • Continue executing startup script after this
  • Set lifecycle to start_error because this failed
  • Result (error message) must be visible in build/startup logs

Copy link
Copy Markdown
Member

@mtojek mtojek Jan 24, 2023

Choose a reason for hiding this comment

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

The worst that can happen is a degraded user experience (git auth in vscode not working).

Yes, that's the situation I was considering. What can user do in this case? Restart the workspace until it works?

Copy link
Copy Markdown
Member Author

@mafredri mafredri Jan 24, 2023

Choose a reason for hiding this comment

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

If the change was done before, failure here doesn't really matter either (btw). Restarting will most likely not help, unless it's a problem mounting FS or similar. They'll most likely need to resolve the issue in the workspace, then restart, or create a new one.

Copy link
Copy Markdown
Member Author

@mafredri mafredri Jan 24, 2023

Choose a reason for hiding this comment

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

For now, I've only added a note. I'll revisit this behavior when I add startup log streaming 👍🏻.

Expand Down