[MESOS-1724] Can't include port in DockerInfo's image

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.20.1
• Component/s: containerization
• Labels: docker
• Assignee: Timothy Chen
• Reporter: Jay Buffington
• Created: 20/Aug/14 03:26
• Updated: 17/Sep/14 01:22
• Resolved: 27/Aug/14 19:25

Description

The current git tree doesn't allow you to specify a docker image with multiple colons. It is valid that multiple colons would exist in a docker image. e.g. docker-registry.example.com:80/centos:6u5

From https://github.com/apache/mesos/blob/02a35ab213fb074f6c532075cada76f13eb9d552/src/slave/containerizer/docker.cpp#L441

  vector<string> parts = strings::split(dockerInfo.image(), ":");

  if (parts.size() > 2) {
    return Failure("Not expecting multiple ':' in image: " +
                   dockerInfo.image());
  }

Issue Links

• duplicates: MESOS-1747 Docker image parsing for private repositories