akka-management icon indicating copy to clipboard operation
akka-management copied to clipboard

kubernetes - clarify remoting port is informational

Open johanandren opened this issue 5 years ago • 5 comments

@agolubev pointed out that the Akka remoting port is not strictly needed in the yaml (it is more of an informational entry?), we should perhaps clarify that or remove it from the sample in https://doc.akka.io/docs/akka-management/current/bootstrap/kubernetes-api.html

johanandren avatar Mar 25 '20 15:03 johanandren

We also have it defined in the kubernetes test projects https://github.com/akka/akka-kubernetes-tests and the akka-sample-cluster-kubernetes-{java/scala} projects, so we should probably update those to align if we remove it.

johanandren avatar Mar 25 '20 15:03 johanandren

I verified with minikube, someone pointed out it may be different with actual cluster nodes and kubernetes and that the port entry may be reuquired.

johanandren avatar Mar 26 '20 08:03 johanandren

I don't think it's needed, the port is discovered via the probing.

chbatey avatar May 18 '20 06:05 chbatey

I think let's remove it. It could just be inconsistent with configuration and confuse users.

chbatey avatar May 18 '20 06:05 chbatey

Update:

  • https://doc.akka.io/docs/akka-management/current/bootstrap/kubernetes-api.html lists the 2552 port
  • https://doc.akka.io/docs/akka-management/current/kubernetes-deployment/preparing-for-production.html#deployment-spec doesn't list it

ignasi35 avatar Sep 18 '20 10:09 ignasi35