container linking updates and fixes

johnmccabe commented 8 years ago

Requires #272 - I'll update the pull when that gets merged.

See individual commits for details (last 3).

@grkvlt can you look at https://github.com/brooklyncentral/clocker/commit/f4afc1b14894dda6957bf820cde38f4613d7f41b in particular, not sure if there will be knock ons from this elsewhere.

You can test this by deploying the following yaml to your docker infrastructure:

location: mydockercluster
services:
- type: docker:mariadb:5.5
  id: db
  openPorts:
  - 3306
  env:
    MYSQL_ROOT_PASSWORD: "password"
- type: docker:wordpress:4.5
  links:
    mysql: $brooklyn:component("db")
  openPorts:
  - 80

johnmccabe commented 8 years ago

fyi @aledsage

johnmccabe commented 8 years ago

This closes #277.

johnmccabe commented 8 years ago

I sanity checked the deployment of a Mesos cluster and app consisting of db/webapp/balancer, didn't observe any issues.

grkvlt commented 8 years ago

Closing, as incorporated in #272

brooklyncentral / clocker

container linking updates and fixes #283