[
https://issues.apache.org/jira/browse/MESOS-9185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16595084#comment-16595084
]
Alexander Rukletsov edited comment on MESOS-9185 at 8/28/18 4:11 PM:
---------------------------------------------------------------------
*1.8.0-dev:*
{noformat}
commit 8496b369d52d27e90da88787242fd6f9d9abb78e
Author: Andrei Budnik <[email protected]>
AuthorDate: Tue Aug 28 16:46:54 2018 +0200
Commit: Alexander Rukletsov <[email protected]>
CommitDate: Tue Aug 28 16:46:54 2018 +0200
Added `AgentAPITest.LaunchNestedContainerWithUnknownParent` test.
This test verifies that launch nested container fails when the parent
container is unknown to the containerizer.
Review: https://reviews.apache.org/r/68234/
{noformat}
{noformat}
commit 5fbfb8da5ad62c40752fa7b7e0a0842c892f6857
Author: Andrei Budnik <[email protected]>
AuthorDate: Tue Aug 28 16:47:04 2018 +0200
Commit: Alexander Rukletsov <[email protected]>
CommitDate: Tue Aug 28 16:47:04 2018 +0200
Cleaned up container on launch failures in composing containerizer.
Previously, if a parent container was unknown to the composing
containerizer during an attempt to launch a nested container
via `ComposingContainerizerProcess::launch()`, the composing
containerizer returned an error without cleaning up the container.
The `containerizer` field was uninitialized, so a further attempt
to remove or destroy the nested container led to segfault.
This patch removes the container when the parent container is unknown.
Review: https://reviews.apache.org/r/68235/
{noformat}
*backport to 1.7.1:*
{noformat}
commit 1660a0552e58ba4407180508f7e4eeed2050b2a2
Author: Andrei Budnik <[email protected]>
AuthorDate: Tue Aug 28 16:47:04 2018 +0200
Commit: Alexander Rukletsov <[email protected]>
CommitDate: Tue Aug 28 18:07:44 2018 +0200
Cleaned up container on launch failures in composing containerizer.
Previously, if a parent container was unknown to the composing
containerizer during an attempt to launch a nested container
via `ComposingContainerizerProcess::launch()`, the composing
containerizer returned an error without cleaning up the container.
The `containerizer` field was uninitialized, so a further attempt
to remove or destroy the nested container led to segfault.
This patch removes the container when the parent container is unknown.
Review: https://reviews.apache.org/r/68235/
(cherry picked from commit 5fbfb8da5ad62c40752fa7b7e0a0842c892f6857)
{noformat}
was (Author: alexr):
{noformat}
commit 8496b369d52d27e90da88787242fd6f9d9abb78e
Author: Andrei Budnik <[email protected]>
AuthorDate: Tue Aug 28 16:46:54 2018 +0200
Commit: Alexander Rukletsov <[email protected]>
CommitDate: Tue Aug 28 16:46:54 2018 +0200
Added `AgentAPITest.LaunchNestedContainerWithUnknownParent` test.
This test verifies that launch nested container fails when the parent
container is unknown to the containerizer.
Review: https://reviews.apache.org/r/68234/
{noformat}
{noformat}
commit 5fbfb8da5ad62c40752fa7b7e0a0842c892f6857
Author: Andrei Budnik <[email protected]>
AuthorDate: Tue Aug 28 16:47:04 2018 +0200
Commit: Alexander Rukletsov <[email protected]>
CommitDate: Tue Aug 28 16:47:04 2018 +0200
Cleaned up container on launch failures in composing containerizer.
Previously, if a parent container was unknown to the composing
containerizer during an attempt to launch a nested container
via `ComposingContainerizerProcess::launch()`, the composing
containerizer returned an error without cleaning up the container.
The `containerizer` field was uninitialized, so a further attempt
to remove or destroy the nested container led to segfault.
This patch removes the container when the parent container is unknown.
Review: https://reviews.apache.org/r/68235/
{noformat}
> An attempt to remove or destroy container in composing containerizer leads to
> segfault.
> ---------------------------------------------------------------------------------------
>
> Key: MESOS-9185
> URL: https://issues.apache.org/jira/browse/MESOS-9185
> Project: Mesos
> Issue Type: Bug
> Components: agent, containerization
> Affects Versions: 1.7.0
> Reporter: Andrei Budnik
> Assignee: Andrei Budnik
> Priority: Major
> Labels: mesosphere
> Fix For: 1.8.0
>
>
> `LAUNCH_NESTED_CONTAINER` and `LAUNCH_NESTED_CONTAINER_SESSION` leads to
> segfault in the agent when the parent container is unknown to the composing
> containerizer. If the parent container cannot be found during an attempt to
> launch a nested container via `ComposingContainerizerProcess::launch()`, the
> composing container returns an error without cleaning up the container. On
> `launch()` failures, the agent calls `destroy()` which accesses uninitialized
> `containerizer` field.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)