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

(aws-msk): Missing L2 Cluster construct in 2.0 RC #15040

Closed
blaenk opened this issue Jun 8, 2021 · 2 comments
Closed

(aws-msk): Missing L2 Cluster construct in 2.0 RC #15040

blaenk opened this issue Jun 8, 2021 · 2 comments
Assignees
Labels
@aws-cdk/aws-msk Related to Amazon Managed Streaming for Apache Kafka (Amazon MSK) bug This issue is a bug. needs-triage This issue or PR still needs to be triaged.

Comments

@blaenk
Copy link
Contributor

blaenk commented Jun 8, 2021

I'm not sure if this is a bug or not, I'm not sure what the strategy is with version 2 RCs, maybe you're not merging in new features since the 2.0 RC came out?

Specifically, it seems that PR #9908 which added the MSK L2 construct for a Cluster and was released as part of 1.103.0 never got into the RC line, but maybe this is to be expected and many other changes aren't making it into 2 RC?

Reproduction Steps

$ curl -OJLR $(npm view aws-cdk-lib@2.0.0-rc.6 dist.tarball)
$ tar xvf aws-cdk-lib-2.0.0-rc.6.tgz
$ find package -path '*aws-msk*' -name '*cluster*'

# I think this is where it should be based on
# https://github.com/aws/aws-cdk/blob/v2.0.0-rc.6/packages/%40aws-cdk/aws-msk/lib/cluster.ts

$ stat package/lib/aws-msk/lib/cluster.js

What did you expect to happen?

I expected the L2 MSK Cluster construct to exist in the 2 RC line.

What actually happened?

It's not there.

Environment

Any.

Other


This is 🐛 Bug Report

@blaenk blaenk added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Jun 8, 2021
@github-actions github-actions bot added the @aws-cdk/aws-msk Related to Amazon Managed Streaming for Apache Kafka (Amazon MSK) label Jun 8, 2021
@otaviomacedo
Copy link
Contributor

Hi, @blaenk

This is expected. Version 2 of the CDK is still in developer preview. During this phase, it won't be possible to use experimental modules. So you basically have two options: use v1 or wait until v2 is released to general availability later this year.

@github-actions
Copy link

github-actions bot commented Jun 9, 2021

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-msk Related to Amazon Managed Streaming for Apache Kafka (Amazon MSK) bug This issue is a bug. needs-triage This issue or PR still needs to be triaged.
Projects
None yet
Development

No branches or pull requests

2 participants