@hangc0276 Can you move the changes under integrations/python2-kafka-python/* a separate pull request? So we can get the dockerfile change merged and a new image can be pushed to docker hub. Then your integration test can use it.

@hangc0276 Can you move the changes under integrations/python2-kafka-python/* a separate pull request? So we can get the docerfile change merged and a new image can be pushed to docker hub. Then your integration test can use it.

@sijie Thanks for your reply. I have changed kafka-python test case dockerfile to integrations/python2-kafka-python/ and all the versions of kafka-python will use the same docker image to run test cases.

@hangc0276 correct. the issue failed with the following error Caused by: com.github.dockerjava.api.exception.NotFoundException: {"message":"pull access denied for streamnative/kop-test-python2-kafka-python, repository does not exist or may require 'docker login': denied: requested access to the resource is denied"}

It means we didn't have the image yet. My suggestion is to create a separate pull request that only includes the docker change. So we can merge that pull request and get the docker image published to streamnative/kop-test-python2-kafka-python. So the integration tests in this pull request can use that image.

@hangc0276 correct. the issue failed with the following error Caused by: com.github.dockerjava.api.exception.NotFoundException: {"message":"pull access denied for streamnative/kop-test-python2-kafka-python, repository does not exist or may require 'docker login': denied: requested access to the resource is denied"}

It means we didn't have the image yet. My suggestion is to create a separate pull request that only includes the docker change. So we can merge that pull request and get the docker image published to streamnative/kop-test-python2-kafka-python. So the integration tests in this pull request can use that image.

@sijie OK, i will use a separate PR to build the docker image.

@hangc0276 any updates on this pull request?

@hangc0276 any updates on this pull request?

Yes, I will update the docker image soon.

@sijie I have update the kafka-python and pykafka integration test framework. Some versions are not supported, and I have exclude them in the integration test.

synced with @hangc0276 , Test failure caused by the docker image, maybe we should fix the image first: