You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the issue
Fail to install kafka-broker with "confluent_server_enabled=false" and "installation_method=archive". Ansible fails to extract 'confluent-kafka.service' file, as it is not present in the .tar.gz.
To Reproduce
Steps to reproduce the behaviour:
Launch playbook kafka_broker.yml with these two variables
TASK [confluent.platform.kafka_broker : Copy Kafka Broker Service from archive file to system] *******************************************************************************************************************************************************************************************************************************
fatal: [master-02]: FAILED! => changed=false
msg: Source /opt/confluent/confluent-7.5.3/lib/systemd/system/confluent-kafka.service not found
Expected behaviour
Since I'm using 'confluent_server_enabled=false', the kafka broker service name is confluent-kafka, not confluent-server.
kafka_broker_service_name: "{{ 'confluent-server' if confluent_server_enabled|bool else 'confluent-kafka'}}"
This var is then used as the filename for the systemd service file. Except the .tar.gz only packages a confluent-server.service, not a confluent-kafka.service.
I have checked using a package installation, the .rpm install includes such file, but I think the files are identical.
I see two way of fixing it:
Package the confluent kafka .tar.gz with a confluent-kafka.service file (which is identical of the current confluent-server.service.) This fix is to be done by the build team of confluent I guess.
Add some logic in ansible to rename confluent-server.service to confluent-kafka.service when copying from the .tar.gz to the systemd
Thank you. I can submit PR with the latter solution, but I think such a fix will always be ugly, adding some unecessary logic to rename a file that I feel should be included in the archive is really not nice.
Describe the issue
Fail to install kafka-broker with "confluent_server_enabled=false" and "installation_method=archive". Ansible fails to extract 'confluent-kafka.service' file, as it is not present in the .tar.gz.
To Reproduce
Steps to reproduce the behaviour:
kafka_broker.yml
with these two variablescp-ansible/roles/kafka_broker/tasks/main.yml
Lines 105 to 117 in 8254c08
Expected behaviour
Since I'm using 'confluent_server_enabled=false', the kafka broker service name is confluent-kafka, not confluent-server.
cp-ansible/roles/variables/vars/main.yml
Line 306 in 8254c08
This var is then used as the filename for the systemd service file. Except the .tar.gz only packages a
confluent-server.service,
not aconfluent-kafka.service
.https://github.com/confluentinc/cp-ansible/blob/8254c0812e70f4e8801a2e6aa812673cdb6fcd55/roles/variables/vars/main.yml#L316C1-L316C77
I have checked using a package installation, the
.rpm
install includes such file, but I think the files are identical.I see two way of fixing it:
confluent-kafka.service
file (which is identical of the currentconfluent-server.service
.) This fix is to be done by the build team of confluent I guess.confluent-server.service
toconfluent-kafka.service
when copying from the .tar.gz to the systemdThank you. I can submit PR with the latter solution, but I think such a fix will always be ugly, adding some unecessary logic to rename a file that I feel should be included in the archive is really not nice.
Inventory File
Environment (please complete the following information):
Thank you for your work, hope this isn't duplicate.
The text was updated successfully, but these errors were encountered: