[MESOS-9695] Remove the duplicate pid check in Docker containerizer

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.4.4, 1.5.4, 1.6.3, 1.7.3, 1.8.1, 1.9.0
• Component/s: containerization
• Labels: containerization
• Sprint: Containerization: RI13 Sp 45
• Story Points: 2
• Assignee: Qian Zhang
• Reporter: Qian Zhang
• Shepherd: Gilbert Song
• Created: 02/Apr/19 07:02
• Updated: 01/May/19 10:28
• Resolved: 30/Apr/19 12:52

Description

In `DockerContainerizerProcess::_recover`, we check if there are two executors use duplicate pid, and error out if we find duplicate pid (see here for details). However I do not see the value this check can give us but it will cause serious issue (agent crash loop when restarting) in rare case (a new executor reuse pid of an old executor), so I think we'd better to remove it from Docker containerizer.

Issue Links

• relates to: MESOS-9672 Docker containerizer should ignore pids of executors that do not pass the connection check.