confluentinc / cp-ansible

Ansible playbooks for the Confluent Platform
Apache License 2.0
43 stars 405 forks source link

[ANSIENG-3838] | use python3 in kafka_connectors.py (#19) #1770

Closed mansi-jain-1206 closed 2 months ago

mansi-jain-1206 commented 2 months ago

Description

This PR adds the capability to run connector module with python 3 instead of python 2

Fixes # (issue)

Type of change

How Has This Been Tested?

Semaphore Job Link

Checklist:

rrbadiani commented 2 months ago

I think we should test this for all the scenarios which sets up connectors. This would include scenarios like archive-plain-rhel-fips, broker-scale-up, connect-scale-up, kerberos-rhel, ksql-scale-up, multi-ksql-connect-rhel etc

mansi-jain-1206 commented 2 months ago

I think we should test this for all the scenarios which sets up connectors. This would include scenarios like archive-plain-rhel-fips, broker-scale-up, connect-scale-up, kerberos-rhel, ksql-scale-up, multi-ksql-connect-rhel etc

The pipeline link has all tests triggered.

rrbadiani commented 2 months ago

I think we should test this for all the scenarios which sets up connectors. This would include scenarios like archive-plain-rhel-fips, broker-scale-up, connect-scale-up, kerberos-rhel, ksql-scale-up, multi-ksql-connect-rhel etc

The pipeline link has all tests triggered.

The link was pointing to the retry failed scenarios section there I saw only 4 of the tests. On checking the complete run I found that all the tests were ran 😅.