Skip to content

Possible flake fix#1508

Draft
jglick wants to merge 1 commit intojenkinsci:masterfrom
jglick:flake
Draft

Possible flake fix#1508
jglick wants to merge 1 commit intojenkinsci:masterfrom
jglick:flake

Conversation

@jglick
Copy link
Copy Markdown
Member

@jglick jglick commented Mar 31, 2026

This flake from #1504 seems to be a test problem

Z:\agent\workspace\hub-branch-source-plugin_PR-1504\target\tmp\j h6248222677314807097\jobs\test0\indexing\indexing.log: The process cannot access the file because it is being used by another process
  64.126 [id=28]	WARNING	o.j.h.t.RemainingActivityListener#onTearDown: BranchIndexing[test0] still seems to be running, which could break deletion of log files or metadata

The patch addresses the warning at least, so may fix the flake as well.

@jglick jglick requested a review from a team as a code owner March 31, 2026 22:29
@jglick jglick added the tests label Mar 31, 2026
@jglick
Copy link
Copy Markdown
Member Author

jglick commented Apr 1, 2026

@jglick jglick marked this pull request as draft April 1, 2026 17:38
@jtnord
Copy link
Copy Markdown
Member

jtnord commented Apr 2, 2026

if it helps at all, if the log uses RewindableFileOutputStream or its rotating variant, there are issues as the logs are not really closed on close and can be magically reopened after closure by an arbitrary write. Same happened with agent logs.
So the indexing thread needs to be stopped before the log is closed.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants