Skip to content

Issue #85#65

Merged
ClProsser merged 32 commits intomainfrom
Issue-#85
Jul 15, 2025
Merged

Issue #85#65
ClProsser merged 32 commits intomainfrom
Issue-#85

Conversation

@ashiven
Copy link

@ashiven ashiven commented Jul 7, 2025

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature

What is the current behavior? (You can also link to an open issue here)
Currently, a worker is marked as unreachable the moment a connection to it fails

What is the new behavior (if this is a feature change)? If possible add a screenshot.
Now, there is a set timeout that can be configured until a worker will be marked as unreachable and any analysis still running on the worker will be reassigned

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No

ashiven added 8 commits July 7, 2025 12:05
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
…s raised by exec_blocking_ssh when client is not connected

Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
@ashiven ashiven self-assigned this Jul 7, 2025
@ashiven ashiven requested a review from ClProsser July 7, 2025 12:33
ashiven added 20 commits July 7, 2025 15:04
…all connection attempts failed

Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
… a worker

Signed-off-by: ashiven <nevisha@pm.me>
…ods for interacting with dependency states

Signed-off-by: ashiven <nevisha@pm.me>
…pendencies

Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
Signed-off-by: ashiven <nevisha@pm.me>
…many debug messages)

Signed-off-by: ashiven <nevisha@pm.me>
@ashiven
Copy link
Author

ashiven commented Jul 14, 2025

I finished testing this issue and the analysis reassignments etc. worked for me.

Signed-off-by: ClProsser <clemens.prosser@gmail.com>
Signed-off-by: ClProsser <clemens.prosser@gmail.com>
Signed-off-by: ClProsser <clemens.prosser@gmail.com>
Copy link

@ClProsser ClProsser left a comment

Choose a reason for hiding this comment

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

Good job @ashiven !

@ClProsser ClProsser merged commit 23b47b3 into main Jul 15, 2025
14 checks passed
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.

2 participants