Skip to content

Move container exit state to after cleanup.#41698

Merged
thaJeztah merged 1 commit into
moby:masterfrom
cpuguy83:fix_shutdown_handling
Feb 11, 2021
Merged

Move container exit state to after cleanup.#41698
thaJeztah merged 1 commit into
moby:masterfrom
cpuguy83:fix_shutdown_handling

Conversation

@cpuguy83

Copy link
Copy Markdown
Member

Before this change, there is no way to know if container (runtime)
resources have been cleaned up unless you actually remove the container.

This change allows callers of the wait API or the events API to know
that all runtime resources for the container are released (e.g. IP
addresses).

@thaJeztah thaJeztah added this to the 20.10.1 milestone Nov 19, 2020
@thaJeztah thaJeztah changed the title Move container exit state to after cleanup. [on hold] Move container exit state to after cleanup. Nov 19, 2020
@thaJeztah

Copy link
Copy Markdown
Member

discussing in the maintainers meeting; putting this on hold for directly after 20.10.0, to give it some time for testing

@thaJeztah thaJeztah modified the milestones: 20.10.1, 20.10.2 Dec 15, 2020
@thaJeztah thaJeztah modified the milestones: 20.10.2, 20.10.3 Jan 5, 2021
@thaJeztah thaJeztah changed the title [on hold] Move container exit state to after cleanup. Move container exit state to after cleanup. Jan 26, 2021
Before this change, there is no way to know if container (runtime)
resources have been cleaned up unless you actually remove the container.

This change allows callers of the wait API or the events API to know
that all runtime resources for the container are released (e.g. IP
addresses).

Signed-off-by: Brian Goff <[email protected]>
@tianon
tianon force-pushed the fix_shutdown_handling branch from c91fdd5 to e192ce4 Compare January 28, 2021 19:29

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@tianon tianon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 🙈

@thaJeztah

Copy link
Copy Markdown
Member

Failures are unrelated

@thaJeztah
thaJeztah merged commit 2643534 into moby:master Feb 11, 2021
@thaJeztah

Copy link
Copy Markdown
Member

Discussing in the maintainers meeting, and we'll keep this one for 21.xx, so removing the cherry-pick label for now

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants