Skip to content

Conversation

@steffen-karlsson
Copy link

@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 <steffen.karlsson@maersk.com>
@steffen-karlsson steffen-karlsson force-pushed the 8732-enhance-kafkabridge branch from 40a7a35 to b8fea43 Compare March 12, 2024 19:05
@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
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
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
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
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.

3 participants