Uploaded image for project: 'Marathon'
  1. Marathon
  2. MARATHON-7204

Bridge Network portMapping support not implemented for Pods

    Details

    • Type: Task
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: Marathon 1.5.0
    • Component/s: None
    • Labels:
    • Sprint:
      Marathon Sprint 1.10-4, Marathon Sprint 1.10-5
    • Story Points:
      3
    • Parent Feature:
      DCOS-28607 - [Marathon] Improve Marathon functionality & quality

      Description

      This was left out from the great D201 network refactoring patch and was noticed during the review of https://phabricator.mesosphere.com/D672

      Acceptance criteria:

      • We should be able to specify host -> port mapping for pods with bridge networking
        • (should we support hostPort undefined for bridge networking? This seems to make no sense)
      • Appropriate validation should be applied for pods with bridge networking, indicating that both hostPort and containerPort are required. (Add a test)
      • TaskGroupBuilder should generate the network appropriate NetworkInfo in buildMesosNetworks.
        • With test
        • Note that we need to zip assigned hostPorts, as is done for container networks; make generic if possible.
      • Container serializer should be able to round-trip serialize a bridge-networked pod with port forwardings; add tests as appropriate.
      • Launching a pod with bridge networking from the v2 Marathon API should invoke the proper CNI commands on taskGroup launch (at least, use the stub CNI plugin and post the results of what CNI activity is happening, here)

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                junterstein Johannes Unterstein
                Reporter:
                tharper Tim Harper
                Team:
                Orchestration Team
                Watchers:
                Aleksey Dukhovniy, Ivan Chernetsky, James DeFelice, Johannes Unterstein, Marco Monaco, Matthias Eichstedt, Orlando Hohmeier, Tim Harper, Tobias Mueller
              • Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: