Port Mapping not work when using external compose file

[chanjarster]

Description

When using compose file, dynamic port selection doesn't work anymore. Furthermore, the port mapping properties disappears too:

<images>
  <image>
    <name>tomcat:8.5-alpine</name>
    <alias>tomcat</alias>
    <external>
      <type>compose</type>
      <basedir>${project.basedir}</basedir>
    </external>
    <run>
      <ports>
        <port>tomcat.port:8080</port>
      </ports>
      <wait>
        <log>Server startup in [0-9.]+ ms</log>
        <time>120000</time>
      </wait>
    </run>
  </image>
</images>

Property tomcat.port is empty.

I have created a sample project at https://github.com/chanjarster/dmp-bug/tree/master/portmapping-compose-not-work

Info

• d-m-p version :
• Maven version (mvn -v) :

  Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; 2018-10-25T02:41:47+08:00)
  Maven home: /usr/local/Cellar/maven/3.6.0/libexec
  Java version: 1.8.0_181, vendor: Oracle Corporation, runtime: /Library/Java/JavaVirtualMachines/jdk1.8.0_181.jdk/Contents/Home/jre
  Default locale: zh_CN, platform encoding: UTF-8
  OS name: "mac os x", version: "10.14.5", arch: "x86_64", family: "mac"

• Docker version : 18.09.2
• Sample project : https://github.com/chanjarster/dmp-bug/tree/master/portmapping-compose-not-work

[rhuss]

Sounds like a legit bug. Does it worked already for you at some version ? And if so, at which version ? (sorry for the delayed response btw)

[chanjarster]

I use d-m-p of version 0.30.0. It doesn't work.

[rhuss]

My question is, did it ever worked for you because you are writing

When using compose file, dynamic port selection doesn't work anymore.

And if so, in which version did it work for you ?

[chanjarster]

For version 0.30.0, it works if I don't use compose file, it never worked if I use compose file. I didn't try other versions.

[rhuss]

Ok, so that might then be a limitation of the docker-compose support (but no regression). Looks like a bug, but unfortunately, I personally can't work on this. However, if someone wants to pick that up, happy to help to integrate a PR for this. But it's probably not trivial as that kind of property replacement is 'special' and dmp and would need to be added to the compose support.