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

Upgrade terraform-provider-kafka to v0.5.4 #195

Merged
merged 4 commits into from
Aug 28, 2023

Conversation

rquitales
Copy link
Contributor

@rquitales rquitales commented Aug 23, 2023

This PR was generated via $ upgrade-provider pulumi/pulumi-kafka --kind=all, with some manual changes required.


@rquitales rquitales self-assigned this Aug 23, 2023
@rquitales rquitales requested a review from a team August 23, 2023 23:39
@github-actions
Copy link

Does the PR have any schema changes?

Does the PR have any schema changes?

Looking good! No breaking changes found.

New resources:

  • index/userScramCredential.UserScramCredential

Maintainer note: consult the runbook for dealing with any breaking changes.

@rquitales rquitales force-pushed the upgrade-terraform-provider-kafka-to-v0.5.4 branch from 2054e1f to 9e97c2e Compare August 25, 2023 03:20
Copy link
Member

@iwahbe iwahbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add token auto-mapping instead of more explicit resources?

@rquitales
Copy link
Contributor Author

rquitales commented Aug 25, 2023

Can we add token auto-mapping instead of more explicit resources?

Done! And removed the new resource from resource.go to test that it works as expected as well.

@rquitales rquitales requested a review from iwahbe August 25, 2023 05:24
Copy link

@thomas11 thomas11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like lost a number of examples, e.g. in sdk/dotnet/Acl.cs, sdk/dotnet/Quota.cs, sdk/dotnet/Topic.cs, sdk/go/kafka/acl.go, etc.

That's a bit of a regression for our users. Can we try and see what happened?

Copy link
Member

@iwahbe iwahbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you confirm that the missing docs were not removed upstream. If they should be in this provider, please make an issue in pulumi-kafka and add it to pulumi/pulumi-terraform-bridge#1290. Lets not block on the missing docs.

LGTM

@iwahbe
Copy link
Member

iwahbe commented Aug 28, 2023

Docs regression seems to be caused by Mongey/terraform-provider-kafka@8b8961d.

@iwahbe iwahbe merged commit 09f01ab into master Aug 28, 2023
15 checks passed
@iwahbe iwahbe deleted the upgrade-terraform-provider-kafka-to-v0.5.4 branch August 28, 2023 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade terraform-provider-kafka to v0.5.4 scram user creation (Upgrade terraform-provider-kafka to v0.5.3)
4 participants