Skip to content

fetchers: add marker at the end of the test - #89

Open
matttbe wants to merge 1 commit into
linux-netdev:mainfrom
matttbe:tester-done
Open

fetchers: add marker at the end of the test#89
matttbe wants to merge 1 commit into
linux-netdev:mainfrom
matttbe:tester-done

Conversation

@matttbe

@matttbe matttbe commented Aug 11, 2026

Copy link
Copy Markdown
Member

This imitates what is done by the poller node: when this marker is written, the uploader service knows it can stop monitoring the current directory. This avoids having too many inotify instances which can cause troubles.

All users of "Fetcher" are using the same pattern: a log message and the creation of the result dir at the beginning, and a log message at the end (except for exec and gh, added here). Now, an empty .tester_done file is created at the end as well.

Note that some of these fetchers don't seem to be used any more.

This imitates what is done by the poller node: when this marker is
written, the uploader service knows it can stop monitoring the current
directory. This avoids having too many inotify instances which can cause
troubles.

All users of "Fetcher" are using the same pattern: a log message and the
creation of the result dir at the beginning, and a log message at the
end (except for exec and gh, added here). Now, an empty '.tester_done'
file is created at the end as well.

Note that some of these fetchers don't seem to be used any more.

Signed-off-by: Matthieu Baerts <matttbe@kernel.org>
@matttbe
matttbe requested a review from kuba-moo August 11, 2026 17:51
@matttbe

matttbe commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

Note that I also modified all log-watcher-* services, except the ones for the pollers (build & build-wireless) to set --timeout-depth to 2 (kunit) or 3 (vmksft & hwksft) to avoid the stale we had last weekend.

After the modifications from this PR, we could reduce the timeouts, e.g. ~3h for the vmksft, 8 for hwksft, and 15min for kunit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant