akka-management
akka-management copied to clipboard
kubernetes - clarify remoting port is informational
@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
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.
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.
I don't think it's needed, the port is discovered via the probing.
I think let's remove it. It could just be inconsistent with configuration and confuse users.
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