Roiocam commented on code in PR #106:
URL: https://github.com/apache/pekko-samples/pull/106#discussion_r1590472128
##########
pekko-sample-cluster-kubernetes-java/src/main/resources/application.conf:
##########
@@ -33,4 +33,9 @@ pekko.management {
example-ready = org.apache.pekko.cluster.bootstrap.demo.DemoHealthCheck
}
}
+ # Add these so the pods won't get killed.
+ http {
+ port = 8558
Review Comment:
You should set it to bind-port instead of port.
https://github.com/apache/pekko-management/blob/1b7a52ba277daec3130baee4bf29a2f0f55a51bb/management/src/main/resources/reference.conf#L33
If the user does not want to change, they just need to set it to
pekko-management port in the k8s health check.(is 7626 not 8558)
https://github.com/apache/pekko-management/blob/1b7a52ba277daec3130baee4bf29a2f0f55a51bb/management/src/main/resources/reference.conf#L19
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
