search

akka / akka-management

Akka Management is a suite of tools for operating Akka Clusters.
https://doc.akka.io/docs/akka-management/
Other
254 stars 160 forks source link

Marathon Api port name #370

Open ignacio-dc opened 6 years ago

ignacio-dc commented 6 years ago

The MarathonApiSimpleServiceDiscovery doesn't take the port name from the Lookup, it only takes it from the configuration. I fixed it with the same method that the kubernetes service uses

val portName = lookup.portName match {
      case Some(name) => name
      case None => settings.appPortName
    }

I could make the PR if you'd like

chbatey commented 6 years ago

Yep a PR would be great