Skip to content

Conversation

thjaeckle
Copy link
Member

@thjaeckle thjaeckle commented Oct 20, 2025

  • made local ask timeout for sending messages to "Enforcer" actor differently configurable than the local ask-timeout for interacting with persistence actor
  • retry things aggregation after searches on RemoteStreamRefActorTerminatedException
    • which are thrown during rolling updates / restarts of things
  • introduce "askWithRetry" pattern to signal enrichment in connectivity in order to stabilise retrieving "extra" fields when timeouts occur

@thjaeckle thjaeckle added this to the 3.8.3 milestone Oct 20, 2025
@thjaeckle thjaeckle added the bug label Oct 20, 2025
…atedException

* which are thrown during rolling updates / restarts of things
@thjaeckle thjaeckle force-pushed the bugfix/restart-stabilizations branch from 95d1549 to eb3ca35 Compare October 21, 2025 05:39
@thjaeckle thjaeckle marked this pull request as ready for review October 21, 2025 07:12
@thjaeckle
Copy link
Member Author

@alstanchev could you have a look please?
System tests passed - except for the one which often fails (also want to tackle this soon): https://github.com/eclipse-ditto/ditto/actions/runs/18674882919

We encountered several ERRORs during e.g. a rolling update of "things" which should be preventable (by retrying asks, by restarting remote streams if they were canceled, by updating ask timeout to "enforcer" actor).

@alstanchev
Copy link
Contributor

alstanchev commented Oct 21, 2025

Sure, already skimmed over it but didn't give it much time since the draft status. I will go over it again now.

Copy link
Contributor

@alstanchev alstanchev left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@thjaeckle thjaeckle merged commit 7567896 into eclipse-ditto:master Oct 21, 2025
3 checks passed
@thjaeckle thjaeckle self-assigned this Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants