Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add MSK/Kafka support to the ACK list #901

Closed
1 task
calliope7 opened this issue Jan 4, 2024 · 2 comments · Fixed by #903
Closed
1 task

Add MSK/Kafka support to the ACK list #901

calliope7 opened this issue Jan 4, 2024 · 2 comments · Fixed by #903
Assignees
Labels

Comments

@calliope7
Copy link

Describe the feature

Kafka ACK version 0.0.1 appears to be available now but it is not accessible from Ack-addons in the serviceMappings.ts file. The feature would add Kafka as an Ack-addon to enable control of Managed Kafka from a K8s cluster.

Use Case

I'm trying to prove out a framework which needs to run on K8s but could well benefit from leveraging AWS managed services. The framework leverages Kafka. Leveraging the CDK-EKS blueprints looks like a good way to build stable K8s environments within the AWS environment and this represents a good opportunity to let AWS do the undifferentiated heavy lifting!

Proposed Solution

The aws-controller-k8s team has already build the required MSK controller.. Thus this looks more or less like a plumbing job to connect the dots from the CDK-EKS blueprints framework.

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request

CDK version used

2.99.1 (build b2a895e)

EKS Blueprints Version

1.13.1

Node.js Version

v20.10.0

Environment details (OS name and version, etc.)

Windows 10 Pro - version 22H2 - OS build 19045.3803

@elamaran11
Copy link
Collaborator

@calliope7 We will add this to our next blueprints release.

@elamaran11 elamaran11 self-assigned this Jan 4, 2024
@elamaran11 elamaran11 linked a pull request Jan 4, 2024 that will close this issue
shapirov103 added a commit that referenced this issue Jan 4, 2024
@calliope7
Copy link
Author

Nice one. I'll test it when the blueprints release is done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants