Sensor docker.port.XX not being mapped

brooklyncentral / clocker

Apache Brooklyn cloud native infrastructure blueprints

Apache License 2.0

426 stars 66 forks source link

Sensor docker.port.XX not being mapped #280

Closed johnmccabe closed 8 years ago

johnmccabe commented 8 years ago

Adding an app using the following YAML successfully deploys but the docker.port.3306 sensor does have a corresponding mapped.docker.port.3306 added (its absent rather than empty), looks like the hostAndPortTransformingEnricher isn't being added.

location: dc
services:
- type: docker:mariadb:5.5
  id: db
  containerName: mysql
  env:
    MYSQL_ROOT_PASSWORD: "password"

The container does get created correctly, and the mapped port is there and accessible just not published in a sensor.

grkvlt commented 8 years ago

Trying some changes in f56675f07ffa727bbab1e55b7a95ada11b6cb537 that may fix this

johnmccabe commented 8 years ago

Thanks @grkvlt this looks good now.

johnmccabe commented 8 years ago

Closing as fixed in #272