- Remove unused localhost wrapper functions that were never called:
- isPortOpen() - wrapper around isPortOpenOnIP with hardcoded 127.0.0.1
- findAvailablePort() - wrapper around findAvailablePortOnIP with hardcoded 127.0.0.1
- ensurePortAvailable() - wrapper around ensurePortAvailableOnIP with hardcoded 127.0.0.1
- ensureAllPortsAvailable() - wrapper around ensureAllPortsAvailableOnIP with hardcoded 127.0.0.1
Since this is new functionality with no backwards compatibility concerns,
these wrapper functions were not needed. The comments claiming they were
'kept for future use or backwards compatibility' are no longer valid.
- Update documentation to reference GrpcPortOffset constant instead of hardcoded 10000:
- Update comment in ensureAllPortsAvailableOnIP to use GrpcPortOffset
- Update admin.port.grpc flag help text to reference GrpcPortOffset
Note: getBindIp() is actually being used and should be retained (contrary to
the review comment suggesting it was unused - it's called in both runMini
and startMiniServices functions)
Chris Lu
85d90e430e