Skip to content

feat(agent/agentcontainers): add file watcher and dirty status #17573

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 23 commits into from
Apr 29, 2025
Merged

feat(agent/agentcontainers): add file watcher and dirty status #17573

Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
40c8de9
feat(agent/agentcontainers): add file watcher and dirty status
mafredri Apr 28, 2025
a997f82
make gen
mafredri Apr 28, 2025
0e568fb
fix copilot nits
mafredri Apr 28, 2025
4f788d9
log event string in case not write
mafredri Apr 28, 2025
bfeb1da
fix chmod on macos
mafredri Apr 28, 2025
45a978c
fix close of closed channel
mafredri Apr 28, 2025
d2721a7
add close protection to fsnotify watcher as well
mafredri Apr 28, 2025
c03bb7d
fix typo
mafredri Apr 28, 2025
aed7c30
fix pr suggestions and improve start loop
mafredri Apr 28, 2025
39526e6
update noop impl.
mafredri Apr 28, 2025
7edee90
s/file/path
mafredri Apr 28, 2025
9cf5415
rewrite watcher impl. to watch dirs
mafredri Apr 28, 2025
de2a42e
add rename and create to fsnotify test
mafredri Apr 28, 2025
0cbee38
add atomic file replacement
mafredri Apr 28, 2025
5ced7e1
add paranoid reset
mafredri Apr 28, 2025
5a96a55
fix oopsie
mafredri Apr 28, 2025
0b16448
s/ErrWatcherClosed/ErrClosed/g
mafredri Apr 28, 2025
eaf5922
inverse continue/return for clarity
mafredri Apr 28, 2025
617fed2
better close handling
mafredri Apr 28, 2025
8ee9914
restore removed comment
mafredri Apr 28, 2025
263c683
add note about name
mafredri Apr 28, 2025
58a70e8
fix test setup/clock advancement
mafredri Apr 28, 2025
9f739ca
outdent
mafredri Apr 28, 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
Next Next commit
fix test setup/clock advancement
  • Loading branch information
@mafredri
mafredri committed Apr 28, 2025
commit 58a70e8d2f9d0e637f59359e85d763241bd591fb
7 changes: 6 additions & 1 deletion agent/agentcontainers/api_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -652,13 +652,18 @@ func TestAPI(t *testing.T) {
assert.Contains(t, fWatcher.addedPaths, configPath,
"watcher should be watching the container's config file")

// Make sure the start loop has been called.
fWatcher.waitNext(ctx)

// Send a file modification event and check if the container is
// marked dirty.
fWatcher.sendEventWaitNextCalled(ctx, fsnotify.Event{
Name: configPath,
Op: fsnotify.Write,
})

mClock.Advance(time.Minute).MustWait(ctx)

// Check if the container is marked as dirty.
req = httptest.NewRequest(http.MethodGet, "/devcontainers", nil)
rec = httptest.NewRecorder()
Expand All @@ -671,7 +676,7 @@ func TestAPI(t *testing.T) {
assert.True(t, response.Devcontainers[0].Dirty,
"container should be marked as dirty after config file was modified")

mClock.Advance(10 * time.Minute).MustWait(ctx)
mClock.Advance(time.Minute).MustWait(ctx)

container.ID = "new-container-id" // Simulate a new container ID after recreation.
container.FriendlyName = "new-container-name"
Expand Down
Loading