search

confluentinc / cp-ansible

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

ansible-container #242

Closed OneCricketeer closed 4 years ago

OneCricketeer commented 4 years ago

Now that these Ansible playbooks are production supported, perhaps they should be used to build the Docker images rather than a combo of bash scripts?

https://docs.ansible.com/ansible-container/

Common:

Zookeeper, Kafka, Connect

Kafkacat

ksqlDB

C3

REST Proxy

Schema Registry

JumaX commented 4 years ago

@cricket007 thanks for the request. I will file an internal feature request to have this reviewed/investigated for viability. Thanks again for this.

OneCricketeer commented 4 years ago

Alternative - https://github.com/ansible-community/ansible-bender

domenicbove commented 4 years ago

There is an entire Jenkins build process for those docker images. And from what I can tell its is built/well rounded. What is the value of redoing it with ansible?

OneCricketeer commented 4 years ago

There is an entire Jenkins build process for those docker images

Not sure I see why that is an issue.

it is built/well rounded

That's good.

What is the value of redoing it with ansible?

Probably none. Other than it creates a single place for maintaining how the platform is installed in individual pieces, as I said initially, rather than bash commands.

OneCricketeer commented 4 years ago

Hmm. This is deprecated - https://github.com/ansible/ansible-container/tree/master

That is mainly why I put the alternative option

JumaX commented 4 years ago

@cricket007 Thanks again for the feedback. Understood about the deprecation. I've filed an internal JIRA for us to research this a bit in the future to see what makes sense. It's not a guarantee we will change it, but by logging it, we can allocate some time to figure out what makes sense here. CC: @domenicbove

JumaX commented 4 years ago

@OneCricketeer Since the original tool is deprecated, and this is a bit further down our "to do" list, I am going to go ahead and close out this issue for now. Thanks again for the suggestions, please keep them coming.