-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Fix] correct bridge env vars names for consumer and producer enabled #10120
Conversation
Signed-off-by: David Kornel <kornys@outlook.com>
Its great that you noticed this issue. Thanks. But maybe its best to first start a discussion on Slack and see what @ppatierno as the SME thinks about it rather than madly opening conflicting PRs all over the place. |
Well I wanted to help and save time so I have opened PR in both repositories. SME can just pick what is better solution and close the second one. So I don't see any issue with this approach... |
cluster-operator/src/main/java/io/strimzi/operator/cluster/model/KafkaBridgeCluster.java
Outdated
Show resolved
Hide resolved
I prefer this way, so fixing in the operator. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (after fixing the comment from Lukas)
Signed-off-by: David Kornel <kornys@outlook.com>
/azp run regression |
Azure Pipelines successfully started running 1 pipeline(s). |
Type of change
Select the type of your PR
Description
We can rename env vars there or in bridge. please see this PR
related bridge issue strimzi/strimzi-kafka-bridge#900