Feature/multiple acl operations

[saiello]

Fixes #70 Fixes #124

Proposed Changes

• Allow to specify multiple acl_operations

 - name: "Create a single ACL with multiple operations"
      kafka_acl:
        name: 'my-topic'
        api_version: "2.6.0"
        acl_resource_type: 'topic'
        acl_principal: 'User:producer-client'
        acl_operations: 
          - 'write'
          - 'describe'
          - 'create'
        acl_permission: 'allow'
        acl_pattern_type: 'literal'
        bootstrap_servers: "localhost:9092"

- name: "Create multiple ACL with multiple operations"
      kafka_acls:

        acls:
          - name: 'my-topic'
            acl_resource_type: 'topic'
            acl_principal: 'User:consumer-client'
            acl_operations: 
              - 'describe'
              - 'read'
            acl_permission: 'allow'
            acl_pattern_type: 'literal'

          - name: 'my-consumer-group'
            acl_resource_type: 'group'
            acl_principal: 'User:consumer-client'
            acl_operations: 
              - 'read'
            acl_permission: 'allow'
            acl_pattern_type: 'literal'

        api_version: "2.6.0"
        bootstrap_servers: "localhost:9092"

• Allow to use state=absent without acl_operation/acl_operations

    - name: "Delete any ACL of a given resource"
      kafka_acls:

        acls:
          - name: 'my-topic'
            acl_principal: 'User:consumer-client'
            state: 'absent'

        bootstrap_servers: "localhost:9092"

[ryarnyah]

We need to look more to this PR but it's seem be to be a great improvement! Thanks!!!!

[ryarnyah]

LGMT! Can you add some test with acl_operations in https://github.com/StephenSorriaux/ansible-kafka-admin/blob/master/molecule/default/tests/test_acl_default.py to ensure that this won't be breaked in the future?

[saiello]

@ryarnyah I mean to rebase on master once the #134 will be merged to use the same testing approach used there.

[ryarnyah]

@saiello In this case it might be a better idea to reuse default scenario to reuse existing tests (ACLs is supported from 0.11.0 to X and in your case it's only to ensure that it won't be breaked in the future). Furthermore we are already limited on concurrent github action that can be done so if we add another scenario we will increase drastically build time.

[saiello]

@saiello In this case it might be a better idea to reuse default scenario to reuse existing tests (ACLs is supported from 0.11.0 to X and in your case it's only to ensure that it won't be breaked in the future). Furthermore we are already limited on concurrent github action that can be done so if we add another scenario we will increase drastically build time.

Yes, you are right! I will add further cases to the existing tests!

I have some idea to optimize the github actions usage. I would like to work on them later. Let me know if you already have some thought about it.

[saiello]

@ryarnyah tests added

[ryarnyah]

@saiello thank you!