Support to Add, Remove and List ACLs (Fixes #6)

[jograca]

Description of your changes

This Pull Request adds feature complete ACL support to Add, Remove and List ACLs.
Fixes #6

I have:

• [ x ] Read and followed Crossplane's contribution process.
• [ x ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Automated tests are forthcoming

1. Created an ACL by applying examples/acl/acl.yaml
2. Verified by running kcl admin acl describe --type any --pattern match --op any --perm any
3. Updated ACL by changing a Principle in examples/acl/acl.yml
4. Applied the ACL and noticed the Update threw an error
5. Deleted ACL by deleting examples/acl/acl.yaml
6. Verified by same process as bullet 2

[stevendborrelli]

Overall it looks good!

Need to run make reviewable test, as there are some linting errors.

apis/acl/v1alpha1/acl_types.go:31: File is not `gofmt`-ed with `-s` (gofmt)
        ResourceName    string `json:"resourceName"`
internal/clients/kafka/acl/acl.go:19: File is not `gofmt`-ed with `-s` (gofmt)
        ResourceName                      string
internal/controller/acl/acl.go:194: File is not `gofmt`-ed with `-s` (gofmt)
        return managed.ExternalCreation{}, acl.Create(ctx, c.kafkaClient, generated)
internal/clients/kafka/topic/topic_test.go:6: File is not `goimports`-ed with -local github.com/crossplane-contrib/provider-kafka (goimports)
        "github.com/crossplane-contrib/provider-kafka/apis/topic/v1alpha1"
internal/controller/acl/acl.go:146:11: ineffectual assignment to `err` (ineffassign)
        extname, err := acl.ConvertFromJSON(meta.GetExternalName(cr))

[stevendborrelli]

I've approved it, but there is one linting error that comes up. (See my comment).

[stevendborrelli]

Linting is coming up with one more issue cyclomatic complexity 11 of func List is high (> 10)

Cyclomatic complexity is used as a metric for the complexity of tests required and we see it often in controllers. (see https://github.com/fzipp/gocyclo).

Maybe in future releases some of the logic can be factored out into functions.

To fix, add the // nolint:gocyclo comment to the function like so:

func List(ctx context.Context, cl *kadm.Client, accessControlList *AccessControlList) (*AccessControlList, error) { // nolint:gocyclo

[elohmrow]

🎉

[turkenh]

🙌