Skip to content

WIP: daemon: add test for order of events on container stop#50227

Draft
thaJeztah wants to merge 1 commit intomoby:masterfrom
thaJeztah:fix_event_ordering_step2
Draft

WIP: daemon: add test for order of events on container stop#50227
thaJeztah wants to merge 1 commit intomoby:masterfrom
thaJeztah:fix_event_ordering_step2

Conversation

@thaJeztah
Copy link
Member

This needs more work; while the test itself is OK, the logic for producing these events and to update state has various issues, as outlined in the linked PR.

It looks like there's just too much logic stacked on top of existing bits over the Years to fix various issues, and it's starting to show. We need to take a few steps back and make an inventory of the logic and rework some of it at least.

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the fix_event_ordering_step2 branch from 0699283 to c4af37c Compare June 23, 2025 11:16
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