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

#8732 Enhance KafkaBridge resource with consumer inactivity timeout and HTTP consumer/producer parts enablement #882

Merged
merged 1 commit into from
May 9, 2024

Conversation

steffen-karlsson
Copy link
Contributor

@steffen-karlsson steffen-karlsson commented Mar 12, 2024

Enabling ENVs as part of changes in: strimzi/strimzi-kafka-operator#9820 related to changes in improvement: strimzi/strimzi-kafka-operator#8732

…nd HTTP consumer/producer parts enablement

Signed-off-by: Steffen Karlsson <[email protected]>
@scholzj scholzj added this to the 0.29.0 milestone Mar 12, 2024
@scholzj scholzj requested a review from ppatierno March 12, 2024 22:40
@ppatierno
Copy link
Member

@steffen-karlsson thanks for the PR. Is it ready for review then? Could you move out of "Draft"? I think there is no that much to add, right?

@steffen-karlsson
Copy link
Contributor Author

Hello @ppatierno there is a few questions and concerns from my side in strimzi/strimzi-kafka-operator#8732 and @scholzj also left some comments here: strimzi/strimzi-kafka-operator#9820 which is related

@steffen-karlsson steffen-karlsson marked this pull request as ready for review March 13, 2024 15:33
@ppatierno
Copy link
Member

@steffen-karlsson any news on this PR now that the corresponding proposal was accepted?

@steffen-karlsson
Copy link
Contributor Author

@steffen-karlsson any news on this PR now that the corresponding proposal was accepted?

I've had a lot of things lately, expected to modify it next week. Sorry for the delay

@steffen-karlsson
Copy link
Contributor Author

@ppatierno strimzi/strimzi-kafka-operator#9820 and this one is now ready. Apologies for the delay.

Copy link
Member

@ppatierno ppatierno left a comment

Choose a reason for hiding this comment

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

LGTM

@ppatierno ppatierno requested a review from scholzj May 8, 2024 14:48
@scholzj scholzj merged commit 3c705ac into strimzi:main May 9, 2024
11 checks passed
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.

None yet

3 participants