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

feat(kafkatopic): creation simplification #1357

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

ivan-savciuc
Copy link
Contributor

@ivan-savciuc ivan-savciuc commented Sep 14, 2023

About this change—what it does

we already have retry logic in the client no need for the retry logic in the TF and fixes acceptance tests

@ivan-savciuc ivan-savciuc added the no changelog No changelog entries are required for this PR label Sep 14, 2023
@ivan-savciuc ivan-savciuc requested a review from a team September 14, 2023 13:36
@Serpentiel Serpentiel force-pushed the ivans-kafka-topic-create-simplifications branch from 2bc6bf4 to 0fe89f6 Compare September 14, 2023 18:16
@ivan-savciuc ivan-savciuc force-pushed the ivans-kafka-topic-create-simplifications branch from 0fe89f6 to a6676d5 Compare September 14, 2023 20:14
Serpentiel
Serpentiel previously approved these changes Sep 18, 2023
Copy link
Contributor

@Serpentiel Serpentiel left a comment

Choose a reason for hiding this comment

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

lgtm, @byashimov wdyt?

@Serpentiel Serpentiel force-pushed the ivans-kafka-topic-create-simplifications branch from a6676d5 to daa9322 Compare September 18, 2023 07:52
@ivan-savciuc
Copy link
Contributor Author

Tests are still failing, lets catch up to discuss this issue.

internal/sdkprovider/service/kafkatopic/kafka_topic.go Outdated Show resolved Hide resolved
@ivan-savciuc ivan-savciuc force-pushed the ivans-kafka-topic-create-simplifications branch from daa9322 to d412c03 Compare September 20, 2023 08:09
@ivan-savciuc ivan-savciuc dismissed byashimov’s stale review September 20, 2023 08:10

deleted all the cache invalidation changes out from this PR

@ivan-savciuc ivan-savciuc force-pushed the ivans-kafka-topic-create-simplifications branch from d412c03 to a3191a3 Compare September 27, 2023 10:57
@ivan-savciuc ivan-savciuc force-pushed the ivans-kafka-topic-create-simplifications branch from a3191a3 to 876143e Compare September 27, 2023 10:58
Copy link
Contributor

@byashimov byashimov left a comment

Choose a reason for hiding this comment

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

lgtm

@ivan-savciuc ivan-savciuc merged commit fb1e435 into main Sep 28, 2023
10 checks passed
@ivan-savciuc ivan-savciuc deleted the ivans-kafka-topic-create-simplifications branch September 28, 2023 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog No changelog entries are required for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants