cbornet commented on a change in pull request #14948:
URL: https://github.com/apache/pulsar/pull/14948#discussion_r839437368
##########
File path:
tests/integration/src/test/java/org/apache/pulsar/tests/integration/io/sinks/KinesisSinkTester.java
##########
@@ -64,16 +63,11 @@ public void prepareSink() throws Exception {
final LocalStackContainer localStackContainer = getServiceContainer();
final URI endpointOverride =
localStackContainer.getEndpointOverride(LocalStackContainer.Service.KINESIS);
sinkConfig.put("awsEndpoint", NAME);
- sinkConfig.put("awsEndpointPort", endpointOverride.getPort());
+ sinkConfig.put("awsEndpointPort", LOCALSTACK_SERVICE_PORT);
Review comment:
yes it is.
We were using the port forwarded to the host. But the sink is not on the
host but in a container so it must use the internal container port.
--
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]
