aicam opened a new pull request, #5629:
URL: https://github.com/apache/texera/pull/5629
## Purpose
`KubernetesClient.generatePodURI` builds the in-cluster address for a
computing unit pod. That address is stored as the computing unit's `uri` (via
`setUri` in `ComputingUnitManagingResource`) and returned to clients as
`nodeAddresses`.
The computing unit container listens on
`KubernetesConfig.computeUnitPortNumber` (declared with
`withContainerPort(KubernetesConfig.computeUnitPortNumber)` in the same file),
but the generated URI omitted the port. As a result the persisted address
resolved to the host without the port the computing unit actually serves on, so
it was not directly connectable.
## Summary of changes
Append the configured port to the generated pod URI:
```scala
s"...svc.cluster.local:${KubernetesConfig.computeUnitPortNumber}"
```
This makes the stored URI consistent with the port the pod's container is
exposed on. Single-line change, no behavioral change beyond producing a
complete, connectable address.
--
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]
