Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Can't specify discovery info for Docker User Network #4054

Closed
ozdanborne opened this issue Jun 30, 2016 · 5 comments
Closed

Can't specify discovery info for Docker User Network #4054

ozdanborne opened this issue Jun 30, 2016 · 5 comments
Assignees
Milestone

Comments

@ozdanborne
Copy link

Trying to launch the following application JSON yields the error: /ipAddress: ipAddress/discovery is not allowed for Docker containers using BRIDGE or USER networks:

{
  "id": "test-1",
  "cpus": 0.1,
  "mem": 64,
  "instances": 1,
  "container": {
    "docker": {
      "image": "nginx",
      "network": "USER"
    },
    "type": "DOCKER"
  },
  "ipAddress": {
      "networkName": "test",
      "discovery": {
          "ports": [
            { "number": 80, "name": "http", "protocol": "tcp" }
          ]
      }
  },
  "healthChecks": [
    {
      "protocol": "HTTP",
      "path": "/",
      "port": 80,
      "gracePeriodSeconds": 300,
      "intervalSeconds": 60,
      "timeoutSeconds": 20,
      "maxConsecutiveFailures": 3
    }
  ]
}

Why is discovery info blocked for Docker User networks?

@gkleiman gkleiman modified the milestones: 1.3, 1.2 Jul 4, 2016
@jdef
Copy link
Contributor

jdef commented Jul 5, 2016

should be possible to specify discovery information via portMappings; can you try that instead?

@unterstein unterstein modified the milestones: 1.2, 1.3 Aug 24, 2016
@aquamatthias
Copy link
Contributor

@jdef any update here?

@ozdanborne
Copy link
Author

It seems like specifiying port information in PortMappings instead of DiscoveryInfo does allow the necessary information to be stored in Marathon.

However, now that the location has changed, Marathon-LB is failing to balance to these tasks, since it checks DiscoveryInfo for ip-per-container tasks.

I would have liked to get Marathon-LB working before closing this issue...

@ozdanborne
Copy link
Author

@brndnmtthws adjusted Marathon-LB to expect the information in the new location. All is working!

@aquamatthias
Copy link
Contributor

@djosborne great to hear!

@d2iq-archive d2iq-archive locked and limited conversation to collaborators Mar 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants